mbox series

[v2,0/6] debian: xfsprogs package clean-up

Message ID 20210116092328.2667-1-bastiangermann@fishpost.de (mailing list archive)
Headers show
Series debian: xfsprogs package clean-up | expand

Message

Bastian Germann Jan. 16, 2021, 9:23 a.m. UTC
Apply some minor changes to the xfsprogs debian packages, including
missing copyright notices that are required by Debian Policy.

v2:
  resend with Reviewed-by annotations applied, Nathan actually sent:
  "Signed-off-by: Nathan Scott <nathans@debian.org>"

Bastian Germann (6):
  debian: cryptographically verify upstream tarball
  debian: remove dependency on essential util-linux
  debian: remove "Priority: extra"
  debian: use Package-Type over its predecessor
  debian: add missing copyright info
  debian: new changelog entry

 debian/changelog                |  11 ++++
 debian/control                  |   5 +-
 debian/copyright                | 111 ++++++++++++++++++++++++++++----
 debian/upstream/signing-key.asc |  63 ++++++++++++++++++
 debian/watch                    |   2 +-
 5 files changed, 175 insertions(+), 17 deletions(-)
 create mode 100644 debian/upstream/signing-key.asc

Comments

Eric Sandeen Jan. 17, 2021, 1:09 a.m. UTC | #1
On 1/16/21 3:23 AM, Bastian Germann wrote:
> Apply some minor changes to the xfsprogs debian packages, including
> missing copyright notices that are required by Debian Policy.
> 
> v2:
>   resend with Reviewed-by annotations applied, Nathan actually sent:
>   "Signed-off-by: Nathan Scott <nathans@debian.org>"

Heya Nate - please confirm that this was your intent.

-Eric

> 
> Bastian Germann (6):
>   debian: cryptographically verify upstream tarball
>   debian: remove dependency on essential util-linux
>   debian: remove "Priority: extra"
>   debian: use Package-Type over its predecessor
>   debian: add missing copyright info
>   debian: new changelog entry
> 
>  debian/changelog                |  11 ++++
>  debian/control                  |   5 +-
>  debian/copyright                | 111 ++++++++++++++++++++++++++++----
>  debian/upstream/signing-key.asc |  63 ++++++++++++++++++
>  debian/watch                    |   2 +-
>  5 files changed, 175 insertions(+), 17 deletions(-)
>  create mode 100644 debian/upstream/signing-key.asc
>
Nathan Scott Jan. 18, 2021, 1:24 a.m. UTC | #2
On Sun, Jan 17, 2021 at 12:09 PM Eric Sandeen <sandeen@sandeen.net> wrote:
>
>
>
> On 1/16/21 3:23 AM, Bastian Germann wrote:
> > Apply some minor changes to the xfsprogs debian packages, including
> > missing copyright notices that are required by Debian Policy.
> >
> > v2:
> >   resend with Reviewed-by annotations applied, Nathan actually sent:
> >   "Signed-off-by: Nathan Scott <nathans@debian.org>"
>
> Heya Nate - please confirm that this was your intent.
>

Yep - thanks!

cheers.

--
Nathan
Eric Sandeen Jan. 18, 2021, 9:33 p.m. UTC | #3
On 1/16/21 3:23 AM, Bastian Germann wrote:
> Apply some minor changes to the xfsprogs debian packages, including
> missing copyright notices that are required by Debian Policy.
> 
> v2:
>   resend with Reviewed-by annotations applied, Nathan actually sent:
>   "Signed-off-by: Nathan Scott <nathans@debian.org>"

I've pushed these, plus Nathan's patch to add you to Uploaders

Thanks,
-Eric

> Bastian Germann (6):
>   debian: cryptographically verify upstream tarball
>   debian: remove dependency on essential util-linux
>   debian: remove "Priority: extra"
>   debian: use Package-Type over its predecessor
>   debian: add missing copyright info
>   debian: new changelog entry
> 
>  debian/changelog                |  11 ++++
>  debian/control                  |   5 +-
>  debian/copyright                | 111 ++++++++++++++++++++++++++++----
>  debian/upstream/signing-key.asc |  63 ++++++++++++++++++
>  debian/watch                    |   2 +-
>  5 files changed, 175 insertions(+), 17 deletions(-)
>  create mode 100644 debian/upstream/signing-key.asc
>
Bastian Germann Jan. 19, 2021, 3:02 p.m. UTC | #4
Am 18.01.21 um 22:33 schrieb Eric Sandeen:
> On 1/16/21 3:23 AM, Bastian Germann wrote:
>> Apply some minor changes to the xfsprogs debian packages, including
>> missing copyright notices that are required by Debian Policy.
>>
>> v2:
>>    resend with Reviewed-by annotations applied, Nathan actually sent:
>>    "Signed-off-by: Nathan Scott <nathans@debian.org>"
> 
> I've pushed these, plus Nathan's patch to add you to Uploaders

Thanks! It would be great to have a 5.10.0 version available in Debian 
bullseye. Currently, it has 5.6.0. The freeze is in three weeks, so to 
give the package time to migrate it should be uploaded in January.

