1 |
On Sun, Jun 20, 2010 at 06:27:00PM -0400, Mike Frysinger wrote: |
2 |
> On Sunday, June 20, 2010 09:55:39 Arfrever Frehtes Taifersar Arahesis wrote: |
3 |
> > 2010-06-19 22:53:37 Mike Frysinger napisał(a): |
4 |
> > > On Thursday, June 10, 2010 16:45:29 Arfrever Frehtes Taifersar wrote: |
5 |
> > > > 2010-06-10 22:20:44 Nirbheek Chauhan napisał(a): |
6 |
> > > > > On Fri, Jun 11, 2010 at 1:30 AM, Arfrever Frehtes Taifersar wrote: |
7 |
> > > > > > 2010-06-10 21:27:40 Jeremy Olexa napisał(a): |
8 |
> > > > > >> I see no reason to *not* add a ChangeLog entry here. |
9 |
> > > > > > |
10 |
> > > > > > ChangeLog entries are not required for trivial changes. |
11 |
> > > > > |
12 |
> > > > > A "trivial" change is fixing a typo, or a manifest problem, a missing |
13 |
> > > > > quotation mark, etc. Anything else is not "trivial". |
14 |
> > > > > |
15 |
> > > > > Anything that changes how an ebuild functions, what it does, or the |
16 |
> > > > > installed files (and/or their contents) is NOT a trivial change. |
17 |
> > > > |
18 |
> > > > This commit only removed some compiler warnings. |
19 |
> > > |
20 |
> > > mucking with CFLAGS without documentation is wrong. compiler warnings |
21 |
> > > come and go, so a flag that was relevant one day could be completely |
22 |
> > > extraneous the next. |
23 |
> > > |
24 |
> > > however, especially with strict aliasing, you arent "just fixing |
25 |
> > > warnings", you're changing optimization behavior of gcc to workaround |
26 |
> > > broken C code. this obviously does not fall anywhere near the "trivial" |
27 |
> > > mark. |
28 |
> > > |
29 |
> > > i see you still havent fixed this, so get on it already. a bug needs to |
30 |
> > > be opened somewhere to get the package properly *fixed* |
31 |
> > |
32 |
> > This problem is probably caused by bugs in Python 2, which have been fixed |
33 |
> > in Python 3. |
34 |
> |
35 |
> the new information you've provided here only reinforces the fact the current |
36 |
> code is wrong. properly document the append-flags and properly restrict it to |
37 |
> when python-2 is being used. |
38 |
> |
39 |
> considering python-2 is going to be in our lives for a while, how hard is it |
40 |
> to backport the changes in question to the headers ? presumably it's |
41 |
> struct/union/cast shuffling in the headers. |
42 |
|
43 |
This is upstream python bug 969718 offhand; basically if cflags exists |
44 |
for distutils consumers, base cflags don't make it fully through- |
45 |
meaning no -fno-strict-aliasing as is generally needed for building |
46 |
python extensions. |
47 |
|
48 |
As for py3k, looks of it py3.1 still suffers it. |
49 |
|
50 |
|
51 |
Either way, this is the wrong thing to fix- python's distutils |
52 |
needs fixing, not consumers. In snakeoil, we detect and fix it on the |
53 |
fly to provide a fixed version of distutils, but obviously not many |
54 |
pkgs consume that... |
55 |
|
56 |
~harring |