From: David Lamparter <equinox@diac24.net>
Date: Wed, 24 Oct 2018 18:57:16 +0000 (+0200)
Subject: debianpkg: update to Debian Policy version 4.2.1
X-Git-Tag: 7.1_pulled~219^2~65
X-Git-Url: https://git.puffer.fish/?a=commitdiff_plain;h=5d2f5756cf1b126ad50c256661a7b0c30e49c16b;p=matthieu%2Ffrr.git

debianpkg: update to Debian Policy version 4.2.1

This bundles up a few smaller changes prompted by walking through the
upgrade checklist from 3.9.6 to 4.2.1.

Signed-off-by: David Lamparter <equinox@diac24.net>
---

diff --git a/debianpkg/README.Debian b/debianpkg/README.Debian
index 09313ac6d5..99003b7d08 100644
--- a/debianpkg/README.Debian
+++ b/debianpkg/README.Debian
@@ -77,6 +77,16 @@ build FRR yourself with SNMP with
 Just distributing it in binary form, linked against OpenSSL, is forbidden.
 
 
+* Debian Policy compliance notes
+================================
+
+- 4.15 Reproducibility
+  FRR build is reproducible as outlined in version 4.2.1 of the Policy, but
+  won't be reproducible when the build directory is varied.  This is because
+  configure parameters are burned into the executables which includes CFLAGS
+  like -fdebug-prefix-map=/build/directory/...
+
+
 * Daemon selection:
 ===================
 
diff --git a/debianpkg/README.Maintainer b/debianpkg/README.Maintainer
index 84b68e1949..9030022c5e 100644
--- a/debianpkg/README.Maintainer
+++ b/debianpkg/README.Maintainer
@@ -1,3 +1,13 @@
+#
+# TODO
+#
+
+- check that tests/{control,daemons} actually do something useful and sensible
+- /usr/share/doc/frr-doc should be named just frr?
+- debian/watch pgpsigurlmangle / signing-key
+- multiarch for DSOs?
+- frr try-restart
+
 #
 # To check if the patches still apply on new upstream versions:
 #
diff --git a/debianpkg/control b/debianpkg/control
index 7c31fad252..f9f22f66af 100644
--- a/debianpkg/control
+++ b/debianpkg/control
@@ -3,7 +3,6 @@ Section: net
 Priority: optional
 Maintainer: Nobody <nobody@frrouting.org>
 Uploaders: Nobody <nobody@frrouting.org>
-XSBC-Original-Maintainer: <maintainers@frrouting.org>
 Build-Depends:
  autotools-dev,
  bison,
@@ -31,8 +30,10 @@ Build-Depends:
  python (>= 2.7) | python3,
  python-sphinx | python3-sphinx,
  texinfo (>= 4.7)
-Standards-Version: 3.9.6
+Standards-Version: 4.2.1
 Homepage: http://www.frrouting.org/
+Vcs-Browser: https://github.com/FRRouting/frr/
+Vcs-Git: https://github.com/FRRouting/frr.git
 
 Package: frr
 Architecture: any
@@ -43,6 +44,7 @@ Depends:
  logrotate (>= 3.2-11)
 Pre-Depends: adduser
 Recommends: frr-pythontools
+Suggests: frr-doc
 Conflicts: zebra, zebra-pj, quagga
 Replaces: zebra, zebra-pj
 Description: BGP/OSPF/RIP/RIPng/ISIS/PIM/LDP routing daemon forked from Quagga
@@ -83,7 +85,7 @@ Architecture: any
 Depends:
  ${misc:Depends},
  frr (= ${binary:Version})
-Priority: extra
+Priority: optional
 Section: debug
 Description: BGP/OSPF/RIP/RIPng/ISIS/PIM/LDP routing daemon (debug symbols)
  This package provides debugging symbols for all binary packages built
diff --git a/debianpkg/frr.preinst b/debianpkg/frr.preinst
index 477e690d0a..1c141f37f9 100644
--- a/debianpkg/frr.preinst
+++ b/debianpkg/frr.preinst
@@ -20,7 +20,7 @@ if ! getent passwd frr >/dev/null; then
         adduser \
           --system \
           --ingroup frr \
-          --home /var/run/frr/ \
+          --home /nonexistent \
           --gecos "Frr routing suite" \
           --shell /bin/false \
           frr  >/dev/null
diff --git a/debianpkg/rules b/debianpkg/rules
index a7f8b8c454..27757863ee 100755
--- a/debianpkg/rules
+++ b/debianpkg/rules
@@ -34,9 +34,7 @@ GENERATE_PDF ?= 0
 #
 ####################################
 
-export DH_VERBOSE=1
 export DEB_BUILD_MAINT_OPTIONS = hardening=+all
-export DH_OPTIONS=-v
 
 ifeq ($(WANT_OSPFAPI), 1)
   USE_OSPFAPI=--enable-ospfapi=yes
@@ -79,15 +77,29 @@ endif
 ifneq (,$(filter parallel=%,$(DEB_BUILD_OPTIONS)))
   DEBIAN_JOBS := $(subst parallel=,,$(filter parallel=%,$(DEB_BUILD_OPTIONS)))
 endif
-
 ifdef DEBIAN_JOBS
 MAKEFLAGS += -j$(DEBIAN_JOBS)
 endif
 
+ifneq (,$(filter terse,$(DEB_BUILD_OPTIONS)))
+MAKE_SILENT="V=0"
+export DH_VERBOSE=0
+else
+MAKE_SILENT="V=1"
+export DH_VERBOSE=1
+export DH_OPTIONS=-v
+endif
+
 #
 # build profiles
 #
 
+ifeq ($(filter nodoc,$(DEB_BUILD_PROFILES)),)
+  USE_DOC=--enable-doc
+else
+  USE_DOC=--disable-doc
+endif
+
 ifeq ($(filter pkg.frr.rtrlib,$(DEB_BUILD_PROFILES)),)
   USE_RPKI=--disable-rpki
 else
@@ -118,6 +130,8 @@ override_dh_auto_configure:
 		--localstatedir=/var/run/frr \
 		--sbindir=/usr/lib/frr \
 		--sysconfdir=/etc/frr \
+		--with-vtysh-pager=/usr/bin/pager \
+		$(USE_DOC) \
 		$(USE_SNMP) \
 		$(USE_OSPFAPI) \
 		$(USE_MULTIPATH) \
@@ -158,6 +172,9 @@ endif
 	rm debian/tmp/usr/lib/frr/modules/*.la
 	rm debian/tmp/usr/lib/frr/libyang_plugins/*.la
 
+override_dh_auto_build:
+	dh_auto_build -- $(MAKE_SILENT)
+
 override_dh_auto_clean:
 # we generally do NOT want a full distclean since that wipes both
 # debianpkg/changelog and config.version