[cairo] [PATCH] gitignore: negate gitignore for static pkgconfig files

Behdad Esfahbod behdad at behdad.org
Mon Mar 18 08:34:30 PDT 2013


On 13-03-18 09:03 AM, Uli Schlachter wrote:
> Hi,
> 
> On 11.03.2013 16:24, Matt Sealey wrote:
>> Commit 781f253 adds a rule cairo-*.*.* to .gitignore in the root dir.
>> Unfortunately this matches several src/cairo-*.pc.in files in the
>> src directory.

All you need is to change that rule to "/cairo-*.*.*".

>> The build system requires these files to be present, but the rule is
>> allowing them to be ignored. For example, when extracting a cgit
>> snapshot tarball and checking it into another git repository, these
>> files get left behind. Any accidental changes to these files will go
>> unnoticed by a 'git status' (possibly creating bad installs) and
>> any intentional changes could not be committed (git commit -a will
>> miss them, and every one will need to be forced). This is not really
>> desirable.
>>
>> We don't want to unignore *.pc.in here since there are many, many
>> autogenerated files with this name, and the cairo-*.*.* rule is in
>> general quite useful and doesn't warrant modification (although it
>> could be made a little more specific), so we just make these 4 files
>> a special case and negate the match with full filenames in src/.
>>
>> Signed-off-by: Matt Sealey <matt at genesi-usa.com>
>> ---
>>  src/.gitignore |    4 ++++
>>  1 file changed, 4 insertions(+)
>>
>> diff --git a/src/.gitignore b/src/.gitignore
>> index fd53c86..534263a 100644
>> --- a/src/.gitignore
>> +++ b/src/.gitignore
>> @@ -31,3 +31,7 @@ check-has-hidden-symbols.i
>>  check-link
>>  check-skiplist
>>  headers-standalone
>> +!cairo.pc.in
>> +!cairo-uninstalled.pc.in
>> +!cairo-features.pc.in
>> +!cairo-features-uninstalled.pc.in
> 
> Thanks, applied.
> 

-- 
behdad
http://behdad.org/


More information about the cairo mailing list