From 699d0e5c2bd5bcd52a1833a802281ca66d7ae111 Mon Sep 17 00:00:00 2001 From: joey Date: Fri, 19 Jul 2002 00:58:01 +0000 Subject: r538: * Make dh_installchangelogs install debian/NEWS files as well, as NEWS.Debian. Make dh_compress always compress them. The idea is to make these files be in a machine parsable form, like the debian changelog, but only put newsworthy info into them. Automated tools can then display new news on upgrade. It is hoped that if this catches on it will reduce the abuse of debconf notes. See discussion on debian-devel for details. --- Debian/Debhelper/Dh_Getopt.pm | 2 ++ debian/changelog | 21 +++++++++++++++++++++ dh_compress | 2 +- dh_install | 41 +++++++++++++++++++++++++++++++++++++++-- dh_installchangelogs | 16 ++++++++++++++++ dh_shlibdeps | 6 ++++-- 6 files changed, 83 insertions(+), 5 deletions(-) diff --git a/Debian/Debhelper/Dh_Getopt.pm b/Debian/Debhelper/Dh_Getopt.pm index 37d6f475..79674144 100644 --- a/Debian/Debhelper/Dh_Getopt.pm +++ b/Debian/Debhelper/Dh_Getopt.pm @@ -143,6 +143,8 @@ sub parseopts { "h|help" => \&showhelp, "mainpackage=s" => \$options{MAINPACKAGE}, + + "list-missing" => \$options{LIST_MISSING}, "<>" => \&NonOption, ); diff --git a/debian/changelog b/debian/changelog index ae50371a..593d2c49 100644 --- a/debian/changelog +++ b/debian/changelog @@ -1,3 +1,24 @@ +debhelper (4.0.19) unstable; urgency=low + + * Make dh_installchangelogs install debian/NEWS files as well, as + NEWS.Debian. Make dh_compress always compress them. The idea is to make + these files be in a machine parsable form, like the debian changelog, but + only put newsworthy info into them. Automated tools can then display new + news on upgrade. It is hoped that if this catches on it will reduce the + abuse of debconf notes. See discussion on debian-devel for details. + + -- Joey Hess Sun, 14 Jul 2002 23:09:24 -0400 + +debhelper (4.0.18) unstable; urgency=low + + * Removed a seemingly useless -dDepends in dh_shlibdeps's call to + dpkg-shalibdeps; this allows for stuff like dh_shlibdeps -- -dRecommends + Closes: #152117 + * Added a --list-missing parameter to dh_install, which calc may find + useful. + + -- Joey Hess Sun, 7 Jul 2002 22:44:01 -0400 + debhelper (4.0.17) unstable; urgency=low * In dh_install, don't limit to -type f when doing the find due to -X. diff --git a/dh_compress b/dh_compress index 5e8bb379..5c04e0c3 100755 --- a/dh_compress +++ b/dh_compress @@ -95,7 +95,7 @@ foreach my $package (@{$dh{DOPACKAGES}}) { # ".tgz", "-gz", "-z", "_z" push @files, split(/\n/,` find usr/info usr/share/info usr/man usr/share/man usr/X11*/man -type f ! -name "*.gz" 2>/dev/null || true; - find usr/doc usr/share/doc -type f \\( -size +4k -or -name "changelog*" \\) \\ + find usr/doc usr/share/doc -type f \\( -size +4k -or -name "changelog*" -or -name "NEWS*" \\) \\ \\( -name changelog.html -or ! -iname "*.htm*" \\) \\ ! -iname "*.gif" ! -iname "*.png" ! -iname "*.jpg" \\ ! -iname "*.jpeg" ! -iname "*.gz" ! -iname "*.taz" \\ diff --git a/dh_install b/dh_install index 69f24365..84588407 100755 --- a/dh_install +++ b/dh_install @@ -7,11 +7,12 @@ dh_install - install files into package build directories =cut use strict; +use File::Find; use Debian::Debhelper::Dh_Lib; =head1 SYNOPSIS -B [B<-X>I] [S>] [S>] +B [B<-X>I] [B<--sutodest>] [B<--list-missing>] [S>] [S>] =head1 DESCRIPTION @@ -65,6 +66,19 @@ Note that if you list only a filename on a line by itself in a debian/package.install file, with no explicit destination, then dh_install will automatically guess the destination even if this flag is not set. +=item B<--list-missing> + +This option makes dh_install keep track of the files it installs, and then at +the end, compare that list with the files in debian/tmp. If any of the files +(and symlinks) in debian/tmp were not installed to somewhere, it will +warn on stderr about that. + +This may be useful if you have a large package and want to make sure that +you don't miss installing newly added files in new upstream releases. + +Note that files that are excluded from being moved via the -X option are not +warned about. + =item B<--sourcedir=dir> Makes all source files relative to "dir". If this is specified, it is akin @@ -88,7 +102,7 @@ The files will be installed into the first package dh_install acts on. init(); -my $ret=0; +my @installed; foreach my $package (@{$dh{DOPACKAGES}}) { my $tmp=tmpdir($package); @@ -134,6 +148,15 @@ foreach my $package (@{$dh{DOPACKAGES}}) { doit("install","-d","$tmp/$dest"); } + # Keep track of what's installed. + if ($dh{LIST_MISSING}) { + # Kill any extra slashes. Makes the + # @installed stuff more robust. + $src=~y:/:/:s; + $src=~s:/+$::; + push @installed, "\Q$src\E\/.*|\Q$src\E"; + } + if (-d $src && $exclude) { my ($dir_basename) = basename($src); # Pity there's no cp --exclude .. @@ -148,6 +171,20 @@ foreach my $package (@{$dh{DOPACKAGES}}) { } } +if ($dh{LIST_MISSING}) { + my @missing; + my $installed=join("|", @installed); + $installed=qr{^$installed$}; + find(sub { + -f || -l || return; + $_="$File::Find::dir/$_"; + push @missing, $_ unless /$installed/; + }, './debian/tmp'); + if (@missing) { + warning "$_ not installed" foreach @missing; + } +} + =head1 SEE ALSO L diff --git a/dh_installchangelogs b/dh_installchangelogs index dc9b5a05..2588d57f 100755 --- a/dh_installchangelogs +++ b/dh_installchangelogs @@ -25,6 +25,12 @@ into usr/share/doc/package/changelog.Debian in the package build directory. (If files named debian/package.changelog exist, they will be used in preference to debian/changelog.) +Parallelling the debian changelog handling, this program also takes care of +debian NEWS files. If there is a debian/NEWS file, it is installed as +usr/share/doc/package/NEWS for native packages, and as +usr/share/doc/package/NEWS.Debian for non-native packages. debian/package.NEWS +files can also be used. + If an upstream changelog file is specified as an option, and the package is not a native debian package, then this upstream changelog will be installed as usr/share/doc/package/changelog in the package build directory. If the @@ -67,17 +73,23 @@ if (isnative($dh{MAINPACKAGE}) && defined $upstream) { } my $changelog_name="changelog.Debian"; +my $news_name="NEWS.Debian"; if (isnative($dh{MAINPACKAGE})) { $changelog_name='changelog'; + $news_name='NEWS'; } foreach my $package (@{$dh{DOPACKAGES}}) { my $tmp=tmpdir($package); my $changelog=pkgfile($package,"changelog"); + my $news=pkgfile($package,"NEWS"); if (!$changelog) { $changelog="debian/changelog"; } + if (!$news) { + $news="debian/NEWS"; + } if (! -e $changelog) { error("could not find changelog $changelog"); @@ -93,6 +105,10 @@ foreach my $package (@{$dh{DOPACKAGES}}) { } doit("install","-o",0,"-g",0,"-p","-m644",$changelog, "$tmp/usr/share/doc/$package/$changelog_name"); + if (-e $news) { + doit("install","-o",0,"-g",0,"-p","-m644",$news, + "$tmp/usr/share/doc/$package/$news_name"); + } if ($upstream) { my $link_to; diff --git a/dh_shlibdeps b/dh_shlibdeps index b3648e51..5001d75f 100755 --- a/dh_shlibdeps +++ b/dh_shlibdeps @@ -89,7 +89,8 @@ foreach my $package (@{$dh{DOPACKAGES}}) { $find_options="! \\( $dh{EXCLUDE_FIND} \\)"; } foreach my $file (split(/\n/,`find $tmp -type f \\( -perm +111 -or -name "*.so*" \\) $find_options`)) { - # TODO: this is slow, optimize. Ie, file can run once on multiple files.. + # TODO this is slow, optimize. Ie, file can run once on + # multiple files.. $ff=`file "$file"`; if ($ff=~m/ELF/ && $ff!~/statically linked/) { push @filelist,$file; @@ -97,7 +98,8 @@ foreach my $package (@{$dh{DOPACKAGES}}) { } if (@filelist) { - doit("dpkg-shlibdeps","-Tdebian/${ext}substvars",@{$dh{U_PARAMS}},'-dDepends',@filelist); + doit("dpkg-shlibdeps","-Tdebian/${ext}substvars", + @{$dh{U_PARAMS}},@filelist); } } -- cgit v1.2.3