Thanks for the comments,

I will send an updated version of this patch with extra local handling
of Makefiles and where it makes sense also for some other files.

Danny

Am 06.02.2013 10:14, schrieb Josh Durgin:
> I like the cleanup in general, but non-generic entries should have
> a leading /, otherwise they match files with the same name in
> subdirectories as well.
> 
> For example, putting Makefile in .gitignore will also ignore static
> makefiles in the qa subdirectories. Some of the existing entries are
> already missing this leading /.
> 
> On 02/04/2013 09:22 AM, Danny Al-Gaaf wrote:
>> Cleanup old entries and sort the existing entries. Remove no longer
>> existing entries/files. Move src/* entries from top level .gitignore
>> to the correct subdir .gitignore file.
>>
>> Signed-off-by: Danny Al-Gaaf <[email protected]>
>> ---
>>   .gitignore           |  78 ++++++++++-------------
>>   doc/.gitignore       |   3 +-
>>   src/.gitignore       | 173
>> +++++++++++++--------------------------------------
>>   src/.gitignore.orig  | 142 ++++++++++++++++++++++++++++++++++++++++++
> 
> No need for this file
> 
>>   src/gtest/.gitignore |   2 +
>>   5 files changed, 221 insertions(+), 177 deletions(-)
>>   create mode 100644 src/.gitignore.orig
>>   create mode 100644 src/gtest/.gitignore
>>

--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to