Re: [PATCH 1/8] debian: Add all temporary files to Gitignore

[Michael Tremer <michael.tremer@ipfire.org>]

[-- Attachment #1: Type: text/plain, Size: 1510 bytes --]

Hello,

> On 16 Apr 2021, at 14:06, Valters Jansons <valter.jansons(a)gmail.com> wrote:
> 
> New packages have been added since the inception of the .gitignore and
> as a result during build we see directories such as location-importer/
> and files such as location-importer.debhelper.log.
> 
> This commit ensures all temporary subdirectories, and additional
> generic build artifact files, are ignored by Git.
> 
> The subdirectory exceptions to this rule are:
> 
> - d/patches/ which may be used by Quilt
>  considering the source format is '3.0 (quilt)',
> 
> - d/source/ for the format file,
> 
> - d/tests/ which may be used by autopkgtest
>  to specify what test suites exist for the source.
>  See: https://salsa.debian.org/ci-team/autopkgtest/-/raw/debian/5.16/doc/README.package-tests.rst
> 
> Signed-off-by: Valters Jansons <valter.jansons(a)gmail.com>
> ---
> debian/.gitignore | 9 +++++----
> 1 file changed, 5 insertions(+), 4 deletions(-)
> 
> diff --git a/debian/.gitignore b/debian/.gitignore
> index 0faf920..8190b92 100644
> --- a/debian/.gitignore
> +++ b/debian/.gitignore
> @@ -2,9 +2,10 @@
> /autoreconf.*
> /debhelper-build-stamp
> /files
> -/libloc/
> -/libloc-dev/
> -/libloc-perl/
> -/tmp
> +/*/

What is /*/ for? Why would we ignore any directory that is created in the root directory.

It feels like a bit much to me.

-Michael

> *.debhelper
> +*.log
> *.substvars
> +!/patches/
> +!/source/
> +!/tests/
> -- 
> 2.31.1
>