Merge branch 'master' into buildsystems

Conflicts:
debian/changelog
dh_auto_configure

releasing version 7.2.12

dh_auto_configure: Revert --skipdeps change Closes: #528647, reopens: #528235

releasing version 7.2.11

Support debian/foo.os files to suppliment previous debian/foo.arch file support. Closes: #494914 (Thanks, Aurelien Jarno)

Merge branch 'master' into buildsystems

Conflicts:
dh_auto_configure

dh_auto_configure: Add --skipdeps when running Makefile.PL, to prevent Module::Install from trying to download dependencies. Closes: #528235

incorporate create_packlist=0 fix from master

Merge branch 'master' into buildsystems

Conflicts:
debian/changelog
dh_auto_configure

dh: Support --with addon,addon,... Closes: #528178

releasing version 7.2.10

dh_auto_configure: Pass packlist=0 when running Makefile.PL, in case it is a Build.PL passthru, to avoid it creating the .packlist file. Closes: #527990

mention quilt

dh_auto_configure: Clarify man page re adding configure parameters. Closes: #527256

Close COMPAT_IN filehandle. Closes: #527464

clarify

releasing version 7.2.9

dh: Add --without to allow disabling sequence addons (particularly useful to disable the default python-support addon).

dh_fixperms: Fix permissions of OCaml .cmxs files. Closes: #526221

Fix permissions of *.cmxs files

