[Git][debian-gis-team/libapache2-mod-tile][master] 4 commits: fixed another spelling error (upstream pr #175)
Felix Delattre
debian at xama.nu
Fri Aug 21 12:47:13 BST 2020
Thanks for the feedback!
I need to get more familiar with quilt and will submit valid patches next week.
Cheers,
Felix
On 8/21/20 5:19 AM, Sebastiaan Couwenberg wrote:
> On 8/20/20 9:39 PM, Felix Delattre wrote:
>> =====================================
>> debian/patches/prevent-duplicates.patch
>> =====================================
>> @@ -0,0 +1,75 @@
>> +Description: Move variables to prevent duplicates (from https://github.com/openstreetmap/mod_tile/pull/202)
>> +Author: Felix Delattre <debian at xama.nu>
>
> The patch headers look wrong, it seems the changes are from the linked
> PR of which you are not the author.
>
> The following is correct for a patch from a PR:
>
> Description: Move variables to prevent duplicates
> Author: Jon Burgess
> Origin: https://github.com/openstreetmap/mod_tile/pull/202
>
> See also: https://dep-team.pages.debian.net/deps/dep3/
>> =====================================
>> debian/patches/series
>> =====================================
>> @@ -2,3 +2,5 @@ iniparser.patch
>> openlayers.patch
>> spelling-errors.patch
>> manpage-section-mismatch.patch
>> +prevent-duplicates.patch
>> +update-link-and-typo.patch
>
> The file is update-link-and-path.patch, so this won't apply.
>
> Did you use `quilt new` to create the patch?
>
> That updates the series file automatically.
>
> See also:
>
> https://debian-gis-team.pages.debian.net/policy/policy.html#quilt
>
>> =====================================
>> debian/patches/spelling-errors.patch
>> =====================================
>
> The additional fixes are not documented in the patch description.
>
>> =====================================
>> debian/patches/update-link-and-path.patch
>> =====================================
>> @@ -0,0 +1,27 @@
>> +Description: Update links and paths to new locations
>> +Author: Felix Delattre <debian at xama.nu>
>
> Patches should be Forwarded upstream and marked accordingly.
>
> lintian started checking for the recently:
>
> $ lintian-info -t patch-not-forwarded-upstream
> I: patch-not-forwarded-upstream
> N:
> N: According to the DEP-3 headers, this patch has not been forwarded
> N: upstream. Please forward the patch upstream and work with them to
> N: ensure the patch is included in the version control system and in
> N: the next upstream release of your package.
> N:
> N: If this patch should not be forwarded upstream please put
> N: not-needed in the Forwarded header.
> N:
> N: Refer to Debian Developer's Reference section 3.1.4 (Coordination
> N: with upstream developers) and Debian Policy Manual section 4.3
> N: (Changes to the upstream sources) for details.
> N:
> N: Severity: info
> N:
> N: Check: debian/patches/dep3
> N:
>
> The openstreetmap-tiles-update-expire changes seem appropriate to
> forward upstream.
>
> Using patches to modify files in the debian directory is not appropriate
> though, those changes should be made directly.
>
> Kind Regards,
>
> Bas
>
More information about the Pkg-grass-devel
mailing list