On 10/11/2011 09:13 PM, Fabian Groffen wrote:
> On 11-10-2011 21:01:40 +0300, Samuli Suominen wrote:
>> On 10/11/2011 08:05 PM, Fabian Groffen wrote:
>>> On 11-10-2011 19:59:13 +0300, Samuli Suominen wrote:
>>>> So I've missed one ${EPREFIX} for docdir= ? How about just fixing that,
>>>> and not crapping all over the package?
>>>
>>> How about first asking the maintainer before you completely rewrite an
>>> ebuild? I'm not innocent on this topic either (ask Diego for example),
>>> and I do allow you to make a lot of changes to my packages. Just don't
>>> force your style and preferences on me.
>>
>> So basically you are saying you reverted tehnically correct changes for
>> cosmetics. What ticked you off, the \ lines changes? I believe that was
>> the only change that wasn't about fixing something broken.
>
> No, you broke the package for Prefix. Next you bumped it to EAPI=4,
> then you removed SRC_URI for no particular reason, dropped libtool files
> and dropped static archives. Next to this you did some reordering and
> other cosmetic changes.
>
>> And so have you, changed dozens of my ebuilds for PREFIX compability or
>> other random fixes, not everything turned out correct, mistakes were
>> made and were clearly accidental. I've fixed them instead of wasting
>> both of our times. Is it too much to ask for same in return?
>
> I already told you that you can change quite a lot to my ebuilds,
> without me complaining. Do you mind me reverting your stuff and redoing
> it now?
>
>
Thanks, the end result of installed files look now OK. Care to reopen
the stabilization bug? The changes are trivial.
I just hope nobody will take an example of the ebuild with code
duplication (multiple epatch calls), overquoting, redudant use of find
when rm is more than enough, ...