On Mon, May 14, 2012 at 12:22 PM, Cristian Morales Vega wrote:

don't do it. %make_install is the correct one, defined in upstream rpm since version 4.8.0. Personally I favour cross-distro compatibility to compatibility with old distros.

Whether it's old or not depends on which version of SLE broke, I'd guess. There it says SLE 11 SP1, which doesn't seem that old. -- To unsubscribe, e-mail: opensuse-packaging+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-packaging+owner@opensuse.org

On Mon, May 14, 2012 5:28 pm, Pavol Rusnak wrote:

On 14/05/12 17:07, Joop Boonen wrote:

...

I wonder if this should be fixed, in either spec-cleaner (use%makeinstall in stead of %make_install) or in the buildserver (add macro %make_install for SLE)?

Spec cleaner was written to support only the latest distributions.

I like spec cleaner a lot and I think it really helps cleaning up a spec file. Might it be option to translate into %makeinstall instead of %make_install? I assume spec-cleaner is still developed?

-- Best Regards / S pozdravom,

Pavol Rusnak PGP 0xB9A02A3D prusnak[at]opensuse.org --

Regards, Joop. -- To unsubscribe, e-mail: opensuse-packaging+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-packaging+owner@opensuse.org

On Mon, May 14, 2012 6:09 pm, Frank Lichtenheld wrote:

...

-----Original Message----- From: Joop Boonen [mailto:joop.boonen@boonen.org] Sent: Monday, May 14, 2012 5:53 PM To: Pavol Rusnak Cc: opensuse-buildservice@opensuse.org; opensuse-packaging@opensuse.org Subject: [opensuse-buildservice] Re: [opensuse-packaging] To %make_install or to %makeinstall (spec-cleaner)

On Mon, May 14, 2012 5:28 pm, Pavol Rusnak wrote:

...

On 14/05/12 17:07, Joop Boonen wrote:

...

I wonder if this should be fixed, in either spec-cleaner (use%makeinstall in stead of %make_install) or in the buildserver (add macro %make_install for SLE)?

Spec cleaner was written to support only the latest distributions.

I like spec cleaner a lot and I think it really helps cleaning up a spec file.

Might it be option to translate into %makeinstall instead of %make_install?

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 option.

Might Christopher Yeleighton purposed work around be a solution, for the time being? Workaround: %if 0%{!?make_install:1} %define make_install make install 'DESTDIR=%{buildroot}' %endif If so then would it be an option to add this define to spec-cleaner?

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 future).

Gruesse, Frank -- Regards,

Joop. -- To unsubscribe, e-mail: opensuse-packaging+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-packaging+owner@opensuse.org

On Mon, 14 May 2012 18:25:30 +0200, Pavol Rusnak wrote:

Sorry. Spec-cleaner aims to remove cruft from spec files, not to add new one.

And? What is so hard about simply adding a line that defines make_install in terms of makeinstall if the former isn't present? That's not what I call cruft. But couldn't the redefinition be done done in the prjconf? That would IMHO be a usable compromise if possible. Philipp -- To unsubscribe, e-mail: opensuse-packaging+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-packaging+owner@opensuse.org

On Tue, 04 Sep 2012 23:31:52 +0200, Dimstar / Dominique Leuenberger wrote:

Could (and should preferably) be done on SUSE:SLE-11:SP[12] prjconf directly.

Care to post an example? I at least couldn't do it, but my knowledge of such OSC internals is limited.

Or even better: get SLE finally up to a level where it uses a bunch of stuff others have been using for a while.

That would, IMNSHO, indeed be the best solution. I'll see if I can at least ask a few people. Philipp -- To unsubscribe, e-mail: opensuse-packaging+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-packaging+owner@opensuse.org

On Wed, Sep 5, 2012 at 1:16 AM, Philipp Thomas wrote:

On Tue, 04 Sep 2012 23:31:52 +0200, Dimstar / Dominique Leuenberger wrote:

...

Or even better: get SLE finally up to a level where it uses a bunch of stuff others have been using for a while.

That would, IMNSHO, indeed be the best solution. I'll see if I can at least ask a few people. Philipp

I already discussed the issue with someone from the SLE team a few months ago, he said they would discuss it and possibly include it (and a few other macros) in SLE11 SP3, but there wasn't a firm commitment. -Todd -- To unsubscribe, e-mail: opensuse-packaging+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-packaging+owner@opensuse.org