Bug#389684: new README does not close the bug
On Mon, Mar 31, 2008 at 07:45:37PM +0200, Carsten Hey wrote: > retitle 389684 deborphan: Please add a wrapper around dpkg -l | grep '^rc' Of course this would not be implemented as a simple shell-wrapper nor would it call dpkg. I want you to rename the bug (if you still feel the need of such an option) to be sure that you understand that it would behave exactly like a wrapper around the command that was mentioned in the README. Carsten -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
Bug#389684: new README does not close the bug
Comparision of both commands: Your proposal with false positives: $ dpkg -l | grep "rc " ii apt-src 0.25.1-0.1 manage Debian source packages rc binfmt-support 1.2.8 Support for extra binary formats rc libgtkhtml3.8-15 3.12.1-2 HTML rendering/editing library - runtime fil rc mono-common 1.2.2.1-1etch1 common files for Mono rc openvpn 2.0.9-4etch1 Virtual Private Network daemon ii sysv-rc 2.86.ds1-38 System-V-like runlevel change mechanism My proposal without false positives: $ dpkg -l | grep "^rc" rc binfmt-support 1.2.8 Support for extra binary formats rc libgtkhtml3.8-15 3.12.1-2 HTML rendering/editing library - runtime fil rc mono-common 1.2.2.1-1etch1 common files for Mono rc openvpn 2.0.9-4etch1 Virtual Private Network daemon -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
Bug#389684: new README does not close the bug
Hi, > The new README you propose does not address the bug at all. All you > did was to copy the commandline recipe I used and write a README > around it. I *fixed* your commandline recipe and wrote a README around it, that is a difference. > It also clearly shows that dpkg does not implement what this bug is > asking for; a command line recipe that will return a few false > positives is not the same as "dpkg already implements this" or as > "adding a --find-removed option to deborphan". dpkg -l | grep '^rc' is excatly what you asked for, the ^ makes the difference and prevents false positives, again: there are no false positives with this fixed command (^ matches the beginning of a line). So your justification "a command line recipe that will return a few false positives is not the same as ..", is not correct. If you think that deborphan should include a wrapper around dpkg -l | grep '^rc' please retitle this bug using a mail to [EMAIL PROTECTED] with the below-mentioned body and I will not close this bug by mentioning it in the README with the next upload: package deborphan retitle 389684 deborphan: Please add a wrapper around dpkg -l | grep '^rc' thanks Thanks for taking care about deborphan :) Carsten -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
Bug#389684: new README does not close the bug
The new README you propose does not address the bug at all. All you did was to copy the commandline recipe I used and write a README around it. It also clearly shows that dpkg does not implement what this bug is asking for; a command line recipe that will return a few false positives is not the same as "dpkg already implements this" or as "adding a --find-removed option to deborphan". -- Martin-Éric Racine http://q-funk.iki.fi
