SPECS: jacl.spec - Version: 1.3.2 - use jpackage and current jdk l...

[Elan Ruusamäe]

On Sunday 21 May 2006 19:51, Jacek Konieczny wrote:
> On Sun, May 21, 2006 at 07:18:06PM +0300, Elan Ruusamäe wrote:
> > On Sunday 21 May 2006 18:56, jajcus wrote:
> > > +unset JAVA_HOME || :
> > > +export JAVA_HOME="%{java_home}"
> >
> > unset is unneccessary when you set value for it on next line
>
> In this case it _is_ neccessary -- %{java_home} uses $JAVA_HOME for
> evaluation.
>
> Should I add the comment to every spec which uses this construction?
> Or maybe just fix %{jave_home}? It uses shell function, for which using
> $JAVA_HOME makes sense, but not during package building.
yeah. rpm macro should be fixed then. didn't even think the macro could be 
shell function.

-- 
glen