On Sat, Aug 14, 2010 at 04:37:04PM +0300, Alexis Ballier wrote:
> On Saturday 14 August 2010 15:50:53 Markos Chandras wrote:
> > On Sat, Aug 14, 2010 at 03:35:34PM +0300, Alexis Ballier wrote:
> > > On Saturday 07 August 2010 00:21:39 Markos Chandras (hwoarang) wrote:
> > > > hwoarang    10/08/06 21:21:39
> > > > 
> > > >   Modified:             ChangeLog
> > > >   Added:                mlt-0.5.4-r1.ebuild
> > > >   Log:
> > > >   Respect {C,LD}FLAGS when building shared library. Bug #308873
> > > >   (Portage version: 2.2_rc67/cvs/Linux x86_64)
> > > 
> > > While fixing bugs can't be bad and I thank you for doing it, I can see a
> > > couple of important quality problems in this commit:
> > > 
> > > - There is absolutely no reference to any patch sent upstream and I have
> > > not seen anything on the upstream dev ml.
> > 
> > Thats because I didn't. I've fixed more than 40 bug wrt LDFLAGS. Do you
> > expect me to subscribe to 40 different ML and send them upstream? 
> 
> you don't need to subscribe, there's usually an AUTHORS file with emails you 
> can use...
As I said, I thought that maintainers was responsible to do it since they
follow all the bug progress after all. So according to you I should do all the
work. Tempting
> 
> > The
> > patch is there, the maintainer is CC on the bug. All he has to do it to
> > send this damn patch to upstream.
> 
> I can use the same reasoning and ask:
> Why don't you do it in the first place if that's "all" ?
Cause I cannot maintain all the tree myself
> 
> > I only care about the QA status on tree.
> 
> As I already said, that's good, but that's better achieved with long term 
> fixes rather than quick hacks IMHO
> 
> > Most of them just use my patches and
> > contact upstream themselves. If this doesn't apply for you just let me
> > know.
> 
> Yes this doesn't apply to me because the most probable scenario will be this: 
> I'll touch the package in a couple of months/years, do a review of the 
> ebuild/patches, find out some patches need porting, waste time trying to 
> figure out why it's there in the first place, see it's been there for ages 
> and 
> that the author didn't consider the fix good enough to upstream it, drop it.
> 
Sure, the changelogs are there though. I am trying to always write down as many
details as I can so the maintainer can easily track down changes.
> > > - If you are not in cc of the gentoo bug nor in the herd alias, please cc
> > > yourself on the bug.
> > > - Please close the bugs, even the dupes (and apply previous point to the
> > > dupes too).
> > > - That way you'll be able to quickly fix (apparently, I didn't check)
> > > obvious mistakes [1].
> > > - You'll have to do a rev. bump for *FLAGS respect, please also check if
> > > you can avoid it by doing a version bump instead.
> > 
> > Well not always. If something is on ~testing then I don't think I should
> > "spam" the tree with revbumps. Stable users are my first priority so
> > unless something is on stable branch, I fix it as it is. I don't want to
> > version bump anything because I don't want to mess with anyones
> > packages.
> 
> You're messing much more with one's package with quick'n'dirty "fixes" than 
> with a clean version bump with upstreamed patches...
Quick and dirty? Fair enough. Will try to contact upstream from now on. Seems
like I will maintain the entire tree in the end.
> 
> > I only do QA fixing. If you have problem touching your
> > packages just say it
> 
> I don't have problems with anyone touching "my" packages (esp. when they're 
> herds packages...); though when I'm not happy with the technical details I 
> let 
> it be known and _really_ appreciate when the comments are taken into account 
> instead of aggressively discarded by trying to argue why it's not been 
> perfect 
> in the first place ;)
> 
> A.
> 
I don't think what I do is perfect. But all this kind of judgement is
quite demotivated I must say.

-- 
Markos Chandras (hwoarang)
Gentoo Linux Developer
Web: http://hwoarang.silverarrow.org
Key ID: 441AC410
Key FP: AAD0 8591 E3CD 445D 6411 3477 F7F7 1E8E 441A C410

Attachment: pgpHlH84uLr9R.pgp
Description: PGP signature

Reply via email to