I was able to reproduce this now: golang-github-kr-binarydist debian/master ± git reset --hard e3f90a2 golang-github-kr-binarydist debian/master(-21) ± debcraft improve ... Improving source package at /home/otto/debian/go-team/golang-github-kr-binarydist Running copyright-review.sh: ++ lrc C: Versions: licenserecon '17.0' licensecheck '3.3.9-1'
Parsing Source Tree .... Reading d/copyright .... Running licensecheck .... No significant differences found Running copyright-years.sh: No recent contributors found for debian/, no copyright update needed Running formatting.sh: ++ debputy reformat --style black --- ./debian/control +++ ./debian/control @@ -2,11 +2,13 @@ Section: devel Priority: optional Maintainer: Debian Go Packaging Team <[email protected]> -Uploaders: Nicolas Braud-Santoni <[email protected]> -Build-Depends: bsdiff, - debhelper-compat (= 12), - dh-golang, - golang-go +Uploaders: + Nicolas Braud-Santoni <[email protected]>, +Build-Depends: + debhelper-compat (= 12), + dh-golang, + bsdiff, + golang-go, Standards-Version: 4.5.0 Homepage: https://github.com/kr/binarydist Vcs-Browser: https://salsa.debian.org/go-team/packages/golang-github-kr-binarydist @@ -15,9 +17,10 @@ Package: golang-github-kr-binarydist-dev Architecture: all -Depends: ${shlibs:Depends}, - ${misc:Depends}, - golang-go +Depends: + golang-go, + ${misc:Depends}, + ${shlibs:Depends}, Description: Go implementation of the bspatch algorithm This Go library implements binary diff and patch as described on http://www.daemonology.net/bsdiff/. It reads and writes Reformatted ./debian/control. DEBCRAFT WARNING: Command exited with code 2. ++ wrap-and-sort -ast [debian/master e3d9243] Apply consistent formatting to packaging files 1 file changed, 11 insertions(+), 8 deletions(-) Running policy-4.7.3-priority-optional.sh: ++ sed -i -E /^\s*Priority:\s*optional\s*$/d debian/control [debian/master 33f30f0] Drop redundant `Priority: optional` 1 file changed, 1 deletion(-) Running policy-4.7.3-rules-requires-root.sh: No 'Rules-Requires-Root: no' found Running policy-version.sh: ++ sed -i -E s/^\s*Standards-Version:\s*.*\s*$/Standards-Version: 4.7.4/ debian/control [debian/master 9023335] Bump Debian Policy version to 4.7.4 1 file changed, 1 insertion(+), 1 deletion(-) Running shellcheck.sh: Running spelling.sh: ++ codespell --write-changes --check-filenames --check-hidden --skip=debian/changelog,debian/patches,debian/vendor,debian/source/lintian-overrides,*.lintian-overrides,*.po,*.pot debian/ ++ debputy lint --spellcheck --auto-fix spelling: File: ./debian/changelog:7:29:7:39: Spelling "upstream's" [debputy] 7: * debian/gbp.conf: Support upstream's release tags, use pristine-tar spelling: File: ./debian/changelog:10:13:10:21: Spelling "Uploader" [debputy] 10: + Update Uploader email address None of the issues in ./debian/changelog could be fixed automatically. Sorry! auto-fixing: File: ./debian/control:17:0:18:0: The Section should be "golang" based on the package name [debputy] spelling: File: ./debian/control:24:38:24:45: Spelling "bspatch" [debputy] 24: Description: Go implementation of the bspatch algorithm Fixes applied to ./debian/control: 1. Number of issues went from 2 to 1 No issues detected in ./debian/copyright No issues detected in ./debian/rules No issues detected in ./debian/upstream/metadata No issues detected in ./debian/watch DEBCRAFT WARNING: Failed with exit code 2. [debian/master 0fb6163] Fix spelling 1 file changed, 1 insertion(+) Improving package complete The result happened because the command `debputy lint --spellcheck --auto-fix` does more than fixing typos, yet the commit message only mentions spelling fixes. I think I will refactor this to run `debputy linit --auto-fix` first and commit that, and only after that with --spellcheck ensuring the latter run only does spell fixes.
