Your message dated Fri, 09 May 2008 13:02:03 +0000 with message-id <[EMAIL PROTECTED]> and subject line Bug#468774: fixed in dh-make 0.45 has caused the Debian Bug report #468774, regarding dh-make: please use finer error checking granularity than -make distclean to be marked as done.
This means that you claim that the problem has been dealt with. If this is not the case it is now your responsibility to reopen the Bug report if necessary, and/or fix the problem forthwith. (NB: If you are a system administrator and have no idea what this message is talking about, this may indicate a serious mail system misconfiguration somewhere. Please contact [EMAIL PROTECTED] immediately.) -- 468774: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=468774 Debian Bug Tracking System Contact [EMAIL PROTECTED] with problems
--- Begin Message ---Package: lintian Version: 1.23.10 Severity: wishlist A lot of packages, particularly those with upstreams that use the GNU autotools, have a very nasty habit of running the upstream clean rule while utterly ignoring any errors from it. This is very, very bad. Please add a check for it. An egrep regex to catch this is probably about as simple as: '\t[[:space:]]*-(\$[({]MAKE[)}]|make)[[:space:]].*(dist)?clean' See transcript of IRC discussion below for further elaboration of rationale. 00:35 < Overfiend> joshk: What didn't you like about my changes to debian/rules? 00:36 < Overfiend> a configure target for autofoo-using packages is common practice AFAICT 00:36 < Overfiend> because of the problem of getting a properly cleaned tree 00:36 < Overfiend> if you were to assert that configure getting run *twice* for every package build from a freshly-unpacked source were a bit stupid, I'd agree. 00:36 < Overfiend> But that stupidity appears to be forced on us by GNU autofoo. 00:37 < joshk> yes - my scripts run a clean and then a build.. 00:37 < Overfiend> your scripts? debuild does that too. dpkg-buildpackage might as well, I don't remember. 00:37 < Overfiend> it's the Right Thing to do. 00:37 < vorlon> Overfiend: meh? Why would it be run twice? 00:37 < Overfiend> Builds should always start from a known state. 00:37 < Overfiend> vorlon: because debuild invokes debian/rules clean before debian/rules binary 00:38 < Overfiend> or dpkg-buildpackage does 00:38 < Overfiend> as I said, I don't remember 00:38 < doogie> why does clean depend on configure? 00:38 < vorlon> Overfiend: there's no reason clean should need to depend on configure 00:38 < Overfiend> can't "make clean" if there's no Makefile... 00:38 < vorlon> not with sane autotoolery 00:38 < doogie> then don't 00:38 < vorlon> yes, so you type "-make clean" instead 00:38 < Overfiend> UGH! 00:38 < doogie> what he said 00:38 < joshk> -$(MAKE) distclean, here 00:38 * Overfiend vomits across the room 00:38 < Overfiend> that's fucking moronic 00:38 < doogie> no, this is how it's done 00:38 < joshk> Overfiend: wow. have you considered entering a contest? 00:38 < Overfiend> "I don't care why make clean failed. Ignore it." 00:39 < vorlon> fine. [ ! -f Makefile ] || make clean 00:39 < vorlon> :P 00:39 < Overfiend> That's like swatting a fly with a neutron star. 00:39 < doogie> however, this is why I hate build systems that don't support a build dir. 00:39 < joshk> build system? what build system? :| 00:39 < Overfiend> vorlon: that's a bit more reasonable. 00:39 < joshk> vorlon: hmm, good idea 00:39 < doogie> ifeq (Makefile,$(wildcard Makefile)) 00:40 < doogie> make clean 00:40 < doogie> endif 00:40 < Overfiend> "Ignore all errors from make on the assumption that the only reason it would fail is because there's no Makefile" is mondo stupid. 00:40 < joshk> well, the reason i chose to do it my way is because it's that way in every single one of my autofoo using packages currently 00:40 < doogie> my way is more efficient. less forking. 00:40 < joshk> so even if you were right, i'd end up having to orchestrate that change over all my packages anyway.. 00:40 < joshk> IOW: someday. 00:41 < Overfiend> I don't care which approach, vorlon's or doogie's, people use. but -$(MAKE) {dist,}clean is just unholy bad. 00:41 < vorlon> doogie: doesn't every line get passed to sh for execution? If [ is a builtin, there's no difference in fork count (Connection timed out)] 00:42 < Overfiend> Recall that people mindlessly ignoring error conditions is why we see most security advisories. 00:42 < doogie> vorlon: try again. if the Makefile doesn't exist, $(wildcard) expands to nothing, so the line isn't sent to the shell 00:42 < vorlon> ah. 00:42 < vorlon> right. 00:42 < joshk> Overfiend: a ton of people, not just myself, use a variation of -make clean - you may want to submit a lintian check for it... 00:42 < doogie> and you have to test for the file's existance anyways. 00:42 < joshk> sounds like something that would get in. and it would compel me to do it more quickly :) 00:42 < doogie> altho, it does make the file harder to read 00:43 < Overfiend> joshk, doogie, vorlon: permission to quote you guys in my bug report. 00:43 < joshk> Overfiend: granted 00:43 < Overfiend> I'd rather paste this discussion than rewrite it. 00:43 < vorlon> Overfiend: as long as you leave out the fart noises [Read error: 104 (Connection reset by peer)] 00:43 < Overfiend> vorlon: oh, very well 00:43 < doogie> you can always quote my makefile foo, if it is meant to scare and intimidate 00:43 < Overfiend> doogie: :) -- System Information: Debian Release: testing/unstable APT prefers unstable APT policy: (500, 'unstable'), (500, 'stable') Architecture: powerpc (ppc) Shell: /bin/sh linked to /bin/bash Kernel: Linux 2.6.9-powerpc-smp Locale: LANG=C, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8) Versions of packages lintian depends on: ii binutils 2.16.1-2 The GNU assembler, linker and bina ii diffstat 1.41-1 produces graph of changes introduc ii file 4.12-1 Determines file type using "magic" ii gettext 0.14.5-2 GNU Internationalization utilities ii intltool-debian 0.30+20040213 Help i18n of RFC822 compliant conf ii man-db 2.4.3-1 The on-line manual pager ii perl [libdigest-md5-perl] 5.8.7-4 Larry Wall's Practical Extraction lintian recommends no packages. -- no debconf information
--- End Message ---
--- Begin Message ---Source: dh-make Source-Version: 0.45 We believe that the bug you reported is fixed in the latest version of dh-make, which is due to be installed in the Debian FTP archive: dh-make_0.45.dsc to pool/main/d/dh-make/dh-make_0.45.dsc dh-make_0.45.tar.gz to pool/main/d/dh-make/dh-make_0.45.tar.gz dh-make_0.45_all.deb to pool/main/d/dh-make/dh-make_0.45_all.deb A summary of the changes between this version and the previous one is attached. Thank you for reporting the bug, which will now be closed. If you have further comments please address them to [EMAIL PROTECTED], and the maintainer will reopen the bug report if appropriate. Debian distribution maintenance software pp. Craig Small <[EMAIL PROTECTED]> (supplier of updated dh-make package) (This message was generated automatically at their request; if you believe that there is a problem with it please contact the archive administrators by mailing [EMAIL PROTECTED]) -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Format: 1.7 Date: Fri, 09 May 2008 22:41:38 +1000 Source: dh-make Binary: dh-make Architecture: source all Version: 0.45 Distribution: unstable Urgency: low Maintainer: Craig Small <[EMAIL PROTECTED]> Changed-By: Craig Small <[EMAIL PROTECTED]> Description: dh-make - tool that converts source archives into Debian package source Closes: 352017 383495 409694 422203 423707 424953 428228 431316 431316 432667 445678 445766 445768 445773 447752 448854 448854 453896 453896 464948 468774 Changes: dh-make (0.45) unstable; urgency=low . [ Craig Small ] * Finally (I hope) fixed manpage.xml.ex Closes: #383495 * lib control uses binary:Version Closes: #428228 * Added Homepage: tag to control file Closes: #445678 * CMakeList.txt is not a document Closes: #422203 * Offering use of dpatch. Closes: #409694 * init.d.lsb fixes from JFS Closes: #445766, #445768, #445773 * Crosscompile patch added Closes: #423707 * init.d.lsb stop_server() errcode fixed Closes: #448854 * Fixed init.d.ex running Closes: #453896 * Typo in dh_make org -> orig Closes: #431316 * Changed package-default.ex to package.default.ex Closes: #424953 * Updated to debhelper version 6 in package and templates * Added examples to man page Closes: #352017 * Fixed distclean target Closes: #432667 [ Tobias Toedter ] * Fix typo in dh_make message. Closes: #431316 * Fix typo in init.d.lsb, thanks to Yauhen Kharuzhy. Closes: #448854 * Correct errors in init.d, thanks to Marcos Talau. Closes: #453896 * Ignore a missing Makefile in clean targets, but not other errors. Thanks to Sune Vuorela. Closes: #468774 * Update to Standards-Version 3.7.3 in generated templates, thanks to Nelson A. de Oliveira. Closes: #464948 * Use untranslated dates for manpages. Closes: #447752 Files: 29e767bf2b410e15b275890c8c716af7 482 devel optional dh-make_0.45.dsc 8882b7a537928ddb8e798ede0a306e40 37978 devel optional dh-make_0.45.tar.gz 265b410142c174ea80e51280c7e3728c 40634 devel optional dh-make_0.45_all.deb -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.6 (GNU/Linux) iD8DBQFIJEavx2zlrBLK36URAhQmAKCGYAG0DV9r4tTuZy236ZGke19R2wCffzYV zWIuNw7CZUQSYSyru+xJTnw= =He56 -----END PGP SIGNATURE-----
--- End Message ---
