On Tue, Jun 08, 2010 at 11:36:44 (CEST), Jonas Smedegaard wrote:

> On Mon, Jun 07, 2010 at 08:28:05PM +0000, 
> fabian-gu...@users.alioth.debian.org wrote:
>>The following commit has been merged in the master branch:
>>commit 552a01450aa8c3492e3264eb8509254488b99d21
>>Author: Fabian Greffrath <fab...@debian-unofficial.org>
>>Date:   Mon Jun 7 21:56:08 2010 +0200
>>    Add debian/gbp.conf.
>>diff --git a/debian/gbp.conf b/debian/gbp.conf
>>new file mode 100644
>>index 0000000..1615f89
>>--- /dev/null
>>+++ b/debian/gbp.conf
>>@@ -0,0 +1,6 @@
>>+upstream-branch = upstream
>>+debian-branch = master
>>+upstream-tag = upstream/%(version)s
>>+debian-tag = debian/%(version)s
>>+pristine-tar = True
> I noticed above several times, and have a general comment:
> I suggest to only state unusual things, not defaults.
> Concretely I suggest to only state the following:
> pristine-tar = True

What problem does this solve?

Removing the lines forces maintainers to stick with the defaults of the
removed fields. Not that I would do that, and TBH, I don't have a
particularly strong opinion on this, I'm just wondering. But it seems
that you have an opinion on this because you mentioned it here. I'm
trying to understand it.

Reinhard Tartler, KeyID 945348A4

pkg-multimedia-maintainers mailing list

Reply via email to