dh_fixperms: Ensure lintian overrides are mode 644. (Patch from #459548)

changelog

Merge branch 'master' into buildsystems

releasing version 7.2.8

update

basic docs about buildsystem classes

Merge branch 'master' into buildsystems

Move dh sequence documentation to PROGRAMMING.

rename autotools to autoconf

It seems bette to use the more specific name in case we later want a
separate module for automake.

unimportant code changes

remove verbose_print of buildsystem selection details

This would be the only place in debhelper where -v enables debugging info
that is not just shell commands being run.

Since --list can be used to see details of build system selection, and
since it will probably be obvious which one is selected in -v mode due to
the commands run, I think this oddity is unnecessary.

reformat list

I think this is a bit easier to understand; I was never a fan of complex
and hard to read column headers in console output (ie, dpkg -l).

reword descriptions

Shorten, remove duplicate verbiage, and list the characteristic file
of the build system.

factor out a buildsystems_list

update copyright

rename Dh_Buildsystem to Buildsystem

This way the root of the class hierarchy has the same name as the directory
holding the classes.

use $this rather than $self

let's write class, not cls

remove _mkdir, use mkdir -p

_mkdir is not necessary, because mkdir's error messages are good enough
if a file by the name of the directory exists ("cannot create directory: File
exists"), or if a file is where the parent directory should be ("cannot
create directory: Not a directory")

Using mkdir -p seems useful, in case someone wants a deeply nested
builddir.

This also changes the return value of mkdir_builddir, but nothing currently
tests it.

remove discussion

removal of comments I'm satisfied with

debhelper modular buildsystems (try 3).

* New feature - when listing buildsystems, list their status too
  (auto/specified).
* Dh_Buildsystem_Basic.pm renamed to Dh_Buildsystem.pm
* Addressed a few issues expressed in the comments, answered a few
  comments.
* Cache DEB_BUILD_GNU_TYPE value. Performance hit is noticable when
  listing build systems.
* is_auto_buildable() renamed to check_auto_buildable() (again).
  Since there is is_buildable() now, I didn't want to use is_ for
  that method.

Signed-off-by: Modestas Vainius <modestas@vainius.eu>

more comments

more comments

more comments

update and remove XXX comments

Modular object-orientied buildsystem implementation (try 2).

Major changess:

* Dh_Buildsystem_Option dropped and Dh_Buildsystem_Chdir functionality
  partitially merged into Dh_Buildsystem_Basic. Dh_Buildsystem_Bases.pm
  renamed to Dh_Buildsystem_Basic.pm to match classname.
* *_impl() ditched completely. Previous {configure,build,test,install,clean}_impl()
  renamed to just configure(), build(), test(), install(), clean() instead.
  Added pre_action($action) and post_action($action) hooks instead which are called
  by Dh_Buildsystems::buildsystems_do().
* Builddir is handled via mkdir_builddir(), doit_in_buildddir(), clean_builddir()
  methods which buildsystems should call directly. Removed get_top* method, added
  get_rel2builddir_path().
* is_buildable() method renamed to is_auto_buildable() to reflect its
  purpose more.
* ::perl_makefile renamed to ::perl_makemaker and which is based on ::makefile
  now. MakeMaker hack moved from ::makefile to ::perl_makemaker where it belongs
  (thanks for the tip).
* Dh_Buildsystems refactored into a simple perl module rather than OO class and
  simplified a bit.
* @BUILDSYSTEMS and is_auto_buildable() modified to 100% match historical order.

TODO: user documentation (e.g. DH_AUTO_BUILDDIRECTORY and DH_AUTO_BUILDSYSTEM
environment variables and common dh_auto_* options (--buildsystem and --builddirectory)).

Current plugin inheritance hierarchy is like this:

    Buildsystem::perl_build -> Dh_Buildsystem_Basic  <- Buildsystem::python_distutils
                                      ^
                                      |
                              Buildsystem::makefile  <- Buildsystem::perl_makemaker
                              ^       ^           ^
                             /        |             \
       Buildsystem::autotools  Buildsystem::cmake   Buildsystem::python_distutils

Signed-off-by: Modestas Vainius <modestas@vainius.eu>

update

code review, added comments

I went through every line of the buildsystem implementation,
and added numerous comments. Search for "XXX JEH" to find them.

Migrates dh_auto_* commands to Dh_Buildsystems.pm

This patch switches dh_auto_* commands to use Dh_Buildsystems.pm
module. Most of them became very minimal by design.

Documentation has not been updated with new features yet.

Signed-off-by: Modestas Vainius <modestas@vainius.eu>

Modular object-orientied buildsystem implementation.

Dh_Buildsystems:

A manager module for buildsystem "plugins". It deals with the following tasks:

* Handles common command line and environment options. As currently implemented
  by the patch they are:
   - DH_AUTO_BUILDSYSTEM envvar, -m/--build-system - disables autoguessing of
     the build system and allows the user to specify which one to use.
   - DH_AUTO_BUILDDIRECTORY envvar, -b/--build-directory - option to enable
     building outside source if supported by the buildsystem. User can specify
     the build directory name or let it be autogenerated (currently
     "obj-`dpkg_architecture('DEB_BUILD_GNU_TYPE')`" as per CDBS convention).
     Outside source building has an advantage of avoiding sourcedir pollution
     which the clean routine cannot deal with properly (at least common in
     cmake or autotools case). The "clean" is simple in such a case - just
     rm -rf builddir.
   - -l/--list - lists all buildsystems known to Dh_Buildsystems along with
     their descriptions.
* Manages buildsystem plugins:
  - provides a way to list them and collect information about them.
  - provides a way to force loading & use of a specific buildsystem.
  - determines which build system is applicable to the source in question using
    common API (::is_buildable() method) exposed by each build system plugin.
* @BUILDSYSTEMS variable contains all buildsystems known to the manager in the
  order of specialization.
-----------------------------

-----------------------------
Dh_Buildsystem_Bases.pm:

Contains a few classes which define a common interface for buildsystem plugins
and implements handling of common features (i.e. two types of the build
directory support, see below). Each specific build system plugin is supposed to
inherit from any of these base classes or from another build system plugin.
Currently implemented classes (packages) inside this .pm are:

-- Dh_Buildsystem_Basic --

a basic class describing buildsystem plugin API. It stores build directory
internally (can be retrieved with ::get_builddir() or path constructed using
::get_buildpath() (useful in is_buildable())) but does nothing with it. This
class is intended to be inherited by the build system plugins which do not
support outside-source tree building or there is no way to control this option
(as far as tell, Build.PL is like this). It also describes common buildsystem
plugin API and lays down the basic architecture:

* ::configure/::build/::test/::install/::clean methods - they will be called to
  perform a respective action. These are wrapper methods by default and provide
  a place to implement common features specific the action itself (like
  creating build directory, see Dh_Buildsystem_Chdir::configure()) before
  calling real buildsystem specific implementation. Default implementations
  call the respective *_impl() method via another invoke_impl() wrapper.

* ::configure_impl/::build_impl/::test_impl/::install_impl/::clean-impl methods
  - placeholders for the buildsystem specific implementation of the action (by
  overriding the methods as needed). Default implementations do nothing.

* ::invoke_impl($method_name, @args) - a convenient way to hook in the code
  which needs to be run before or after respective ::*_impl() of *each* action
  (e.g. a simple case like setting envvar, see perl_build.pm). Default
  implementation calls $self->$method_name(@args) by default.

So we have such a chain by default (and each can be overriden by any derived
class):

$self->$action() calls:
  $self->invoke_impl("${action}_impl", @_) calls:
    $self->$action_impl(@_) <- does buildsystem specific stuff here;

-- Dh_Buildsystem_Option --

extends Dh_Buildsystem_Basic and adds support for passing build directory name
via command line option to the build script (specific plugins should override
::get_builddir_option() method). ::invoke_impl() is overriden to pass value of
$self->get_builddir_option() to each ::$action_impl() method (python distutils
use such a way to set "build place", i.e. --build-place=builddir, see
python_distutils.pm).

-- Dh_Buildsystem_Chdir --

extends Dh_Buildsystem_Option. This class implements support for outside source
building when you need to chdir to the building directory before building (like e.g.
makefile.pm and its derivatives: autotools.pm and cmake.pm). All the code in there
deals with chdir'ing/mkdir'ing to the build directory as needed before calling
::$action_impl() and finally going back. This is done by overriding ::invoke_impl()
method.
-----------------------------

-----------------------------
And finally we have build system specific plugins as Debian/Debhelper/Buildsystem/*.pm.
Currently I have implemented 100% functionality of the former dh_auto_* tools
inside these plugins + cmake support in the cmake.pm:

$ ./dh_auto_configure -l
autotools - support for building GNU Autotools based packages.
cmake - support for building CMake based packages (outside-source tree only).
perl_build - support for building Perl Build.PL based packages (in-source only).
perl_makefile - support for building Perl Makefile.PL based packages (in-source only).
python_distutils - support for building Python distutils based packages.
makefile - support for building Makefile based packages (make && make install).

Current plugin inheritance hierarchy is like this:

Buildsystem::perl_build       -> Dh_Buildsystem_Basic  <- Buildsystem::perl_makefile
                                      ^              (maybe it should derive from ::perl_build?)
                                      |
Buildsystem::python_distutils -> Dh_Buildsystem_Option
                                      ^
                                      |
                                Dh_Buildsystem_Chdir
                                      ^
                                      |
                                Buildsystem::makefile
                                   ^            ^
                                  /              \
                   Buildsystem::autotools    Buildsystem::cmake

Signed-off-by: Modestas Vainius <modestas@vainius.eu>

Add dpkg_architecture_value and sourcepackage to Dh_Lib

Both these function are taken from dh_auto_configure. I believe they
are useful enough to be in Dh_Lib (esp. dpkg_architecture_value()).

The patch removes these funtions from dh_auto_configure too.

Signed-off-by: Modestas Vainius <modestas@vainius.eu>

dh_desktop: Now a deprecated no-op, since desktop-file-utils uses triggers. Closes: #523474 (also Closes: #521960, #407701 as no longer applicable)

dh_desktop: Avoid using find -execdir as it will fail with certian badly configured PATHs (and is not a benefit in this context anyway). Closes: #521960

releasing version 7.2.7

dh_icons: ignore gnome and hicolor themes (will be handled by triggers). Closes: #521181

dh_icons: ignore gnome and hicolor themes (will be handled by triggers). Closes: #521181

Pass -L UTF-8 to po4a to work around bug #520942

changelog

export write_log

Avoids the ugly thunk in dh

document load_load, write_log

minor coding stlye

Add a global --remaining-packages option.

Add a global --remaining-packages option which allows to skip the command on
the packages which it has already been run on (i.e. if the command helper is
already present in the package debhelper log).

Signed-off-by: Modestas Vainius <modestas@vainius.eu>

fix corner case

If DH_INTERNAL_OPTIONS does not specify any packages, we don't want to
exclude them all!

remove dead code

This case can never happen, because code in Dh_Getopt makes the same
check earlier.

Fix calling the same helper for separate packages in the override of dh binary-indep/binary-arch. Closes: #520567

This is based on some work by Modestas Vainius, somewhat simplified
by a trick using excludes.

Note that the error in the case where there are no packages to build was
changed to a warning. That can easily happen now, and doesn't seem
particilarly error-worthy anyway; just exiting w/o doing anything seems
fine in that case.

pass -N into DH_INTERNAL_OPTIONS

I think I didn't do this before because it could result in
parseoptions() erroring because there were no packages to act on.
That is not going to be an error soon though, and it makes sense to
pass in the -N excludes.

releasing version 7.2.6

dh_auto_test: Support DEB_BUILD_OPTIONS=nocheck. Closes: #519374

Examples files updated to add dh_bugfiles, remove obsolete dh_python.

releasing version 7.2.5

Revert "dh_installmenus: Now that a triggers capable menu and dpkg are in stable, menu does not need to be explicitly run in maintainer scripts, except for packages with menu-methods files. (See #473467)"

This reverts commit 651be44c078b91112bf18d87d3629d23beb1a5c7.

Conflicts:

debian/changelog

changelog

Set MODULEBUILDRC environment variable

When building packages using Module::Build and a ~/.modulebuildrc
containing

  install install_base=~

the build will end up installing files in /root.

This patch makes debhelper export MODULEBUILDRC="/dev/null" whenever
Module::Build is used to avoid using ~/.modulebuildrc.

Closes: #517423
Signed-off-by: Ansgar Burchardt <ansgar@43-1.org>

releasing version 7.2.4

dh_makeshlibs: Fix --add-udeb, for real. Closes: #518706

releasing version 7.2.3

dh_makeshlibs: Re-add --add-udeb support. Closes: #518655

* dh_makeshlibs: Re-add --add-udeb support. Closes: #518655
* dh_shlibdeps: Remove --add-udeb switch (was accidentially added here).

releasing version 7.2.3-1

remove item already done

Compatability level 4 is now deprecated.

minor/style changes

add cruft for dh_bugfiles

Add dh_bugfiles - a helper for reportbug files

* Add a new helper: dh_bugfiles. It can be used for installing
  bug reporting customization files (Closes: #326874).

Signed-off-by: Modestas Vainius <modestas@vainius.eu>

dh_installchangelogs: Support -X to exclude automatic installation of specific upstream changelogs. Closes: #490937

dh: Override LC_ALL, not LANG. Closes: #517617

dh_installdocs: No longer add maintainer script code to call doc-base, as it supports triggers in stable.

dh_installmenus: Now that a triggers capable menu and dpkg are in stable, menu does not need to be explicitly run in maintainer scripts, except for packages with menu-methods files. (See #473467)

add docs

releasing version 7.2.2

conffile moving idiocy

* dh_installmodules: Give files in /etc/modprobe.d a .conf
  syntax, as required by new module-init-tools.
* dh_installmodules: Add preinst and postinst code to handle
  cleanly renaming the modprobe.d files on upgrade.
* Two updates to conffile moving code from wiki:
  - Support case where the conffile name is a substring of another
    conffile's name.
  - Support case where dpkg-query says the file is obsolete.

Merge branch 'dh_overrides'

Conflicts:
debian/changelog

releasing version 7.0.52

dh: Fix make parsing to not be broken by locale settings. Closes: #517617

Merge branch 'dh_overrides'

Conflicts:
debian/changelog

releasing version 7.0.51

dh: Man page typo. Closes: #517549, #517550

dh: Man page typo. Closes: #517549

releasing version 7.2.0

Merge branch 'dh_overrides'

Conflicts:
Debian/Debhelper/Dh_Getopt.pm
debian/changelog