[Pkg-nagios-devel] Bug#893723: 1.9.10 closing 4 bugs

Sébastien Delafond seb at debian.org
Wed Dec 12 09:02:42 GMT 2018


Hi fellows,

I've got a 1.9.10 nagvis package ready in salsa[0], that fixes four of
the currently open bugs including this one. I've also manually included
1:1.7.10+dfsg1-3.2, which wasn't present in the salsa repository.

Would you like an actual MR ? I'm also attaching a debdiff of debian/*
to this report, and pasting the corresponding changelog here:

  ,----
  | Source: nagvis
  | Version: 1:1.9.10-1
  | Distribution: unstable
  | Urgency: medium
  | Maintainer: Sebastien Delafond <seb at debian.org>
  | Timestamp: 1544597179
  | Date: Wed, 12 Dec 2018 07:46:19 +0100
  | Closes: 858042 893723 903369 903706
  | Changes:
  |  nagvis (1:1.9.10-1) unstable; urgency=medium
  |  .
  |    [ Bas Couwenberg ]
  |    * Team upload.
  |    * Update watch file for move to GitHub.
  |    * Bump Standards-Version to 4.2.1, no changes.
  |  .
  |    [ Sebastien Delafond ]
  |    * NMU
  |    * Imported Debian patch 1:1.7.10+dfsg1-3.2
  |    * Update php-gettext link
  |    * Add Files-Excluded to debian/copyright, even though those files are
  |      gone in 1.9.x: this will make further repacking more straightforward,
  |      should it be needed
  |    * Add repacksuffix to debian/watch
  |    * New upstream version 1.9.10 (Closes: #893723, #903706)
  |    * Use db_stop at the end of postinst (Closes: #858042)
  |    * Add pt_BR translation (Closes: #903369)
  |    * Update Vcs-* URLs to point to salsa
  `----

Unless you object, I plan on uploading this to delayed/10 later this
week: the RC bug is 9 months old, and the last uploads of nagvis are all
quite old and were all NMUs as well.

Cheers,

-- 
Seb

[0] https://salsa.debian.org/seb/pkg-nagvis
-------------- next part --------------
diff --git a/debian/changelog b/debian/changelog
index 3d2348a2..bcd3fec8 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,25 @@
+nagvis (1:1.9.10-1) unstable; urgency=medium
+
+  [ Bas Couwenberg ]
+  * Team upload.
+  * Update watch file for move to GitHub.
+  * Bump Standards-Version to 4.2.1, no changes.
+
+  [ Sebastien Delafond ]
+  * NMU
+  * Imported Debian patch 1:1.7.10+dfsg1-3.2
+  * Update php-gettext link
+  * Add Files-Excluded to debian/copyright, even though those files are
+    gone in 1.9.x: this will make further repacking more straightforward,
+    should it be needed
+  * Add repacksuffix to debian/watch
+  * New upstream version 1.9.10 (Closes: #893723, #903706)
+  * Use db_stop at the end of postinst (Closes: #858042)
+  * Add pt_BR translation (Closes: #903369)
+  * Update Vcs-* URLs to point to salsa
+  
+ -- Sebastien Delafond <seb at debian.org>  Wed, 12 Dec 2018 07:46:19 +0100
+
 nagvis (1:1.7.10+dfsg1-3.2) UNRELEASED; urgency=medium
 
   * Non-maintainer upload.
diff --git a/debian/control b/debian/control
index ea96db7a..3f07e28f 100644
--- a/debian/control
+++ b/debian/control
@@ -8,10 +8,10 @@ Uploaders: Markus Frosch <markus at lazyfrosch.de>,
 Build-Depends: debhelper (>= 7.0.50~),
                quilt,
                po-debconf
-Standards-Version: 3.9.5
+Standards-Version: 4.2.1
 Homepage: http://www.nagvis.org
-Vcs-Git: git://anonscm.debian.org/pkg-nagios/pkg-nagvis.git
-Vcs-Browser: http://anonscm.debian.org/gitweb/?p=pkg-nagios/pkg-nagvis.git;a=summary
+Vcs-Git: https://salsa.debian.org/nagios-team/pkg-nagvis.git
+Vcs-Browser: https://salsa.debian.org/nagios-team/pkg-nagvis
 
 Package: nagvis
 Architecture: all
diff --git a/debian/copyright b/debian/copyright
index 9a80d868..e5d43b49 100644
--- a/debian/copyright
+++ b/debian/copyright
@@ -1,6 +1,7 @@
 Format: http://www.debian.org/doc/packaging-manuals/copyright-format/1.0/
 Upstream-Name: NagVis
 Source: http://sourceforge.net/projects/nagvis/files/
+Files-Excluded: uifx share/netmap/shell.html share/netmap/shell.swf share/netmap/modules/gmap
 
 Files: *
 Copyright: Copyright 2007-2013, Lars Michelsen <lars at vertical-visions.de>
diff --git a/debian/nagvis.links b/debian/nagvis.links
index 2a2622e0..f00b2429 100644
--- a/debian/nagvis.links
+++ b/debian/nagvis.links
@@ -3,7 +3,7 @@
 /usr/share/nagvis/docs /usr/share/nagvis/share/docs
 /etc/nagvis	/usr/share/nagvis/etc
 /var/cache/nagvis /usr/share/nagvis/var
-/usr/share/php/php-php-gettext /usr/share/nagvis/share/server/core/ext/php-gettext-1.0.9
+/usr/share/php/php-php-gettext /usr/share/nagvis/share/server/core/ext/php-gettext-1.0.12
 /usr/share/nagvis/defaults/nagvis.ini.php-sample /usr/share/doc/nagvis/nagvis.ini.php-sample
 /usr/share/nagvis/defaults/apache2-nagvis.conf-sample /usr/share/doc/nagvis/apache2-nagvis.conf-sample
 etc/nagvis/apache2.conf etc/apache2/conf-available/nagvis.conf
diff --git a/debian/nagvis.postinst b/debian/nagvis.postinst
old mode 100644
new mode 100755
index 4220b70e..c5243614
--- a/debian/nagvis.postinst
+++ b/debian/nagvis.postinst
@@ -198,6 +198,8 @@ case "$1" in
     ;;
 esac
 
+db_stop
+
 # dh_installdeb will replace this with shell code automatically
 # generated by other debhelper scripts.
 
diff --git a/debian/patches/config.patch b/debian/patches/config.patch
index e056f9f6..27cb63af 100644
--- a/debian/patches/config.patch
+++ b/debian/patches/config.patch
@@ -1,12 +1,40 @@
-Description: Change nagvis config paths for Debian
-Author: Alexander Reichle-Schmehl <tolimar at debian.org>
+From: Alexander Reichle-Schmehl <tolimar at debian.org>
+Date: Wed, 12 Dec 2018 07:53:28 +0100
+Subject: Change nagvis config paths for Debian
+
 Reviewed-by: Markus Frosch <markus at lazyfrosch.de>
 Last-Update: 2013-12-12
 ---
-This patch header follows DEP-3: http://dep.debian.net/deps/dep3/
+ share/server/core/classes/GlobalMainCfg.php | 10 +++++-----
+ share/server/core/defines/global.php        |  6 +++---
+ 2 files changed, 8 insertions(+), 8 deletions(-)
+
+diff --git a/share/server/core/classes/GlobalMainCfg.php b/share/server/core/classes/GlobalMainCfg.php
+index 65e5ecb..2460503 100644
+--- a/share/server/core/classes/GlobalMainCfg.php
++++ b/share/server/core/classes/GlobalMainCfg.php
+@@ -1604,11 +1604,11 @@ class GlobalMainCfg {
+      * @author 	Lars Michelsen <lm at larsmichelsen.com>
+      */
+     private function setPathsByBase($base, $htmlBase) {
+-        $this->validConfig['paths']['cfg']['default']                = $base.'etc/';
+-        $this->validConfig['paths']['mapcfg']['default']             = $base.'etc/maps/';
+-        $this->validConfig['paths']['geomap']['default']             = $base.'etc/geomap';
+-        $this->validConfig['paths']['profiles']['default']           = $base.'etc/profiles';
+-        $this->validConfig['global']['authorisation_group_perms_file']['default'] = $base.'etc/perms.db';
++        $this->validConfig['paths']['cfg']['default']                = '/etc/nagvis/';
++        $this->validConfig['paths']['mapcfg']['default']             = '/etc/nagvis/maps/';
++        $this->validConfig['paths']['geomap']['default']             = '/etc/nagvis/geomap';
++        $this->validConfig['paths']['profiles']['default']           = '/etc/nagvis/profiles';
++        $this->validConfig['global']['authorisation_group_perms_file']['default'] = '/etc/perms.db';
+ 
+         $this->validConfig['paths']['var']['default']                = $base.'var/';
+         $this->validConfig['paths']['sharedvar']['default']          = $base.HTDOCS_DIR.'/var/';
+diff --git a/share/server/core/defines/global.php b/share/server/core/defines/global.php
+index 983cd96..2778f28 100644
 --- a/share/server/core/defines/global.php
 +++ b/share/server/core/defines/global.php
-@@ -79,11 +79,11 @@
+@@ -75,11 +75,11 @@ define('DEBUGFILE', '../../../var/nagvis-debug.log');
  // The last value wins.
  //
  // Path to the main configuration file
@@ -21,20 +49,3 @@ This patch header follows DEP-3: http://dep.debian.net/deps/dep3/
  
  // The directory below the NagVis root which is shared by the webserver
  define('HTDOCS_DIR', 'share');
---- a/share/server/core/classes/GlobalMainCfg.php
-+++ b/share/server/core/classes/GlobalMainCfg.php
-@@ -1344,10 +1344,10 @@
-      * @author 	Lars Michelsen <lars at vertical-visions.de>
-      */
-     private function setPathsByBase($base, $htmlBase) {
--        $this->validConfig['paths']['cfg']['default']                = $base.'etc/';
--        $this->validConfig['paths']['mapcfg']['default']             = $base.'etc/maps/';
--        $this->validConfig['paths']['geomap']['default']             = $base.'etc/geomap';
--        $this->validConfig['paths']['profiles']['default']           = $base.'etc/profiles';
-+        $this->validConfig['paths']['cfg']['default']                = '/etc/nagvis/';
-+        $this->validConfig['paths']['mapcfg']['default']             = '/etc/nagvis/maps/';
-+        $this->validConfig['paths']['geomap']['default']             = '/etc/nagvis/geomap';
-+        $this->validConfig['paths']['profiles']['default']           = '/etc/nagvis/profiles';
- 
-         $this->validConfig['paths']['var']['default']                = $base.'var/';
-         $this->validConfig['paths']['sharedvar']['default']          = $base.HTDOCS_DIR.'/var/';
diff --git a/debian/patches/privacy-breach b/debian/patches/privacy-breach
deleted file mode 100644
index 862ffe73..00000000
--- a/debian/patches/privacy-breach
+++ /dev/null
@@ -1,27 +0,0 @@
-Description: Cleanup for possible privacy breaches
-Author: Markus Frosch <markus at lazyfrosch.de>
-Last-Update: 2013-12-27
----
-This patch header follows DEP-3: http://dep.debian.net/deps/dep3/
---- a/docs/de_DE/toc.html
-+++ b/docs/de_DE/toc.html
-@@ -77,7 +77,7 @@
- -->
-         <div style="text-align:center;margin-top:20px;">
-         <p><a href="http://www.nagvis.org" style="margin: 0px;" class="nostyle"><img width="88" height="23" border="0" alt="NagVis Logo" src="../general/img/logo-88x23.png" style="margin: 0px;" /></a></p>
--        <p><a href="http://sourceforge.net" style="margin: 0px;" class="nostyle"><img width="88" height="31" border="0" alt="SourceForge.net Logo" src="http://sflogo.sourceforge.net/sflogo.php?group_id=132019&type=1" style="margin: 0px;" /></a></p>
-+        <p><a href="http://sourceforge.net" style="margin: 0px;" class="nostyle">SourceForge.net</a></p>
-         </div>
-     </body>
- </html>
---- a/docs/en_US/toc.html
-+++ b/docs/en_US/toc.html
-@@ -68,7 +68,7 @@
-         <p>(<font style="color:#ff0000;">*</font>) New in 1.7</p>
-         <div style="text-align:center;margin-top:20px;">
-         <p><a href="http://www.nagvis.org" style="margin: 0px;" class="nostyle"><img width="88" height="23" border="0" alt="NagVis Logo" src="../general/img/logo-88x23.png" style="margin: 0px;" /></a></p>
--        <p><a href="http://sourceforge.net" style="margin: 0px;" class="nostyle"><img width="88" height="31" border="0" alt="SourceForge.net Logo" src="http://sflogo.sourceforge.net/sflogo.php?group_id=132019&type=1" style="margin: 0px;" /></a></p>
-+        <p><a href="http://sourceforge.net" style="margin: 0px;" class="nostyle">SourceForge.net</a></p>
-         </div>
-     </body>
- </html>
diff --git a/debian/patches/remove-shell-swf b/debian/patches/remove-shell-swf
deleted file mode 100644
index f88acec2..00000000
--- a/debian/patches/remove-shell-swf
+++ /dev/null
@@ -1,52 +0,0 @@
-Description: Disabling netmap component for Debian
- The component has been removed because it's non-free.
-Author: Alexander Reichle-Schmehl <tolimar at debian.org>
-Reviewed-by: Markus Frosch <markus at lazyfrosch.de>
-Last-Update: 2013-12-12
----
-This patch header follows DEP-3: http://dep.debian.net/deps/dep3/
---- a/share/frontend/nagvis-js/js/frontend.js
-+++ b/share/frontend/nagvis-js/js/frontend.js
-@@ -1410,8 +1410,8 @@
-         oTr.appendChild(oTd);
- 
-         // Link
--        var oLink = document.createElement('a');
--        oLink.href = oGeneralProperties.path_base+'/netmap/shell.html';
-+//        var oLink = document.createElement('a');
-+//        oLink.href = oGeneralProperties.path_base+'/netmap/shell.html';
- 
-         // Status image
-         var oImg = document.createElement('img');
-@@ -1419,17 +1419,17 @@
-         oImg.src = oGeneralProperties.path_iconsets+'std_small_unknown.png';
-         oImg.alt = 'geomap';
- 
--        oLink.appendChild(oImg);
-+//        oLink.appendChild(oImg);
-         oImg = null;
- 
-         // Title
-         var h2 = document.createElement('h2');
-         h2.appendChild(document.createTextNode('Geomap'));
--        oLink.appendChild(h2);
-+//        oLink.appendChild(h2);
-         h2 = null;
- 
-         var br = document.createElement('br');
--        oLink.appendChild(br);
-+//        oLink.appendChild(br);
-         br = null;
- 
-         // Only show map thumb when configured
-@@ -1442,8 +1442,8 @@
-             oImg = null;
-         }
- 
--        oTd.appendChild(oLink);
--        oLink = null;
-+//        oTd.appendChild(oLink);
-+//        oLink = null;
- 
-         oTd = null;
- 
diff --git a/debian/patches/series b/debian/patches/series
index bf44efdf..4462183f 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1,3 +1 @@
 config.patch
-remove-shell-swf
-privacy-breach
diff --git a/debian/po/pt_BR.po b/debian/po/pt_BR.po
new file mode 100644
index 00000000..59eba369
--- /dev/null
+++ b/debian/po/pt_BR.po
@@ -0,0 +1,74 @@
+# Debconf translations for nagvis
+# Copyright (C) 2012 THE nagvis'S COPYRIGHT HOLDER
+# This file is distributed under the same license as the nagvis package.
+# Eder L. Marques <eder at edermarques.net>, 2012
+#
+msgid ""
+msgstr ""
+"Project-Id-Version: nagvis\n"
+"Report-Msgid-Bugs-To: nagvis at packages.debian.org\n"
+"POT-Creation-Date: 2013-12-19 21:38+0100\n"
+"PO-Revision-Date: 2018-07-08 21:27-0300\n"
+"Last-Translator: Eder L. Marques <eder at edermarques.net>\n"
+"Language-Team: Brazilian Portuguese <debian-l10n-portuguese at lists.debian."
+"org>\n"
+"Language: pt_BR\n"
+"MIME-Version: 1.0\n"
+"Content-Type: text/plain; charset=UTF-8\n"
+"Content-Transfer-Encoding: 8bit\n"
+
+#. Type: select
+#. Choices
+#: ../nagvis.templates:2001
+msgid "other"
+msgstr "outra"
+
+#. Type: select
+#. Description
+#: ../nagvis.templates:2002
+msgid "Monitoring suite used with NagVis:"
+msgstr "Suíte de monitoramento usada com o NagVis:"
+
+#. Type: select
+#. Description
+#: ../nagvis.templates:2002
+msgid ""
+"The NagVis package supports Icinga as well as Nagios, using the check-mk-"
+"live broker backend."
+msgstr ""
+"O pacote NagVis suporta tanto o Icinga quanto o Nagios, usando o backend "
+"broker check-mk-live."
+
+#. Type: select
+#. Description
+#: ../nagvis.templates:2002
+msgid ""
+"If you would like to use NagVis with a different backend or a different "
+"monitoring suite, please choose \"other\". You'll have to configure it "
+"manually."
+msgstr ""
+"Se você quiser usar o NagVis com um backend diferente ou uma suíte de "
+"monitoramento diferente, por favor, escolha \"outra\". Você terá que "
+"configurá-la manualmente."
+
+#. Type: boolean
+#. Description
+#: ../nagvis.templates:3001
+msgid "Delete NagVis data when purging the package?"
+msgstr ""
+"Apagar os dados do NagVis quando remover completamente (\"purge\") o pacote?"
+
+#. Type: boolean
+#. Description
+#: ../nagvis.templates:3001
+msgid ""
+"NagVis creates files in /var/{cache,lib}/nagvis and /etc/nagvis (for "
+"instance background images and map files), including a small database for "
+"authentification. If you don't need any of these files, they can be removed "
+"now, or you may want to keep them and clean up by hand later."
+msgstr ""
+"O NagVis cria arquivos em /var/{cache,lib}/nagvis e /etc/nagvis (por "
+"exemplo, imagens de fundo e arquivos de mapa), incluindo uma pequena base de "
+"dados para autenticação. Se você não precisa de nenhum desses arquivos, eles "
+"podem ser removidos agora, ou você pode querer mantê-los e fazer a limpeza "
+"manualmente depois."
diff --git a/debian/rules b/debian/rules
index 98d7f7b1..c645d895 100755
--- a/debian/rules
+++ b/debian/rules
@@ -19,7 +19,7 @@ override_dh_install:
 	# /usr/share/nagvis/share
 	rm -rf debian/nagvis/usr/share/nagvis/share/userfiles/
 	# We use the external php-gettext lib to avoid copy copies
-	rm -rf debian/nagvis/usr/share/nagvis/share/server/core/ext/php-gettext-1.0.9
+	rm -rf debian/nagvis/usr/share/nagvis/share/server/core/ext/php-gettext-1.0.12
 	# Fixing some lintian issues
 	rm -f debian/nagvis/usr/share/nagvis/share/frontend/nagvis-js/ext/dwoo-1.1.0/LICENSE
 	find debian/nagvis/ -name ".gitignore" -delete
diff --git a/debian/watch b/debian/watch
index 978841f2..4e98232f 100644
--- a/debian/watch
+++ b/debian/watch
@@ -1,3 +1,8 @@
 version=3
-opts=dversionmangle=s/\+dfsg\.?\d+$// \
-http://sf.net/nagvis/nagvis-([\d\.]*)\.tar\.gz
+opts=\
+dversionmangle=s/\+(debian|dfsg|ds|deb)\d*$//,\
+uversionmangle=s/(\d)[_\.\-\+]?((RC|rc|pre|dev|beta|alpha)\d*)$/$1~$2/;s/RC/rc/;s/\-/\./g,\
+repacksuffix=+dfsg,\
+filenamemangle=s/(?:.*?)?(?:rel|v|nagvis)?[\-\_]?(\d\S+)\.(tgz|tbz|txz|(?:tar\.(?:gz|bz2|xz)))/nagvis-$1.$2/ \
+https://github.com/NagVis/nagvis/releases \
+(?:.*?/)?(?:rel|v|nagvis)?[\-\_]?(\d[\d\-\.]+)\.(?:tgz|tbz|txz|(?:tar\.(?:gz|bz2|xz)))


More information about the Pkg-nagios-devel mailing list