diff options
author | Joey Hess <joey@gnu.kitenet.net> | 2009-04-10 17:05:23 -0400 |
---|---|---|
committer | Joey Hess <joey@gnu.kitenet.net> | 2009-04-10 17:05:23 -0400 |
commit | 2b7f42f9ef70c08bb7bc138fb8b24dc993da54ac (patch) | |
tree | b215c75636fc08ed9e2571effdbd8854e441fcd4 /Debian/Debhelper/Buildsystem/autotools.pm | |
parent | 776abbb23a899ebba954774c8b3740706a21f72b (diff) |
code review, added comments
I went through every line of the buildsystem implementation,
and added numerous comments. Search for "XXX JEH" to find them.
Diffstat (limited to 'Debian/Debhelper/Buildsystem/autotools.pm')
-rw-r--r-- | Debian/Debhelper/Buildsystem/autotools.pm | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/Debian/Debhelper/Buildsystem/autotools.pm b/Debian/Debhelper/Buildsystem/autotools.pm index 65694b36..945ca408 100644 --- a/Debian/Debhelper/Buildsystem/autotools.pm +++ b/Debian/Debhelper/Buildsystem/autotools.pm @@ -37,6 +37,16 @@ sub configure_impl { push @opts, "--infodir=\${prefix}/share/info"; push @opts, "--sysconfdir=/etc"; push @opts, "--localstatedir=/var"; + # XXX JEH this is where the sheer evil of Dh_Buildsystem_Chdir + # becomes evident. Why is exec_in_topdir needed here? + # Because: + # - The parent class happens to be derived from Dh_Buildsystem_Chdir. + # - sourcepage() happens to, like many other parts of debhelper's + # library, assume it's being run in the top of the source tree, + # and fails if it's not. + # Having to worry about interactions like that for every line of + # every derived method is simply not acceptable. + # Dh_Buildsystem_Chdir must die! -- JEH push @opts, "--libexecdir=\${prefix}/lib/" . $self->exec_in_topdir(\&sourcepackage); push @opts, "--disable-maintainer-mode"; push @opts, "--disable-dependency-tracking"; @@ -48,6 +58,10 @@ sub configure_impl { push @opts, "--host=" . dpkg_architecture_value("DEB_HOST_GNU_TYPE"); } + # XXX JEH the reason it needs to use get_toppath here, + # but does not need to in the is_buildable method is not clear, + # unless one is familiar with the implementation of its parent + # class. I think that speaks to a bad design.. doit($self->get_toppath("configure"), @opts, @_); } |