On Thursday, June 10, 2010 16:45:29 Arfrever Frehtes Taifersar Arahesis wrote: > 2010-06-10 22:20:44 Nirbheek Chauhan napisał(a): > > On Fri, Jun 11, 2010 at 1:30 AM, Arfrever Frehtes Taifersar wrote: > > > 2010-06-10 21:27:40 Jeremy Olexa napisał(a): > > >> I see no reason to *not* add a ChangeLog entry here. > > > > > > ChangeLog entries are not required for trivial changes. > > > > A "trivial" change is fixing a typo, or a manifest problem, a missing > > quotation mark, etc. Anything else is not "trivial". > > > > Anything that changes how an ebuild functions, what it does, or the > > installed files (and/or their contents) is NOT a trivial change. > > This commit only removed some compiler warnings.
mucking with CFLAGS without documentation is wrong. compiler warnings come
and go, so a flag that was relevant one day could be completely extraneous the
next.
however, especially with strict aliasing, you arent "just fixing warnings",
you're changing optimization behavior of gcc to workaround broken C code.
this obviously does not fall anywhere near the "trivial" mark.
i see you still havent fixed this, so get on it already. a bug needs to be
opened somewhere to get the package properly *fixed*, and your change either
out right reverted or add a comment around the flag to indicate the open bug
on the issue.
append-flags ... #12345
-mike
signature.asc
Description: This is a digitally signed message part.
