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.
- 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.


A.



[1] https://bugs.gentoo.org/show_bug.cgi?id=332523

Reply via email to