[opensuse-factory] Complain about conditional used run_permissions macro
Hi, we're using %{?run_permissions:%{run_permissions}} in the post scriptlet of the samba-krb-printing package. While building for opensuse-factory we get this warning [ 1762s] WARNING: run_permissions is obsolete and will be removed january 2013 As we execute %{run_permissions} only if it is defined all looks good from our side. Therefore we consider to simply ignore the warning. But I fear we'll come to the point where the package sources will not get accepted for openSUSE Factory. Our goal is to use the same spec file also for older products like SUSE Linux Enterprise 10 and 11. Cheers, Lars -- Lars Müller [ˈlaː(r)z ˈmʏlɐ] Samba Team + SUSE Labs SUSE Linux, Maxfeldstraße 5, 90409 Nürnberg, Germany
Hello, Am Mittwoch, 31. Oktober 2012 schrieb Lars Müller:
[ 1762s] WARNING: run_permissions is obsolete and will be removed january 2013
As we execute %{run_permissions} only if it is defined all looks good from our side.
Well, not really - if it is removed, nothing will be done anymore. And that's not what you want... (but "at least" you won't see an error message ;-)
Therefore we consider to simply ignore the warning.
But I fear we'll come to the point where the package sources will not get accepted for openSUSE Factory. Our goal is to use the same spec file also for older products like SUSE Linux Enterprise 10 and 11.
I can't speak for SLE 10/11, but... # from /usr/lib/rpm/suse_macros %run_permissions() \ echo "WARNING: run_permissions is obsolete and will be removed january 2013" echo "use set_permissions (available since 11.4) with a file argument" Maybe you should test if %set_permissions is available (and use it), with a fallback to %run_permissions. Or just use an %if block that checks for %suse_version > 1140 and switches between %set_permissions and %run_permissions ;-) Regards, Christian Boltz --
He Mann, isse krasse, erste Wort: "Kyptographie", isse schon falsch. Was machst du? Uuuhhh. Ist in der Listenkasse noch genug Geld für ein Wörterbuch? [> Ratti und Thorsten Haude in sl-etikette]
-- To unsubscribe, e-mail: opensuse-factory+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-factory+owner@opensuse.org
On Thu, Nov 01, 2012 at 03:27:45PM +0100, Christian Boltz wrote:
Hello,
Am Mittwoch, 31. Oktober 2012 schrieb Lars Müller:
[ 1762s] WARNING: run_permissions is obsolete and will be removed january 2013
As we execute %{run_permissions} only if it is defined all looks good from our side.
Well, not really - if it is removed, nothing will be done anymore. And that's not what you want... (but "at least" you won't see an error message ;-)
Therefore we consider to simply ignore the warning.
But I fear we'll come to the point where the package sources will not get accepted for openSUSE Factory. Our goal is to use the same spec file also for older products like SUSE Linux Enterprise 10 and 11.
I can't speak for SLE 10/11, but...
# from /usr/lib/rpm/suse_macros %run_permissions() \ echo "WARNING: run_permissions is obsolete and will be removed january 2013" echo "use set_permissions (available since 11.4) with a file argument"
Maybe you should test if %set_permissions is available (and use it), with a fallback to %run_permissions. Or just use an %if block that checks for %suse_version > 1140 and switches between %set_permissions and %run_permissions ;-)
Use %suse_version if you want to support both versions. CIao, Marcus -- To unsubscribe, e-mail: opensuse-factory+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-factory+owner@opensuse.org
participants (3)
-
Christian Boltz -
Lars Müller -
Marcus Meissner