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