Looks much better, thanks.

One thing I didn't notice in the previous round:

On Wed, 2009-02-11 at 00:05 +0800, Ke Wang wrote:
> %define _demodir %(echo %{_bindir} | sed -e 
> 's,%{_prefix},%{_prefix}/demo/jds,')

Since %{_bindir} is %{_prefix}/bin, this could be written as:

%define _demodir %{_prefix}/demo/jds/bin

Laca



Reply via email to