[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#988398: unblock: aprx/2.9.0+dfsg-3



Control: tags -1 moreinfo

On 2021-05-12 00:28:26, Dave Hibberd wrote:
> Package: release.debian.org
> Severity: normal
> User: release.debian.org@packages.debian.org
> Usertags: unblock
> 
> Please unblock package aprx
> 
> [ Reason ]
> This is a fix for bug #987332, which impacts a web service the daemon
> connects to. Our package installs, enables and starts by default with a 
> useless config, connecting to their server and doing nothing else.
> 
> [ Impact ]
> The grave bug outstanding #987332 for the package will remain standing,
> the remote service shall continue to see dead connections.
> 
> [ Tests ]
> This package has been manually tested to ensure new behaviour is as
> expected (package exits cleanly on start), and importantly doesn't
> cause an update to fail. No autopkgtests have been included as I've not
> yet read enough to feel competent at implementing them.
> 
> [ Risks ]
> This is quite a niche package with 73 Debian users on popcon. Currently, the
> default config is causing issue for the service provider it connects to,
> and it will continue to if left. The only change to the package is made
> to the default configs, commenting out one of the connection details.
> 
> [ Checklist ]
>   [X] all changes are documented in the d/changelog
>   [X] I reviewed all changes and I approve them
>   [X] attach debdiff against the package in testing
> 
> [ Other info ] 
> Complicating the issue is my relative inexperience with freeze - bundled
> into this upload is a Debian janitor bump from dh11->12 that's been on
> salsa for a while - I included this in the upload, and it is detailed in
> d/changelog, however I didn't want to bring standards or dh any further
> up to date than janitor did a year back.

debhelper compat changes are not approprtiate for this phase of the
freeze. Please revert that change but make sure that the fix for RC bug
still works as intended.

Cheers

> 
> unblock aprx/2.9.0+dfsg-3
> 
> -- System Information:
> Debian Release: bullseye/sid
>   APT prefers testing-security
>   APT policy: (500, 'testing-security'), (500, 'testing'), (10, 'unstable')
> Architecture: amd64 (x86_64)
> Foreign Architectures: i386
> 
> Kernel: Linux 5.10.0-1-amd64 (SMP w/16 CPU threads)
> Kernel taint flags: TAINT_OOT_MODULE, TAINT_UNSIGNED_MODULE
> Locale: LANG=en_GB.UTF-8, LC_CTYPE=en_GB.UTF-8 (charmap=UTF-8), LANGUAGE=en_GB:en
> Shell: /bin/sh linked to /usr/bin/dash
> Init: systemd (via /run/systemd/system)
> LSM: AppArmor: enabled
> 
> thx
> -- 
>   Hibby
>   MM0RFN

> diff -Nru aprx-2.9.0+dfsg/debian/changelog aprx-2.9.0+dfsg/debian/changelog
> --- aprx-2.9.0+dfsg/debian/changelog	2018-09-27 04:20:51.000000000 +0100
> +++ aprx-2.9.0+dfsg/debian/changelog	2021-05-09 23:15:56.000000000 +0100
> @@ -1,3 +1,19 @@
> +aprx (2.9.0+dfsg-3) unstable; urgency=medium
> +  [ Dave Hibberd ]
> +  * Add debian/gitlab-ci.yml
> +  * Change installinit behaviour in debian/rules
> +    - aprx is now disabled by default upon install
> +    - Belt & Braces interrupting installsystemd too
> +  * Added patch to modify default config commenting out default callsign
> +    - Closes: #987332
> +  [ Debian Janitor]
> +  * Use secure URI in debian/watch.
> +  * Use secure URI in Homepage field.
> +  * Bump debhelper from old 11 to 12.
> +  * Update renamed lintian tag names in lintian overrides.
> +
> + -- Dave Hibberd <d@vehibberd.com>  Sun, 09 May 2021 23:15:56 +0100
> +
>  aprx (2.9.0+dfsg-2) unstable; urgency=medium
>  
>    * debian/aprx.init
> diff -Nru aprx-2.9.0+dfsg/debian/compat aprx-2.9.0+dfsg/debian/compat
> --- aprx-2.9.0+dfsg/debian/compat	2018-09-27 04:20:51.000000000 +0100
> +++ aprx-2.9.0+dfsg/debian/compat	1970-01-01 01:00:00.000000000 +0100
> @@ -1 +0,0 @@
> -11
> diff -Nru aprx-2.9.0+dfsg/debian/control aprx-2.9.0+dfsg/debian/control
> --- aprx-2.9.0+dfsg/debian/control	2018-09-27 04:20:51.000000000 +0100
> +++ aprx-2.9.0+dfsg/debian/control	2020-04-20 18:22:23.000000000 +0100
> @@ -3,9 +3,9 @@
>  Priority: optional
>  Maintainer: Debian Hamradio Maintainers <debian-hams@lists.debian.org>
>  Uploaders: Chris Knadle <Chris.Knadle@coredump.us>, Colin Tuckley  <colint@debian.org>, Dave Hibberd <d@vehibberd.com>
> -Build-Depends: debhelper (>= 11)
> +Build-Depends: debhelper-compat (= 12)
>  Standards-Version: 4.2.1
> -Homepage: http://thelifeofkenneth.com/aprx/
> +Homepage: https://thelifeofkenneth.com/aprx/
>  Vcs-Browser: https://salsa.debian.org/debian-hamradio-team/aprx
>  Vcs-Git: https://salsa.debian.org/debian-hamradio-team/aprx.git
>  
> diff -Nru aprx-2.9.0+dfsg/debian/gitlab-ci.yml aprx-2.9.0+dfsg/debian/gitlab-ci.yml
> --- aprx-2.9.0+dfsg/debian/gitlab-ci.yml	1970-01-01 01:00:00.000000000 +0100
> +++ aprx-2.9.0+dfsg/debian/gitlab-ci.yml	2021-05-09 22:09:08.000000000 +0100
> @@ -0,0 +1,6 @@
> +include:
> +  - https://salsa.debian.org/salsa-ci-team/pipeline/raw/master/salsa-ci.yml
> +  - https://salsa.debian.org/salsa-ci-team/pipeline/raw/master/pipeline-jobs.yml
> +
> +reprotest:
> +  extends: .test-reprotest-diffoscope
> diff -Nru aprx-2.9.0+dfsg/debian/NEWS aprx-2.9.0+dfsg/debian/NEWS
> --- aprx-2.9.0+dfsg/debian/NEWS	1970-01-01 01:00:00.000000000 +0100
> +++ aprx-2.9.0+dfsg/debian/NEWS	2021-05-09 23:15:56.000000000 +0100
> @@ -0,0 +1,8 @@
> +aprx (2.9.0+dfsg-3) unstable; urgency=medium
> +
> +  aprx now ships with NOCALL-1 commented out in the default config, causing
> +  the program to exit upon running with default config. Please read the
> +  supplied example configs and edit them with your own callsign to avoid
> +  spamming the APRS-IS servers, as we have been doing for some time now.  
> +
> + -- Dave Hibberd <d@vehibberd.com>  Sun, 09 May 2021 22:34:00 +0100
> diff -Nru aprx-2.9.0+dfsg/debian/patches/03-Config-callsign.patch aprx-2.9.0+dfsg/debian/patches/03-Config-callsign.patch
> --- aprx-2.9.0+dfsg/debian/patches/03-Config-callsign.patch	1970-01-01 01:00:00.000000000 +0100
> +++ aprx-2.9.0+dfsg/debian/patches/03-Config-callsign.patch	2021-05-09 22:04:07.000000000 +0100
> @@ -0,0 +1,43 @@
> +Description: Patch to prevent default config from connecting to APRS-IS
> +Authod: Dave Hibberd <d@vehibberd.com>
> +Bug: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=987332
> +Last-Updated: 2021-05-09
> +
> +--- a/aprx-complex.conf.in
> ++++ b/aprx-complex.conf.in
> +@@ -19,7 +19,8 @@
> + # Usable for 99+% of cases.
> + #
> + 
> +-mycall  N0CALL-1
> ++# Uncomment the below line and add your own callsign
> ++# mycall  N0CALL-1
> + 
> + #
> + # Global macro for simplified "my location" definition in
> +--- a/aprx-rxigate.conf.in
> ++++ b/aprx-rxigate.conf.in
> +@@ -25,8 +25,8 @@
> + # Global macro for simplified callsign definition:
> + # Usable for 99+% of cases.
> + #
> +-
> +-mycall  N0CALL-1
> ++# Uncomment the below line and add your own callsign
> ++# mycall  N0CALL-1
> + 
> + <aprsis>
> + #login     OTHERCALL-7	# login defaults to $mycall
> +--- a/aprx.conf.in
> ++++ b/aprx.conf.in
> +@@ -29,8 +29,8 @@
> + # Global macro for simplified callsign definition:
> + # Usable for 99+% of cases.
> + #
> +-
> +-mycall  N0CALL-1
> ++# Uncomment the below line and add your own callsign
> ++# mycall  N0CALL-1
> + 
> + #
> + # Global macro for simplified "my location" definition in
> diff -Nru aprx-2.9.0+dfsg/debian/patches/series aprx-2.9.0+dfsg/debian/patches/series
> --- aprx-2.9.0+dfsg/debian/patches/series	2018-09-27 04:20:51.000000000 +0100
> +++ aprx-2.9.0+dfsg/debian/patches/series	2021-05-09 21:56:04.000000000 +0100
> @@ -1,2 +1,3 @@
> +03-Config-callsign.patch
>  01-man-page.patch
>  02-reproducible-builds.patch
> diff -Nru aprx-2.9.0+dfsg/debian/rules aprx-2.9.0+dfsg/debian/rules
> --- aprx-2.9.0+dfsg/debian/rules	2018-09-27 04:20:51.000000000 +0100
> +++ aprx-2.9.0+dfsg/debian/rules	2021-05-09 23:09:33.000000000 +0100
> @@ -31,7 +31,10 @@
>  	rm -vf ./test ./aprx-complex.conf Makefile
>  
>  override_dh_installinit:
> -	dh_installinit --restart-after-upgrade
> +	dh_installinit --restart-after-upgrade --no-enable
> +
> +override_dh_installsystemd:
> +	dh_installsystemd --no-enable
>  
>  override_dh_installdirs:
>  	dh_installdirs
> diff -Nru aprx-2.9.0+dfsg/debian/source/lintian-overrides aprx-2.9.0+dfsg/debian/source/lintian-overrides
> --- aprx-2.9.0+dfsg/debian/source/lintian-overrides	2017-12-11 17:48:21.000000000 +0000
> +++ aprx-2.9.0+dfsg/debian/source/lintian-overrides	2020-04-20 18:22:23.000000000 +0100
> @@ -1,2 +1,2 @@
>  # Upstream does not sign the source, override watch-may-check-gpg-sig warning
> -aprx source: debian-watch-may-check-gpg-signature
> +aprx source: debian-watch-does-not-check-gpg-signature
> diff -Nru aprx-2.9.0+dfsg/debian/watch aprx-2.9.0+dfsg/debian/watch
> --- aprx-2.9.0+dfsg/debian/watch	2017-12-11 17:48:21.000000000 +0000
> +++ aprx-2.9.0+dfsg/debian/watch	2020-04-20 18:22:23.000000000 +0100
> @@ -3,5 +3,5 @@
>  version=4
>  opts="repack,repacksuffix=+repack,\
>  dversionmangle=s/(\da?)[\+\.\-~](?:dfsg|debian|ds|repack|repacked)\.?\d*$/$1/" \
> -http://thelifeofkenneth.com/aprx/release/ \
> +https://thelifeofkenneth.com/aprx/release/ \
>  (?:|.*/)aprx(?:[_\-]v?|)(\d[^\s/]*)\.(?:tar\.xz|txz|tar\.bz2|tbz2|tar\.gz|tgz)


-- 
Sebastian Ramacher

Attachment: signature.asc
Description: PGP signature


Reply to: