Re: build/ .gitignore pattern matches files in repo

[David Kastrup]

markdblackwell <address@hidden> writes:

>> Any chance to see an uncensored version of this on the actual mailing list?
>> --David Kastrup 
>
> Thanks for that. :) Now, I've added the archive to my toolbar for future
> checking.
>
> The full text follows (without HTML tags; I had used {raw}). 
>
> Currently, the top level .gitignore (of LilyPond's repository) contains this
> pattern:
>
> build/
>
> I found no pattern negating this, in any .gitignore file (at any directory
> level).
>
> Currently, the repository on the server contains files (see below) which
> match this pattern. Because of this, Git will ignore changes to those files.
> Therefore:
>
> * Should the pattern be changed to /build/ , so it doesn't match the files?
>
> * Should the matching files be removed?
>
> * Should the .gitignore pattern be deleted?
>
> * (If the present, peculiar situation is exactly what is desired, then)
> should the top level .gitignore include some comment explaining it?
>
> The following files match the pattern in the top-level .gitignore:
>
> $ find scripts/build | sort
> scripts/build
> scripts/build/GNUmakefile
> scripts/build/pl
> scripts/build/pl/help2man.pl
> scripts/build/pl/mf2pt1.pl
> scripts/build/py
> scripts/build/py/bib2texi.py
> scripts/build/py/catmidi.py
> scripts/build/py/create-version-itexi.py
> scripts/build/py/create-weblinks-itexi.py
> scripts/build/py/extract_texi_filenames.py
> scripts/build/py/gen-emmentaler-scripts.py
> scripts/build/py/genicon.py
> scripts/build/py/grand-replace.py
> scripts/build/py/html-to-texi.py
> scripts/build/py/install.py
> scripts/build/py/lilypond-words.py
> scripts/build/py/lys-to-tely.py
> scripts/build/py/makesnippets.py
> scripts/build/py/make-version.py
> scripts/build/py/mass-link.py
> scripts/build/py/mf-to-table.py
> scripts/build/py/output-distance.py
> scripts/build/py/pytt.py
> scripts/build/py/relative.py
> scripts/build/py/texi2omf.py
> scripts/build/py/text2html.py
> scripts/build/py/website_post.py
> scripts/build/py/www_post.py
> scripts/build/py/yyout2grammar.py
> scripts/build/sh
> scripts/build/sh/install-info-html.sh
> scripts/build/sh/run-and-check.sh
> scripts/build/txt
> scripts/build/txt/website-known-missing-files.txt
>
> $ du -ch `find scripts/build`
> 284K  total

I think this is most likely an oversight.

Reading "man gitignore" shows that "build" should work, but maybe
"/build/" is more explicit.  The alternative is that we find a new name
for scripts/build.  But it is rather descriptive.  Which is exactly the
reason build in the top directory is descriptive for the build process
as well.

I'll do some experiments and submit a patch, likely using /build/ as
pattern.

-- 
David Kastrup