Mailinglist Archive: opensuse-buildservice (132 mails)

< Previous Next >
Re: [opensuse-buildservice] Re: [opensuse-packaging] To %make_install or to %makeinstall (spec-cleaner)
On Tue, May 15, 2012 7:02 am, Pascal Bleser wrote:
On 2012-05-15 01:42:36 (+0200), Ruediger Meier <sweet_f_a@xxxxxx> wrote:
Spec cleaner was written to support only the latest

The I can only recommend and ask that nobody uses spec cleaner.
There are packagers who care about also building for older
versions of openSUSE/Evergreen, for SLE, and even older RHEL.

It's sad to break package builds just because of spec file
syntax (if the deps are too old or not available, etc..., fine,
but not because of a spec syntax gimmick, cmon.)

I agree, so why not a spec cleaner that does all the nice things of the
current spec cleaner.
Puts the BuildRequires and Requires each in one line and in an
alphabetical order.
The replaces /usr/lib(64) by %{_libdir} etc

And it replaces %makeinstall %make_install etc by a universal %__make
install DESTDIR='%{buildroot}' ?

I think spec-cleaner is a very good idea. But I think it's moving a bit to
fast, i's ahead of the current situation.

Might it be option to translate into %makeinstall instead of

Using %makeinstall in a spec file is a bad idea. While it makes it
work on SLE, it breaks it on Fedora where %makeinstall is still the
old broken variant. Defining %make_install on SLE might be the better

I currently just replace %make_install directly with its first-level
expansion, that seems to be only thing working across all
OBS-supported Distros (but obviously might cause problems in the

I'm using

%{?make_install} %{!?make_install:make install DESTDIR=%{buildroot}}

for all my packages. This works for all repos and spec-cleaner does not
touch it.

... and entirely defeats the point of having a macro as a
shortcut :)

Why not just do
%__make install DESTDIR='%{buildroot}'
It's shorter, and works ;)

I mean, the expansion shouldn't depend on the distro.
It depends on the Makefile: either the Makefile has DESTDIR
support, or it doesn't.

We need one macro that _always_ does DESTDIR, on every distro,
and possibly one macro that always does it "the old way" (though
it's probably better to patch DESTDIR support into the Makefile

%define __makeinstall %__make install DESTDIR='%{buildroot}'

What are the thongs about the removal of %clean part of the .spec file?

-o) Pascal Bleser
/\\ -- we haz green
_\_v -- we haz conf



To unsubscribe, e-mail: opensuse-buildservice+unsubscribe@xxxxxxxxxxxx
To contact the owner, e-mail: opensuse-buildservice+owner@xxxxxxxxxxxx

< Previous Next >