[Git][lintian/lintian][master] 4 commits: Replace egrep and fgrep in all test suite dummy packages with grep -E/-F
Axel Beckert pushed to branch master at lintian / lintian Commits: 99fb9320 by Axel Beckert at 2022-09-11T01:08:19+02:00 Replace egrep and fgrep in all test suite dummy packages with grep -E/-F - - - - - 96735a72 by Axel Beckert at 2022-09-11T01:08:19+02:00 Add build-dependencies of the test suite They have been forgotten in commit 258fb7fb. Taken from debian/tests/control and added "" to each. - - - - - 29cb2a01 by Axel Beckert at 2022-09-11T01:13:05+02:00 Fix test broken by dpatch removal - - - - - c4998558 by Axel Beckert at 2022-09-11T01:13:29+02:00 Fix test broken by updating the list of virtual packages - - - - - 30 changed files: - debian/control - t/recipes/checks/build-systems/debhelper/maintainer-script/token/scripts-maintainer-general/build-spec/debian/postinst - t/recipes/checks/debhelper/debhelper-no-depends/eval/hints - t/recipes/checks/debhelper/legacy-maintainer-scripts/build-spec/debian/preinst - t/recipes/checks/debian/changelog/legacy-maintainer-scripts/build-spec/debian/preinst - t/recipes/checks/debian/copyright/legacy-maintainer-scripts/build-spec/debian/preinst - t/recipes/checks/debian/watch/legacy-maintainer-scripts/build-spec/debian/preinst - t/recipes/checks/documentation/files-uses-dpkg-database-directly/build-spec/orig/test-1 - t/recipes/checks/fields/dm-upload-allowed/legacy-maintainer-scripts/build-spec/debian/preinst - t/recipes/checks/fields/mail-address/legacy-maintainer-scripts/build-spec/debian/preinst - t/recipes/checks/fields/package-relations/fields-depends-general/eval/hints - t/recipes/checks/fields/standards-version/legacy-maintainer-scripts/build-spec/debian/preinst - t/recipes/checks/files/contents/files-uses-dpkg-database-directly/build-spec/orig/test-1 - t/recipes/checks/files/contents/legacy-maintainer-scripts/build-spec/debian/preinst - t/recipes/checks/files/contents/scripts-maintainer-general/build-spec/debian/postinst - t/recipes/checks/init-d/legacy-maintainer-scripts/build-spec/debian/preinst - t/recipes/checks/init-d/maintainer-script/legacy-maintainer-scripts/build-spec/debian/preinst - t/recipes/checks/maintainer-scripts/empty/legacy-maintainer-scripts/build-spec/debian/preinst - t/recipes/checks/maintainer-scripts/generated/scripts-maintainer-general/build-spec/debian/postinst - t/recipes/checks/maintainer-scripts/helper/dpkg/scripts-maintainer-general/build-spec/debian/postinst - t/recipes/checks/maintainer-scripts/ldconfig/legacy-maintainer-scripts/build-spec/debian/preinst - t/recipes/checks/maintainer-scripts/mknod/scripts-maintainer-general/build-spec/debian/postinst - t/recipes/checks/maintainer-scripts/temporary-files/legacy-maintainer-scripts/build-spec/debian/preinst - t/recipes/checks/maintainer-scripts/temporary-files/scripts-maintainer-general/build-spec/debian/postinst - t/recipes/checks/md5sums/legacy-maintainer-scripts/build-spec/debian/preinst - t/recipes/checks/nmu/legacy-maintainer-scripts/build-spec/debian/preinst - t/recipes/checks/script/syntax/legacy-maintainer-scripts/build-spec/debian/preinst - t/recipes/checks/script/syntax/scripts-maintainer-general/build-spec/debian/postinst - t/recipes/checks/scripts/legacy-maintainer-scripts/build-spec/debian/preinst - t/recipes/checks/scripts/legacy-maintainer-scripts/eval/hints The diff was not included because it is too large. View it on GitLab: https://salsa.debian.org/lintian/lintian/-/compare/be8c92f08d16a4a151e8f81d1c1d5922ab5b7ec8...c49985584b737628c6dde27b6d71e84dfa715ae9 -- View it on GitLab: https://salsa.debian.org/lintian/lintian/-/compare/be8c92f08d16a4a151e8f81d1c1d5922ab5b7ec8...c49985584b737628c6dde27b6d71e84dfa715ae9 You're receiving this email because of your account on salsa.debian.org.
[Git][lintian/lintian][master] .gitignore: Also ignore debian/*.debhelper files and drop wrong trailing slash for doc/lintian.html
Axel Beckert pushed to branch master at lintian / lintian Commits: e048f0e7 by Axel Beckert at 2022-09-10T17:52:51+02:00 .gitignore: Also ignore debian/*.debhelper files and drop wrong trailing slash for doc/lintian.html Merge potentially three very similar ignore rules into a single one. - - - - - 1 changed file: - .gitignore Changes: = .gitignore = @@ -1,6 +1,5 @@ /*-stamp -/debian/.debhelper/ -/debian/*.debhelper.* +/debian/*.debhelper* /debian/debhelper-build-stamp /debian/files /debian/lintian.substvars @@ -10,7 +9,7 @@ /doc/api.html/ /doc/README /doc/help.tmp -/doc/lintian.html/ +/doc/lintian.html /doc/lintian.txt /l10n/ /man/man*/ View it on GitLab: https://salsa.debian.org/lintian/lintian/-/commit/e048f0e74ff77cd4853e1f35620b04f8ca48405b -- View it on GitLab: https://salsa.debian.org/lintian/lintian/-/commit/e048f0e74ff77cd4853e1f35620b04f8ca48405b You're receiving this email because of your account on salsa.debian.org.
Are we back on adding debian/changelog entries with commits and running test suite at build time without discussion?
Hi, I was very surprised to see debian/changelog entries added with commits despite the * WIP (generated at release time: please do not add entries below). clearly being at top of debian/changelog. This breaks our current "gbp dch" workflow severly as it cannot determine anymore which commits are needed to be added to debian/changelog and which not. Bastien: I'm really hapy that you are back on Lintian development and especially thankful for your work on performance and the sliding window implementation stuff, but I would have been more happy if you first discussed or at least announced such a severe change in our workflow. (I'm fine with both workflows, but we should strictly abide to one of them, otherwise our changelog becomes highly unreliable.) So let's please first discuss which workflow we'll use in the future (with reasoning) before changing it again. So what's your reason for editing the debian/changelog directly? BTW: Your commit 198c3645c88496bcb08614e64e2892d01c42f13a already shows what happens if we don't abide to one of the workflows. Additionally these older commits are currently missing in the current debian/changelog entry due to this unannounced workflow change: * 9279fb89fd7fbad7963c8d3ba8b8f8fea58e3c1d * acb2073d805aead92080ddcadfad74e83db8401a I've added the according debian/changelog entries now. Please stop adding new debian/changelog entries until shortly before releasing or uploading or until we _decide_ together to change our workflow back to editing debian/changelog with each commit. I've also added a more prominent note than the existing one as that one didn't seem to be read. We can remove it (and update the WIP note) if we decide to switch back to editing debian/changelog with each commit again. The same counts for running the test suite at build time. The test suite IMHO currently takes way too long for running at build time. IMHO we first need revert splitting up running separate tests for each tag back to running combined tests again to get down the test suite run-time before we can really enable running tests at build time again. In addition to that we need to document how to build lintian without running the test suite. Waiting 40 minutes to just test some changes is inacceptable. I've though haven't reverted that change. I will like add proper documentation for how to build lintian without waiting 40 minutes for a build. Besides: salsa.debian.org is spelled "Salsa", not "salci". I've fixed that, too, in debian/changelog. Regards, Axel -- ,''`. | Axel Beckert , https://people.debian.org/~abe/ : :' : | Debian Developer, ftp.ch.debian.org Admin `. `' | 4096R: 2517 B724 C5F6 CA99 5329 6E61 2FF9 CD59 6126 16B5 `-| 1024D: F067 EA27 26B9 C3FC 1486 202E C09E 1D89 9593 0EDE signature.asc Description: PGP signature
Re: [lintian] spelling: Add several corrections
Hi Francois-Xavier, Francois-Xavier Le Bail wrote: > The attached patch add several corrections to data/spelling/corrections. Just ran into a false positive included in this commit: > +licence||license "licence" is a very common and valid variant of "license", see e.g. https://en.wiktionary.org/wiki/licence In UK, Canada, Australia, South Africa, Ireland, New Zealand, Malaysia and Singapore it is even the standard spelling. (And hence it is even likely the original spelling.) Will revert that part. Regards, Axel -- ,''`. | Axel Beckert , https://people.debian.org/~abe/ : :' : | Debian Developer, ftp.ch.debian.org Admin `. `' | 4096R: 2517 B724 C5F6 CA99 5329 6E61 2FF9 CD59 6126 16B5 `-| 1024D: F067 EA27 26B9 C3FC 1486 202E C09E 1D89 9593 0EDE signature.asc Description: PGP signature
