Bug#520615: Option to act on the packages not logged yet

2009-03-22 Thread Joey Hess
Modestas Vainius wrote: --- a/dh +++ b/dh @@ -486,17 +486,7 @@ sub run { sub loadlog { my $package=shift; - my $ext=pkgext($package); - - my @log; - open(LOG, , debian/${ext}debhelper.log) || return; - while (LOG) { - chomp; - push

Bug#520615: Option to act on the packages not logged yet

2009-03-22 Thread Joey Hess
Modestas Vainius wrote: +sub load_log { + my ($package, $db)=...@_; + my $ext=pkgext($package); + + my @log; + open(LOG, , debian/${ext}debhelper.log) || return; + while (LOG) { + chomp; + push @log, $_; + $db-{$package}{$_}=1 if

Bug#520615: Option to act on the packages not logged yet

2009-03-22 Thread Modestas Vainius
Hello, On 2009 m. March 23 d., Monday 01:42:26 you wrote: --- a/dh +++ b/dh @@ -486,17 +486,7 @@ sub run { =20 sub loadlog { my $package=3Dshift; - my $ext=3Dpkgext($package); -=09 - my @log; - open(LOG, , debian/${ext}debhelper.log) || return; - while (LOG) {

Bug#520615: Option to act on the packages not logged yet

2009-03-21 Thread Modestas Vainius
Package: debhelper Version: 7.2.6 Severity: wishlist Tags: patch Hello, lets suppose there are 5 binary packages: foo, bar, abc, def, ghi. Consider this debian/rules snippet: override_dh_foo: dh_foo -pfoo --option1 dh_foo -pbar --option2 # The following is not resistant to

Bug#520615: Option to act on the packages not logged yet

2009-03-21 Thread Joey Hess
Modestas Vainius wrote: + exclude-logged = \$dh{EXCLUDE_LOGGED}, + remaining-packages = \$dh{EXCLUDE_LOGGED}, I don't see a reason to have two names for the same option. --exclude-logged somewhat exposes internals so I prefer the other one. +sub is_helper_in_log { +

Bug#520615: Option to act on the packages not logged yet

2009-03-21 Thread Modestas Vainius
Hello, fixed patch is attached. -- Modestas Vainius modes...@vainius.eu From 830d6634ed3525cf2e3e816880971f8d5ff29dc6 Mon Sep 17 00:00:00 2001 From: Modestas Vainius modes...@vainius.eu Date: Sat, 21 Mar 2009 18:43:28 +0200 Subject: [PATCH] Add a global --remaining-packages option. Add a

Bug#520615: Option to act on the packages not logged yet

2009-03-21 Thread Modestas Vainius
Hello, On 2009 m. March 21 d., Saturday 18:46:31 Modestas Vainius wrote: fixed patch is attached. Removed bogus || return; from load_log. Full version of fixed (2) patch attached. Sorry. --- a/Debian/Debhelper/Dh_Lib.pm +++ b/Debian/Debhelper/Dh_Lib.pm @@ -106,7 +106,7 @@ sub END { sub