> 
> Thanks,
> -Eric
> 
>> Bastian Germann (6):
>>    debian: cryptographically verify upstream tarball
>>    debian: remove dependency on essential util-linux
>>    debian: remove "Priority: extra"
>>    debian: use Package-Type over its predecessor
>>    debian: add missing copyright info
>>    debian: new changelog entry
>>
>>   debian/changelog                |  11 ++++
>>   debian/control                  |   5 +-
>>   debian/copyright                | 111 ++++++++++++++++++++++++++++----
>>   debian/upstream/signing-key.asc |  63 ++++++++++++++++++
>>   debian/watch                    |   2 +-
>>   5 files changed, 175 insertions(+), 17 deletions(-)
>>   create mode 100644 debian/upstream/signing-key.asc
>>
Eric Sandeen Jan. 19, 2021, 5:07 p.m. UTC | #5
On 1/19/21 9:02 AM, Bastian Germann wrote:
> Am 18.01.21 um 22:33 schrieb Eric Sandeen:
>> On 1/16/21 3:23 AM, Bastian Germann wrote:
>>> Apply some minor changes to the xfsprogs debian packages, including
>>> missing copyright notices that are required by Debian Policy.
>>>
>>> v2:
>>>    resend with Reviewed-by annotations applied, Nathan actually sent:
>>>    "Signed-off-by: Nathan Scott <nathans@debian.org>"
>>
>> I've pushed these, plus Nathan's patch to add you to Uploaders
> 
> Thanks! It would be great to have a 5.10.0 version available in Debian bullseye. Currently, it has 5.6.0. The freeze is in three weeks, so to give the package time to migrate it should be uploaded in January.

It's not clear to me who you're asking to do this, but I'm
afraid it's not my role (or my ability). :)

-Eric
Bastian Germann Jan. 19, 2021, 5:16 p.m. UTC | #6
Am 19.01.21 um 18:07 schrieb Eric Sandeen:
> On 1/19/21 9:02 AM, Bastian Germann wrote:
>> Thanks! It would be great to have a 5.10.0 version available in Debian bullseye. Currently, it has 5.6.0. The freeze is in three weeks, so to give the package time to migrate it should be uploaded in January.
> 
> It's not clear to me who you're asking to do this, but I'm
> afraid it's not my role (or my ability). :)
> 
> -Eric
> 

Nathan uploaded most of the versions, so I guess he can upload.
The Debian package is "team maintained" though with this list address as 
recorded maintainer.
Nathan Scott Jan. 19, 2021, 9:26 p.m. UTC | #7
Hey all,

On Wed, Jan 20, 2021 at 4:16 AM Bastian Germann
<bastiangermann@fishpost.de> wrote:
> [...]
> Nathan uploaded most of the versions, so I guess he can upload.
> The Debian package is "team maintained" though with this list address as
> recorded maintainer.
>

You should have the necessary permissions to do uploads since
yesterday Bastian - is that not the case?

BTW Eric, not directly related but I think the -rc versions being
added to the debian/changelog can lead to some confusion - I *think*
dpkg finds that version suffix to be more recent than the actual
release version string (seems odd, but that seemed to be the behaviour
I observed recently).  Could the release script(s) skip adding -rc
versions to debian/changelog?

cheers.

--
Nathan
Bastian Germann Jan. 19, 2021, 9:35 p.m. UTC | #8
Am 19.01.21 um 22:26 schrieb Nathan Scott:
> You should have the necessary permissions to do uploads since
> yesterday Bastian - is that not the case?

I just checked. Yes, it is the case. However, I was not notified about 
the upload permissions, which usually is the case. Thanks.
Darrick J. Wong Jan. 19, 2021, 9:42 p.m. UTC | #9
On Wed, Jan 20, 2021 at 08:26:45AM +1100, Nathan Scott wrote:
> Hey all,
> 
> On Wed, Jan 20, 2021 at 4:16 AM Bastian Germann
> <bastiangermann@fishpost.de> wrote:
> > [...]
> > Nathan uploaded most of the versions, so I guess he can upload.
> > The Debian package is "team maintained" though with this list address as
> > recorded maintainer.
> >
> 
> You should have the necessary permissions to do uploads since
> yesterday Bastian - is that not the case?
> 
> BTW Eric, not directly related but I think the -rc versions being
> added to the debian/changelog can lead to some confusion - I *think*
> dpkg finds that version suffix to be more recent than the actual
> release version string (seems odd, but that seemed to be the behaviour
> I observed recently).  Could the release script(s) skip adding -rc
> versions to debian/changelog?

The release maintainer could also skip adding the ~rc tags to
debian/changelog since it's not like we're going to upload /that/ into
debian anyway.  All of my internal builds add their own tag with dch, so
it wouldn't matter much to me.

(But let's see if Dave has opinions...)

--D

> cheers.
> 
> --
> Nathan
>
Eric Sandeen Jan. 19, 2021, 10 p.m. UTC | #10
On 1/19/21 3:26 PM, Nathan Scott wrote:
> Hey all,
> 
> On Wed, Jan 20, 2021 at 4:16 AM Bastian Germann
> <bastiangermann@fishpost.de> wrote:
>> [...]
>> Nathan uploaded most of the versions, so I guess he can upload.
>> The Debian package is "team maintained" though with this list address as
>> recorded maintainer.
>>
> 
> You should have the necessary permissions to do uploads since
> yesterday Bastian - is that not the case?
> 
> BTW Eric, not directly related but I think the -rc versions being
> added to the debian/changelog can lead to some confusion - I *think*
> dpkg finds that version suffix to be more recent than the actual
> release version string (seems odd, but that seemed to be the behaviour
> I observed recently).  Could the release script(s) skip adding -rc
> versions to debian/changelog?

Er, I got yelled at (in the nicest possible way of course) for
/omitting/ -rc versions from the changelog, IIRC.

Tell me more about how upstream should be maintaining distro-specific
metadata?  :/

-Eric