bug#20710: gitignore patterns for ps, html etc are too broad

bug#20710: gitignore patterns for ps, html etc are too broad

[Glenn Morris]

Package: emacs
Severity: minor

git ls-files -i --exclude-standard
admin/unidata/copyright.html
etc/ps-prin0.ps
etc/ps-prin1.ps
leim/MISC-DIC/CTLau-b5.html
leim/MISC-DIC/CTLau.html
nextstep/Cocoa/Emacs.base/Contents/Resources/Credits.html
test/indent/ps-mode.ps
test/indent/sgml-mode-attribute.html

Due to the presence of

*.html
*.ps

in .gitignore.

With bzr, those forms were the correct:

doc/**/*.ps
doc/**/*.html

A comment at the start of .gitignore says it avoids '**' for the sake of
old git versions.

Explicitly listing all current and future versioned html and ps files
for the sake of old git versions seems like a PITA.

bug#20710: gitignore patterns for ps, html etc are too broad

[Eli Zaretskii]

> From: Glenn Morris <rgm@gnu.org>
> Date: Mon, 01 Jun 2015 12:10:03 -0400
> 
> git ls-files -i --exclude-standard
> admin/unidata/copyright.html
> etc/ps-prin0.ps
> etc/ps-prin1.ps
> leim/MISC-DIC/CTLau-b5.html
> leim/MISC-DIC/CTLau.html
> nextstep/Cocoa/Emacs.base/Contents/Resources/Credits.html
> test/indent/ps-mode.ps
> test/indent/sgml-mode-attribute.html
> 
> Due to the presence of
> 
> *.html
> *.ps
> 
> in .gitignore.
> 
> With bzr, those forms were the correct:
> 
> doc/**/*.ps
> doc/**/*.html
> 
> A comment at the start of .gitignore says it avoids '**' for the sake of
> old git versions.
> 
> Explicitly listing all current and future versioned html and ps files
> for the sake of old git versions seems like a PITA.

There aren't too many subdirectories in doc/, so perhaps something
like

  doc/emacs/*.html
  doc/lispintro/*.html
  doc/lispref/*.html
  doc/man/*.html
  doc/mis/*.html

and similarly with *.ps will do?

OTOH, is "**" supported by Git 1.7.1?

bug#20710: gitignore patterns for ps, html etc are too broad

[Andreas Schwab]

Eli Zaretskii <eliz@gnu.org> writes:

> There aren't too many subdirectories in doc/, so perhaps something
> like
>
>   doc/emacs/*.html
>   doc/lispintro/*.html
>   doc/lispref/*.html
>   doc/man/*.html
>   doc/mis/*.html

doc/*/*.html should work too.

Andreas.

-- 
Andreas Schwab, schwab@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."

bug#20710: gitignore patterns for ps, html etc are too broad

[Eli Zaretskii]

> From: Andreas Schwab <schwab@linux-m68k.org>
> Cc: Glenn Morris <rgm@gnu.org>,  20710@debbugs.gnu.org
> Date: Tue, 02 Jun 2015 00:55:24 +0200
> 
> Eli Zaretskii <eliz@gnu.org> writes:
> 
> > There aren't too many subdirectories in doc/, so perhaps something
> > like
> >
> >   doc/emacs/*.html
> >   doc/lispintro/*.html
> >   doc/lispref/*.html
> >   doc/man/*.html
> >   doc/mis/*.html
> 
> doc/*/*.html should work too.

Thanks, I pushed that.

bug#20710: gitignore patterns for ps, html etc are too broad

[Andreas Schwab]

Glenn Morris <rgm@gnu.org> writes:

> Due to the presence of
>
> *.html
> *.ps
>
> in .gitignore.

The pattern could be moved to doc/.gitignore.

Andreas.

-- 
Andreas Schwab, schwab@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."