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

Bug#926475: Review + debdiff



Hi Bruno,


On Mon, Apr 08, 2019 at 08:14:02PM +0200, Bruno Kleinert wrote:
> 
> Hi Stefan,
> 
> I only had minor improvements, see attached debdiff. Those include:
>  * Set debhelper compatibility level by build dependency magic.
>  * Update the .desktop file patch to remove a broken line and add
>    another category for the tool.
>  * Override lintian warning for source tarball repack and provide a
>    rationale.

Looks good in the debdiff!

> 
> I *think* the "Copyright (c)" part is redundant in debian/copyright and
> could possibly be removed from lines 27-32.

<https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/#copyright-field>

Actually, the examples all have "Copyright" there.

Debdiff looks fine to me, thanks for the review including the proposed changes.

How to proceed?
Do you want to upload the package with your debdiff applied?
Or should I apply it and propose a new .dsc file?

Cheers,
  Stefan.

Attachment: signature.asc
Description: PGP signature


Reply to: