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 @@ >>+[DEFAULT] >>+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: > > [DEFAULT] > 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. -- Gruesse/greetings, Reinhard Tartler, KeyID 945348A4 _______________________________________________ pkg-multimedia-maintainers mailing list pkg-multimedia-maintainers@lists.alioth.debian.org http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers