| 1 |
On Thursday 10 June 2010 23:45:29 Arfrever Frehtes Taifersar Arahesis wrote: |
| 2 |
[...] |
| 3 |
> This commit only removed some compiler warnings. |
| 4 |
|
| 5 |
This adds a cflag preventing the compiler to make some assumptions on the code |
| 6 |
for its optimisations, hence making the code slower/bigger. While the |
| 7 |
changelog entry will help figuring out why this was added, a comment in the |
| 8 |
ebuild explaining why will be even better. |
| 9 |
Once you'll have to dig into 4 years of cvs log to figure out wth a cflag |
| 10 |
filtering/addition was added into an ebuild maybe you'll reconsider your |
| 11 |
opinion. |
| 12 |
|
| 13 |
Please always keep in mind that you're not the only one that will ever commit |
| 14 |
to $package and even less the only one that will have a look at a given |
| 15 |
ebuild. |
| 16 |
|
| 17 |
A. |
Archives