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") || retur

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 () { > + chomp; > + push @log, $_; > + $db->{$package}

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 () { > - chomp; > -

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 loa

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

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

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_lo

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 th