SPECS: zsh.spec - allow to pass CPPFLAGS from environment

[Tomasz Wittner]

On Mon 25. September 2006 10:44, Arkadiusz Miskiewicz wrote:
> On Monday 25 September 2006 10:26, Jakub Bogusz wrote:
> > On Sun, Sep 24, 2006 at 09:06:24PM +0200, Tomasz Wittner wrote:
> > > On Sun 24. September 2006 19:47, twittner wrote:
> > > > Author: twittner                     Date: Sun Sep 24 17:47:20 2006 GMT
> > > > Module: SPECS                         Tag: HEAD
> > > > ---- Log message:
> > > > - allow to pass CPPFLAGS from environment
> > >
> > > [...]
> > >
> > > > -CPPFLAGS="-I/usr/include/ncurses"
> > > > +CPPFLAGS="-I/usr/include/ncurses $CPPFLAGS"
> > >
> > > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> > > To się nie podoba arekm (a podoba mi).
> > > Chodzi o to, czy wszelkie *FLAGS (i inne zmienne) ze środowiska mają mieć
> > > wpływ na budowanie rpm'a, czy nie (część ze zmiennych jest przez rpm
> > > unsetowana lub ustawiana na żądane wartości). Z jednej strony mamy
> > > zapewnione bardziej określone warunki budowania, z drugiej - mamy wygodną
> > > możliwość tuningu ad-hoc. Po za tym, czy ktoś wie, jaka jest powszechnie
> > > przyjęta praktyka odnśnie wpływu środowiska na budowanie?
> >
> > IMO środowisko nie powinno mieć wpływu - za łatwo coś zmienić
> > niekoniecznie świadomie. Dla CFLAGS, CXXFLAGS i LDFLAGS są makra.
> 
> Powinniśmy dodać czyszczenie środowiska przed uruchomieniem 
> skryptów %build, %install itd - wtej chwili tak naprawdę nasze 
> makro %configure jest wrażliwe na w/w zmienne środowiska jeśli ich w specu 
> jawnie nie nadpisano.
> 
I IMO dlatego powinno być:
CPPFLAGS="-I/usr/include/ncurses $CPPFLAGS"
zamiast
CPPFLAGS="-I/usr/include/ncurses"

dla spójności - skoro są wrażliwe, to wszystkie i w jednakowy sposób.

> Opinia jbj na ten temat: nie jest przeciw ale nie chce tego wprowadzić do rpma 
> bo nie chce potem dyskutować z ,,morons''.
> 
Czy wszystko ma być "czyszczone"?:
[inter w asus ~]$ bash
[inter w asus ~]$ set | grep CCACHE
CCACHE_DIR=/mnt/hda5/home/users/inter/.ccache
CCACHE_NOHASH_SIZE_MTIME=yes
CCACHE_NOLINK=yes
CCACHE_UMASK=007

alternatywa:

$ rpm -E %__cc
"CCACHE_DIR=/mnt/hda5/home/users/inter/.ccache CCACHE_NOHASH_SIZE_MTIME=yes 
CCACHE_NOLINK=yes CCACHE_UMASK=007 ccache athlon-pld-linux-gcc"

nie wygląda ciekawie.

BTW:
[inter w asus ~]$ set | grep 'CCACHE'
[inter w asus ~]$
[inter w asus ~]$ set | perl -n -e 'print $&, "\n" if /.*CCACHE.*/'
CCACHE_DIR=/mnt/hda5/home/users/inter/.ccache
CCACHE_NOHASH_SIZE_MTIME=yes
CCACHE_NOLINK=yes
CCACHE_UMASK=007
[inter w asus ~]$ echo $SHELL
/bin/zsh
????????

-- 
Tomasz Wittner