Bug#988341: unblock: nis/4.3

["Francesco P. Lovergine" <frankie@debian.org>]

I found also a pending doc-only change still seating in my repo:

diff --git a/debian/nis.debian.howto b/debian/nis.debian.howto
index e90e549..2641b86 100644
--- a/debian/nis.debian.howto
+++ b/debian/nis.debian.howto
@@ -66,6 +66,13 @@ The NIS how-to on Debian

  2.1 FOR LIBC6:

+     Ensure to have libnss-nis package installed. It is currently
+     only recommended by both libc and ypbind-mt, because it is not an
+     essential component for the system. Even, for your own reasons you
+     could be interested in binding a NIS domain to access the NIS maps via
+     yptools, but not activating it as an account information provider for
+     the system.
+      
      Check your /etc/nsswitch.conf file and make sure that the entries for
      passwd, group, shadow and netgroup look like this:

I could add this note to the source-only upload, possibly. Is it ok?

On Tue, May 11, 2021 at 10:34:32AM +0200, Sebastian Ramacher wrote:
> Control: tags -1 confirmed moreinfo
>
> On 2021-05-10 20:43:26 +0200, Francesco P. Lovergine wrote:
> > Package: release.debian.org
> > Severity: normal
> > User: release.debian.org@packages.debian.org
> > Usertags: unblock
> >
> > Please unblock package nis
> >
> > [ Reason ]
> >
> > Fixes serious bug #988227 (bashism in postinst).
> >
> > [ Impact ]
> >
> > Upgrade not smoothly done from stable.
> >
> > [ Tests ]
> >
> > No autopkg test. Manually tested with dash.
> >
> > [ Risks ]
> >
> > None.
> >
> > [ 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 ]
> >
> > Native migration package only.
> >
> > unblock nis/4.3
>
> Not built on buildd: arch all binaries uploaded by frankie, a new source-only upload is needed to allow migration
>
> Please perform a source-only upload and remove the moreinfo tag once
> that's done.
>
> Cheers
>
> > --
> > Francesco P. Lovergine
>
> > diff -Nru nis-4.2/debian/changelog nis-4.3/debian/changelog
> > --- nis-4.2/debian/changelog	2021-01-31 10:22:32.000000000 +0100
> > +++ nis-4.3/debian/changelog	2021-05-08 17:19:24.000000000 +0200
> > @@ -1,3 +1,10 @@
> > +nis (4.3) unstable; urgency=medium
> > +
> > +  * Fixed a sort-of bashism in postinst.
> > +    (closes: #988227)
> > +
> > + -- Francesco Paolo Lovergine <frankie@debian.org>  Sat, 08 May 2021 17:19:24 +0200
> > +
> >  nis (4.2) unstable; urgency=medium
> >
> >    * Missed removing of /etc/init.d/nis at upgrade time added.
> > diff -Nru nis-4.2/debian/postinst nis-4.3/debian/postinst
> > --- nis-4.2/debian/postinst	2021-01-31 10:22:32.000000000 +0100
> > +++ nis-4.3/debian/postinst	2021-05-08 17:19:24.000000000 +0200
> > @@ -73,10 +73,13 @@
> >  case "$1" in
> >      configure)
> >          PREV_VER="$2"
> > -		if [ ! -z "$PREV_VER" -a $(dpkg --compare-versions "$PREV_VER" lt '4~'; echo $?) -eq 0 ]
> > -        then
> > -            upgrade_old
> > -        fi
> > +		if [ ! -z "$PREV_VER" ]
> > +		then
> > +			if dpkg --compare-versions "$PREV_VER" lt '4~'
> > +        	then
> > +            	upgrade_old
> > +        	fi
> > +		fi
> >  	  	rm -f /etc/init.d/nis
> >          ;;
> >      *)
>
>
> --
> Sebastian Ramacher



--
Francesco P. Lovergine