From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michael Tremer To: location@lists.ipfire.org Subject: Re: [PATCH 1/8] debian: Add all temporary files to Gitignore Date: Mon, 19 Apr 2021 14:10:55 +0100 Message-ID: <85231BA2-81AD-4E34-AD4E-427D267FE6E2@ipfire.org> In-Reply-To: <20210416130612.15959-2-valter.jansons@gmail.com> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============5551233537481080747==" List-Id: --===============5551233537481080747== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Hello, > On 16 Apr 2021, at 14:06, Valters Jansons wrot= e: >=20 > 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. >=20 > This commit ensures all temporary subdirectories, and additional > generic build artifact files, are ignored by Git. >=20 > The subdirectory exceptions to this rule are: >=20 > - d/patches/ which may be used by Quilt > considering the source format is '3.0 (quilt)', >=20 > - d/source/ for the format file, >=20 > - 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/RE= ADME.package-tests.rst >=20 > Signed-off-by: Valters Jansons > --- > debian/.gitignore | 9 +++++---- > 1 file changed, 5 insertions(+), 4 deletions(-) >=20 > 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 roo= t directory. It feels like a bit much to me. -Michael > *.debhelper > +*.log > *.substvars > +!/patches/ > +!/source/ > +!/tests/ > --=20 > 2.31.1 >=20 --===============5551233537481080747==--