Junio C Hamano [off-list ref] writes:
Instead of crafting a new message, why not just stop passing IGNORE_ERRORS
and have add_file_to_cache() report the failure? That is:
if (add_file_to_cache(item->string, 0))
return -1;
Err, that should be exit(128) to mimic die().
That way, the user will get more useful diagnosis because there are
different reasons why an "add" may fail and we give different error
messages to them.