]> git.donarmstrong.com Git - debhelper.git/commit
Get rid of is_buildable and build_step flags. Broken by design.
authorModestas Vainius <modestas@vainius.eu>
Thu, 11 Jun 2009 08:18:55 +0000 (11:18 +0300)
committerModestas Vainius <modestas@vainius.eu>
Thu, 11 Jun 2009 08:54:31 +0000 (11:54 +0300)
commit82f043bf5f2d1257aefa639cb058e8e3f3ef00f6
tree25735b3b050da112b97e5ca7aa1e2e2929f27852
parent6f84d7f67943b67d2630985722085c7b4132ce0a
Get rid of is_buildable and build_step flags. Broken by design.

* Auto-setting is_buildable flag in base constructor was pointless
  and broken by design because:
  - is_buildable = check_auto_buildable() used to be called *before*
    constructor of the derivative class could call enforce_* methods.  The
    result of check_auto_buildable() might change after calling enforce_*
    methods (in case check_auto_buildable() use get_buildpath() tests).
  - it isn't used widely. Refactor those a few places.
* Due to above, 'build_step' does not need to be passed to the Buildsystem
  anymore. Remove it from code.
* As a result of is_buidable removal, move warning of
  enforce_in_source_building() to pre_building_step(). It caused unnecessary
  noise when the object was constructed during test. It belongs to
  pre_building_step stage anyway.

Signed-off-by: Modestas Vainius <modestas@vainius.eu>
Debian/Debhelper/Buildsystem.pm
Debian/Debhelper/Dh_Buildsystems.pm