| 1 |
On Monday, August 23, 2010 12:24:54 Michał Górny wrote: |
| 2 |
> The third revision, almost all suggestions applied. The ChangeLog: |
| 3 |
|
| 4 |
if there are suggestions you've actively ignored, those should be noted |
| 5 |
|
| 6 |
> 44188e0 escons(): set SCONSOPTS implicitly if they are unset. |
| 7 |
> ccf1ef9 Introduce SCONSOPTS and use it instead of MAKEOPTS. |
| 8 |
|
| 9 |
i'm not sure caching the value and using it in between runs is a good idea. |
| 10 |
unless you also cache the thing you're caching and compare it to make sure |
| 11 |
your cache is no longer invalid. |
| 12 |
|
| 13 |
i.e. export _SCONSOPTS_MAKEOPTS=${MAKEOPTS}, and then on every escons |
| 14 |
invocation, make sure ${MAKEOPTS} hasnt changed in which case you need to |
| 15 |
regenerate it. or just avoid the cache altogether and leave SCONSOPTS as a |
| 16 |
hook specific to scons. |
| 17 |
-mike |
Archives