Bug#925383: unblock: shorewall/5.2.3.2-1

classic Classic list List threaded Threaded
6 messages Options
Reply | Threaded
Open this post in threaded view
|

Bug#925383: unblock: shorewall/5.2.3.2-1

Roberto C. Sánchez-2
Package: release.debian.org
Severity: normal
User: [hidden email]
Usertags: unblock

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Shorewall upstream released version 5.2.3.2 a few days ago.  While this
unblock request is for 6 packages, only shorewall contains any
functional changes.  The other five packages contain only a version
increment and release notes entry.

Here is the explanation of the upstream change:

5.2.3.2

1)  Shorewall 5.2 automatically converts and existing 'masq' file to an
    equivalent 'snat' file. Regrettably, Shorewall 5.2.3 broke that
    automatic update, such that the following error message was issued:

       Use of uninitialized value $Shorewall::Nat::raw::currentline in
       pattern match (m//) at /usr/share/shorewall/Shorewall/Nat.pm
       line 511, <$currentfile> line nnn.

    and the generted 'masq' file contains only initial comments.

    That has been corrected.

I have attached debdiffs for all 6 packages.

Given the small scope and size of the fix and that the Shorewall
upstream project is undergoing some changes that most likely make this
the last release for quite some time, I would very much like to see this
make it into Buster.

Regards,

- -Roberto

unblock shorewall/5.2.3.2-1
unblock shorewall6/5.2.3.2-1
unblock shorewall-lite/5.2.3.2-1
unblock shorewall6-lite/5.2.3.2-1
unblock shorewall-core/5.2.3.2-1
unblock shorewall-init/5.2.3.2-1

-----BEGIN PGP SIGNATURE-----

iQIzBAEBCgAdFiEEz9ERzDttUsU/BH8iLNd4Xt2nsg8FAlyW5u0ACgkQLNd4Xt2n
sg+C4w/9FvQBTnKgjT3o+jNamHNw5fTofLCM0x+7uVSWzI4v9YjwJnBBjhGSvd/7
t8NsOLTj6bjplylRiiOrvqUAgfl3p8LxkSNYHtxZdgUhU8kKcTHYmWOgOtHDrp/+
2wB2YSGTjkMl6QzsvK9e/P35xZ7Sd/iQlEU9miwmYWlT5b3l18M5ekZRZmroM8VU
Wn8yhfJzWBZs1MKcaV58rtVM3yl/ZfwhtjFIIvWVqc94w20POFTZR7FkEMTxd5NQ
9C+bzrZS13+QP6wYs7XLekFDtNGGn9s5PK/k67syuT7q6YdjRfhyHKMxHM/hsY22
ppGUd1qqcRspxOod/EPI5AqE4BUmUpVkfF3I2OmbWPFgK+UNrkYd3NIzdkBn1ahW
T2LrSPJPYvmN3/bo+tSP1U1m0Hh5+g0XDllTNBExQZq7JiAgoHPKVdgmUzihEl6c
71ao9kS/FzpuwwciFcg/TrTEFFJdm1VLBMviNatZhMTttkD8Fmrot2SBtpVGvOkg
ddiEHzpR7Ba95hrBOI3Jyu7Z8Ff3iF2hrd31IrMCjYKiJmXCisPHyUeJReKo6pIB
kj4Qm8STO6rwc2hm4/nKHrrbP3pMo9nRaY8Kumqo7Qc/hv1/90WhMJdPdd75kg/s
SaQjkiyfe/egZihShkJYsZNhRXfSSTsPSkgp7csN1kDzJiYZva4=
=SmkF
-----END PGP SIGNATURE-----

shorewall_5.2.3.1-1_5.2.3.2-1.diff (7K) Download Attachment
shorewall6_5.2.3.1-1_5.2.3.2-1.diff (3K) Download Attachment
shorewall-lite_5.2.3.1-1_5.2.3.2-1.diff (3K) Download Attachment
shorewall6-lite_5.2.3.1-1_5.2.3.2-1.diff (3K) Download Attachment
shorewall-init_5.2.3.1-1_5.2.3.2-1.diff (3K) Download Attachment
shorewall-core_5.2.3.1-1_5.2.3.2-1.diff (4K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Bug#925383: unblock: shorewall/5.2.3.2-1

Jonathan Wiltshire via nm
Control: tag -1 moreinfo

On Sat, Mar 23, 2019 at 10:09:49PM -0400, Roberto C. Sanchez wrote:

> 5.2.3.2
>
> 1)  Shorewall 5.2 automatically converts and existing 'masq' file to an
>     equivalent 'snat' file. Regrettably, Shorewall 5.2.3 broke that
>     automatic update, such that the following error message was issued:
>
>        Use of uninitialized value $Shorewall::Nat::raw::currentline in
>        pattern match (m//) at /usr/share/shorewall/Shorewall/Nat.pm
>        line 511, <$currentfile> line nnn.
>
>     and the generted 'masq' file contains only initial comments.
>
>     That has been corrected.
>
> I have attached debdiffs for all 6 packages.

It can't be unblocked until it's in unstable; are you asking for
pre-approval? I didn't read the diffs in detail yet but it sounds like a
fix we'd want so I suggest you go ahead and remove the moreinfo tag when
it's ready for review.

Thanks,

--
Jonathan Wiltshire                                      [hidden email]
Debian Developer                         http://people.debian.org/~jmw

4096R: 0xD3524C51 / 0A55 B7C5 1223 3942 86EC  74C3 5394 479D D352 4C51

Reply | Threaded
Open this post in threaded view
|

Bug#925383: unblock: shorewall/5.2.3.2-1

Roberto C. Sánchez-2
tags 925383 - moreinfo
thanks

On Sun, Mar 24, 2019 at 04:39:20PM +0000, Jonathan Wiltshire wrote:

> Control: tag -1 moreinfo
>
> On Sat, Mar 23, 2019 at 10:09:49PM -0400, Roberto C. Sanchez wrote:
> > 5.2.3.2
> >
> > 1)  Shorewall 5.2 automatically converts and existing 'masq' file to an
> >     equivalent 'snat' file. Regrettably, Shorewall 5.2.3 broke that
> >     automatic update, such that the following error message was issued:
> >
> >        Use of uninitialized value $Shorewall::Nat::raw::currentline in
> >        pattern match (m//) at /usr/share/shorewall/Shorewall/Nat.pm
> >        line 511, <$currentfile> line nnn.
> >
> >     and the generted 'masq' file contains only initial comments.
> >
> >     That has been corrected.
> >
> > I have attached debdiffs for all 6 packages.
>
> It can't be unblocked until it's in unstable; are you asking for
> pre-approval? I didn't read the diffs in detail yet but it sounds like a
> fix we'd want so I suggest you go ahead and remove the moreinfo tag when
> it's ready for review.
>
Yes, it was my intent to request pre-approval.  My apologies for the
confusion.

Regards,

-Roberto

--
Roberto C. Sánchez

signature.asc (849 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Bug#925383: Processed: Re: Bug#925383: unblock: shorewall/5.2.3.2-1

Roberto C. Sánchez-2
In reply to this post by Roberto C. Sánchez-2
On Sun, Mar 24, 2019 at 06:51:04PM +0000, Debian Bug Tracking System wrote:
> Processing commands for [hidden email]:
>
> > tags 925383 - moreinfo
> Bug #925383 [release.debian.org] unblock: shorewall/5.2.3.2-1
> Removed tag(s) moreinfo.
> > thanks
> Stopping processing here.
>
Hi Jonathan,

I removed the moreinfo tag nearly a week ago.  Did I misunderstand what
else I needed to do?  Did I need to go ahead and upload as well?  I was
waiting for a pre-approval, but if uploading now makes more sense and
saves you work (since you could review and then unblock the waiting
package), I can upload right away.

Regards,

-Roberto

--
Roberto C. Sánchez

Reply | Threaded
Open this post in threaded view
|

Bug#925383: Processed: Re: Bug#925383: unblock: shorewall/5.2.3.2-1

Paul Gevers-4
Control: tags -1 moreinfo confirmed

Hi Roberto,

On 30-03-2019 13:24, Roberto C. Sánchez wrote:
> I removed the moreinfo tag nearly a week ago.  Did I misunderstand what
> else I needed to do?  Did I need to go ahead and upload as well?

That. Jonathan said "I suggest you go ahead and remove the moreinfo tag
when it's ready for review". I agree that there is a slight ambiguity
there, but he meant, "I suggest you go ahead *with the upload* and
remove the moreinfo tag when it's ready for review *of the difference
between what is in buster and what is in sid*".

> I was
> waiting for a pre-approval, but if uploading now makes more sense and
> saves you work (since you could review and then unblock the waiting
> package), I can upload right away.

I have added the moreinfo tag again, please remove it again once the
upload happened and we can review the delta between sid and buster.

Paul


signature.asc (499 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Bug#925383: Processed: Re: Bug#925383: unblock: shorewall/5.2.3.2-1

Roberto C. Sánchez-2
Control: tags -1 - moreinfo

Hi Paul,

On Sun, Mar 31, 2019 at 10:35:35AM +0200, Paul Gevers wrote:

> Control: tags -1 moreinfo confirmed
>
> Hi Roberto,
>
> On 30-03-2019 13:24, Roberto C. Sánchez wrote:
> > I removed the moreinfo tag nearly a week ago.  Did I misunderstand what
> > else I needed to do?  Did I need to go ahead and upload as well?
>
> That. Jonathan said "I suggest you go ahead and remove the moreinfo tag
> when it's ready for review". I agree that there is a slight ambiguity
> there, but he meant, "I suggest you go ahead *with the upload* and
> remove the moreinfo tag when it's ready for review *of the difference
> between what is in buster and what is in sid*".
>
OK.  That was clearly a reading comprehension failure on my part.  Thank
you for clarifying.

> > I was
> > waiting for a pre-approval, but if uploading now makes more sense and
> > saves you work (since you could review and then unblock the waiting
> > package), I can upload right away.
>
> I have added the moreinfo tag again, please remove it again once the
> upload happened and we can review the delta between sid and buster.
>
I have uploaded the packages and also removed the moreinfo tag.

Regards,

-Roberto

--
Roberto C. Sánchez