[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: build/ .gitignore pattern matches files in repo
From: |
David Kastrup |
Subject: |
Re: build/ .gitignore pattern matches files in repo |
Date: |
Mon, 28 Sep 2015 15:54:03 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.0.50 (gnu/linux) |
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
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: build/ .gitignore pattern matches files in repo,
David Kastrup <=