]> git.puffer.fish Git - mirror/frr.git/commit
debianpkg: normalize rules files
authorsmccroskey <smccroskey@cumulusnetworks.com>
Wed, 31 Jan 2018 01:00:48 +0000 (17:00 -0800)
committersmccroskey <smccroskey@cumulusnetworks.com>
Wed, 31 Jan 2018 22:57:47 +0000 (14:57 -0800)
commit9782a8db773b8c5d9094eb3743aae43cc04e12f0
tree93a933af2648643aa33c51205db8b34c7f015c88
parent08951735bd9cf05a0abdd1a1c2eb17fca2043618
debianpkg: normalize rules files

downstream packagers prefer to change defaults in a trackable way by
changing the rules file directly, rather than setting environment
variables.  Use '?=' (set if not already set in the environment or on
the make cmdline) to set defaults rather than alternating between
ifeq/ifneq clauses, which is harder to follow and edit.  Change any
existing `ifneq(...,0)' cases to `ifeq(...,1)', and get rid of any
ifdef/ifndefs, as ?= guarantees he value will be defined in one way or
another.  This allows the old behavior of overriding via the
environment while simplifying the workflow for anyone editing or
extending the current logic and defaults.

Portability note about '?=': it is GNU-make specific, but so is
ifeq/ifneq, which is/was used in this file, and this file is specific
to debian-based system as it is, so I don't consider it to be a
problem in this case.

Added any missing defaults (WANT_SNMP, WANT_CUMULUS_NODE) and
made it so that USE_XXX is always set for the sake of consistency.
Also brought a few changes from base debianpkg/rules into the
backports versions of the files where they were missing.

Signed-off-by: Silas McCroskey <smccroskey@cumulusnetworks.com>
debianpkg/backports/ubuntu12.04/debian/rules
debianpkg/backports/ubuntu14.04/debian/rules
debianpkg/rules