[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: RFS: opendchub (updated package)



-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Zak B. Elep schrieb:
> On Tue, 2009-08-25 at 18:43 +0200, Patrick Matthäi wrote:
>> Sorry, you made a step in the wrong way:
>> * Remove debian/patches, now merged in git
>>
>>  Makefile              |  588
>> ++++++++++++++++++++++++++++++++++++++++++++++++++
>>  src/commands.c        |    2
>>  src/perl_utils.c      |    9
>>  19 files changed, 1196 insertions(+), 2 deletions(-)
> 
> Good call, that Makefile shouldn't be there, its not in the new upstream
> sources but seems to have snuck in the last NMU.  I have uploaded
> 0.8.0-2 fixing this:

 src/commands.c        |    2
 src/perl_utils.c      |    9 ++-

There are still those files left.
I will not sponsor packages who have direct changes in sources.

> 
> http://mentors.debian.net/debian/pool/main/o/opendchub/opendchub_0.8.0-2.dsc
> 
>> You should *never* do modifications directly in the source, the
>> debian/patches behaviour was correct.
> 
> I never did modifications on the upstream source directly; I had git
> branches[0] set up with their own copies of the source, so modifications
> are self-contained within those branches.
> 
>> Please revert it.
> 
> Please see this[1] for my rationale behind the debian/patches move.
> 
> [0]  See the gitweb at http://code.zakame.net/opendchub.git
> [1]  Message-Id: <1251211701.3814.57.camel@perlis.zakame.net>
> 


- --
/*
Mit freundlichem Gruß / With kind regards,
 Patrick Matthäi
 GNU/Linux Debian Developer

E-Mail: pmatthaei@debian.org
        patrick@linux-dev.org

Comment:
Always if we think we are right,
we were maybe wrong.
*/
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)

iEYEARECAAYFAkqUIYIACgkQ2XA5inpabMenUACfaYD293Xo0S5UXGoO6kT5DWp4
PxIAoI2fVPs+SCkz9EhhkO40kVu5pK4b
=Npsb
-----END PGP SIGNATURE-----


Reply to: