[meego-packaging] Spectacle: invalid .spec file created for python packages

[Ville M. Vainio]

On Tue, Jul 27, 2010 at 10:50 PM,  <fathi.boudra at nokia.com> wrote:
>> I meant that adding the special case for opensuse would have been the
>> right thing to do (because it wouldn't do any harm for other distros,
>> apart from slightly bloating the .spec file).
>
> I tend to disagree here. We aren't always sure it wouldn't do any harm.
> Different distributions means also different Python policy, including MeeGo.
> Adding the prefix could work in some case and not in others...
> and if we change/update policy, it will break packages.
> IMHO, adding the prefix is an exception, not the default use case.

But the prefix is explicitly special cased in the generated .spec file:

%if 0%{?suse_version}
%{__python} setup.py install --root=$RPM_BUILD_ROOT --prefix=%{_prefix}
%else
%{__python} setup.py install --root=$RPM_BUILD_ROOT -O1
%endif

OTOH, I don't really care that much, as it works okay with the
spectacle option you specified. It's just that there is not much of a
googleable culture around the spectacle tool yet, so I got tripped by
this simple matter.

-- 
Ville M. Vainio @@ Forum Nokia