On Thu, Nov 08, 2007 at 03:59:44PM +0100, Bas Wijnen wrote:
> Making changes to make the build work is always good, of course.
> However, when changes are made for the Debian package, this should be
> done in a way which doesn't hide them.  When a user sees a package where
> the tarball is repackaged "because some files had to be removed", she's
> not going to expect any changes other than the removal of some files.
> For other changes, we have a nicely working patch system.

That is all documented in README.Debain-source. All the repackaged
packages of mine have such a file which describe, in detail, what I
do. In that sense, it is not hidden.

> > Also, my patches looked too ugly.
> 
> And that's not an argument not to use it. :-)  If you need to do ugly
> things, then it should look ugly.  Putting those changes somewhere where
> they're hard to see may cause other problems which are hard to track
> down (not for you, but for other people who are expecting the have the
> original source).  Especially if those changes are needed to make the
> build work, it is important that they are visible like other changes,
> and that's in the .diff.gz (directly or through a patch system, whatever
> you prefer).

I agree on this, and you are very correct. But my mentor recommended
the change, as we were repackaging it anyway. Besides, noting whatever
I've done in README.Debian-source does alleviate suffering a
little. But it's always tough when repackaging... *sigh*

Kumar
-- 
Kumar Appaiah,
458, Jamuna Hostel,
Indian Institute of Technology Madras,
Chennai - 600 036


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to