[SCM] libebml packaging branch, master, updated. debian/0.7.7-3.1-14-g552a014

Jonas Smedegaard dr at jones.dk
Tue Jun 8 12:11:19 UTC 2010


On Tue, Jun 08, 2010 at 01:37:24PM +0200, Reinhard Tartler wrote:
>On Tue, Jun 08, 2010 at 11:36:44 (CEST), Jonas Smedegaard wrote:
>
>> On Mon, Jun 07, 2010 at 08:28:05PM +0000, fabian-guest at users.alioth.debian.org wrote:
>>>The following commit has been merged in the master branch:
>>>commit 552a01450aa8c3492e3264eb8509254488b99d21
>>>Author: Fabian Greffrath <fabian at 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.

It solves no "problem", but improves readability to not list unnecessary 
information.

It seems from above that you consider the information necessary.  Could 
you elaborate on that, as I fail to understand how that is.


Kind regards,

  - Jonas

-- 
  * Jonas Smedegaard - idealist & Internet-arkitekt
  * Tlf.: +45 40843136  Website: http://dr.jones.dk/

  [x] quote me freely  [ ] ask before reusing  [ ] keep private
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.alioth.debian.org/pipermail/pkg-multimedia-maintainers/attachments/20100608/14d7cfe2/attachment.pgp>


More information about the pkg-multimedia-maintainers mailing list