Tommi Höynälänmaa <[email protected]> writes:

> Hello
>
> I have uploaded Guile Fluidsynth to Salsa. I have to wait for nyacc to
> be updated in Debian unstable before I can upload Guile Fluidsynth to
> Debian unstable. See
> <https://salsa.debian.org/scheme-team/guile-fluidsynth>.

Welcome to the team!

Quick review:

- Add d/upstream/metadata

- Send d/patches/ upstream, and mark them as Forwarded.

- Fix licenserecon warnings about d/copyright -- there are many
  copyright notices in the sources not present in d/copyright.

- Is it possible to write some non-superficial debian/tests/?  Based on
  the manual maybe?  Upstream really ought to have a test-suite as part
  of 'make check', so maybe ask upstream.

- Use d/watch v5 - upstream signs git tags, so I suggest adding a
  d/upstream/signing-key.asc and using a debian/watch like this:

Version: 5
Source: https://https.git.savannah.gnu.org/git/guile-fluidsynth.git
Matching-Pattern: refs/tags/v@ANY_VERSION@
Mode: git
Pgpmode: gittag

- Update Standards-Version, dropping Priority and Rules-Requires-Root.

- I see no need for a versioned dependency on texinfo in d/control.

- What is the purpose of this in debian/rules?
export DEB_HOST_MULTIARCH ?= $(shell dpkg-architecture -qDEB_HOST_MULTIARCH)

- Consider adding sign-tags=True and upstream-vcs-tag = v%(version%~%-)s
  to debian/gbp.conf.

/Simon

Attachment: signature.asc
Description: PGP signature

Reply via email to