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

Message

Bastian Germann Jan. 14, 2021, 6:37 p.m. UTC

Apply some minor changes to the xfsprogs debian packages, including
missing copyright notices that are required by Debian Policy.

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

Nathan Scott Jan. 15, 2021, 3:40 a.m. UTC | #1

Heya,

On Fri, Jan 15, 2021 at 5:39 AM Bastian Germann
<[e-mail]> wrote:
>
> Apply some minor changes to the xfsprogs debian packages, including
> missing copyright notices that are required by Debian Policy.
>
> 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

Having reviewed each of these, please add for each:

Signed-off-by: Nathan Scott <[e-mail]>


Also, please add Bastion to the list of deb uploaders - thanks!

diff --git a/debian/control b/debian/control
index 49ffd340..06b92400 100644
--- a/debian/control
+++ b/debian/control
@@ -2,7 +2,7 @@ Source: xfsprogs
 Section: admin
 Priority: optional
 Maintainer: XFS Development Team <[e-mail]>
-Uploaders: Nathan Scott <[e-mail]>, Anibal Monsalve Salazar
<[e-mail]>
+Uploaders: Nathan Scott <[e-mail]>, Anibal Monsalve Salazar
<[e-mail]>, Bastian Germann <[e-mail]>
 Build-Depends: libinih-dev, uuid-dev, dh-autoreconf, debhelper (>=
5), gettext, libtool, libedit-dev, libblkid-dev (>= 2.17),
linux-libc-dev, libdevmapper-dev, libattr1-dev, libicu-dev, dh-python,
pkg-config
 Standards-Version: 4.0.0
 Homepage: https://xfs.wiki.kernel.org/

Darrick J. Wong Jan. 15, 2021, 4:14 a.m. UTC | #2

On Fri, Jan 15, 2021 at 02:40:39PM +1100, Nathan Scott wrote:
> Heya,
> 
> On Fri, Jan 15, 2021 at 5:39 AM Bastian Germann
> <[e-mail]> wrote:
> >
> > Apply some minor changes to the xfsprogs debian packages, including
> > missing copyright notices that are required by Debian Policy.
> >
> > 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
> 
> Having reviewed each of these, please add for each:
> 
> Signed-off-by: Nathan Scott <[e-mail]>

Assuming that SOB also applies to the patch below, someone please add a
commit message, and:

Reviewed-by: Darrick J. Wong <[e-mail]>

--D


> 
> 
> Also, please add Bastion to the list of deb uploaders - thanks!
> 
> diff --git a/debian/control b/debian/control
> index 49ffd340..06b92400 100644
> --- a/debian/control
> +++ b/debian/control
> @@ -2,7 +2,7 @@ Source: xfsprogs
>  Section: admin
>  Priority: optional
>  Maintainer: XFS Development Team <[e-mail]>
> -Uploaders: Nathan Scott <[e-mail]>, Anibal Monsalve Salazar
> <[e-mail]>
> +Uploaders: Nathan Scott <[e-mail]>, Anibal Monsalve Salazar
> <[e-mail]>, Bastian Germann <[e-mail]>
>  Build-Depends: libinih-dev, uuid-dev, dh-autoreconf, debhelper (>=
> 5), gettext, libtool, libedit-dev, libblkid-dev (>= 2.17),
> linux-libc-dev, libdevmapper-dev, libattr1-dev, libicu-dev, dh-python,
> pkg-config
>  Standards-Version: 4.0.0
>  Homepage: https://xfs.wiki.kernel.org/
>

Eric Sandeen Jan. 15, 2021, 6:31 p.m. UTC | #3

On 1/14/21 10:14 PM, Darrick J. Wong wrote:
> On Fri, Jan 15, 2021 at 02:40:39PM +1100, Nathan Scott wrote:
>> Heya,
>>
>> On Fri, Jan 15, 2021 at 5:39 AM Bastian Germann
>> <[e-mail]> wrote:
>>>
>>> Apply some minor changes to the xfsprogs debian packages, including
>>> missing copyright notices that are required by Debian Policy.
>>>
>>> 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
>>
>> Having reviewed each of these, please add for each:
>>
>> Signed-off-by: Nathan Scott <[e-mail]>
> 
> Assuming that SOB also applies to the patch below, someone please add a
> commit message, and:
> 
> Reviewed-by: Darrick J. Wong <[e-mail]>

Will do.  Nathan, it's easier if you send a proper patch ;)

Thanks,
-Eric

Eric Sandeen Jan. 15, 2021, 6:35 p.m. UTC | #4

On 1/14/21 9:40 PM, Nathan Scott wrote:
> Heya,
> 
> On Fri, Jan 15, 2021 at 5:39 AM Bastian Germann
> <[e-mail]> wrote:
>>
>> Apply some minor changes to the xfsprogs debian packages, including
>> missing copyright notices that are required by Debian Policy.
>>
>> 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
> 
> Having reviewed each of these, please add for each:
> 
> Signed-off-by: Nathan Scott <[e-mail]>

Do you maybe mean Reviewed-by: ?

-Eric

Bastian Germann Jan. 16, 2021, 9:34 a.m. UTC | #5

Am 15.01.21 um 04:40 schrieb Nathan Scott:
> Heya,
> 
> On Fri, Jan 15, 2021 at 5:39 AM Bastian Germann
> <[e-mail]> wrote:
>>
>> Apply some minor changes to the xfsprogs debian packages, including
>> missing copyright notices that are required by Debian Policy.
>>
>> 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
> 
> Having reviewed each of these, please add for each:
> 
> Signed-off-by: Nathan Scott <[e-mail]>
> 
> 
> Also, please add Bastion to the list of deb uploaders - thanks!

Hi Nathan,

There is no point in adding me unless you allow uploads for my key
2861 2573 17C7 AEE4 F880  497E C386 0AC5 9F57 4E3A via

https://wiki.debian.org/DebianMaintainer#Granting_Permissions

Thanks,
Bastian

> 
> diff --git a/debian/control b/debian/control
> index 49ffd340..06b92400 100644
> --- a/debian/control
> +++ b/debian/control
> @@ -2,7 +2,7 @@ Source: xfsprogs
>   Section: admin
>   Priority: optional
>   Maintainer: XFS Development Team <[e-mail]>
> -Uploaders: Nathan Scott <[e-mail]>, Anibal Monsalve Salazar
> <[e-mail]>
> +Uploaders: Nathan Scott <[e-mail]>, Anibal Monsalve Salazar
> <[e-mail]>, Bastian Germann <[e-mail]>
>   Build-Depends: libinih-dev, uuid-dev, dh-autoreconf, debhelper (>=
> 5), gettext, libtool, libedit-dev, libblkid-dev (>= 2.17),
> linux-libc-dev, libdevmapper-dev, libattr1-dev, libicu-dev, dh-python,
> pkg-config
>   Standards-Version: 4.0.0
>   Homepage: https://xfs.wiki.kernel.org/
>

Nathan Scott Jan. 18, 2021, 1:37 a.m. UTC | #6

Hi Bastian,

On Sat, Jan 16, 2021 at 8:34 PM Bastian Germann
<[e-mail]> wrote:
> [...]
> There is no point in adding me unless you allow uploads for my key
> 2861 2573 17C7 AEE4 F880  497E C386 0AC5 9F57 4E3A via
>

Will do, thanks for the pointer.

cheers.

--
Nathan