https://bugzilla.redhat.com/show_bug.cgi?id=156477
--- Comment #52 from Mike FABIAN <mfabian(a)redhat.com> ---
(In reply to Carlos O'Donell from comment #50)
> (In reply to Mike FABIAN from comment #49)
> > I can try whether rpm.expand does the job.
>
> Mike, Would you give it a try and see? That way we can unblock the copr
> rawhide builds by removing the dependency on this apparently new feature.
> Otherwise we'll have broken rawhide builds for a month or more before the
> copr upgrade completes (to using F21 build hosts).
Yes, this change works:
diff --git a/glibc.spec b/glibc.spec
index a845182..2c166ad 100644
--- a/glibc.spec
+++ b/glibc.spec
@@ -1579,22 +1579,22 @@ end
%postun -p /sbin/ldconfig
-%triggerin common -e -p <lua> -- glibc
+%triggerin common -p <lua> -- glibc
if posix.stat("%{_prefix}/lib/locale/locale-archive.tmpl", "size") > 0 then
pid = posix.fork()
if pid == 0 then
- posix.exec("%{_prefix}/sbin/build-locale-archive", "--install-langs",
"%%{_install_langs}")
+ posix.exec("%{_prefix}/sbin/build-locale-archive", "--install-langs",
rpm.expand("%%{_install_langs}"))
elseif pid > 0 then
posix.wait(pid)
end
end
-%post common -e -p <lua>
+%post common -p <lua>
if posix.access("/etc/ld.so.cache") then
if posix.stat("%{_prefix}/lib/locale/locale-archive.tmpl", "size") > 0 then
pid = posix.fork()
if pid == 0 then
- posix.exec("%{_prefix}/sbin/build-locale-archive", "--install-langs",
"%%{_install_langs}")
+ posix.exec("%{_prefix}/sbin/build-locale-archive", "--install-langs",
rpm.expand("%%{_install_langs}"))
elseif pid > 0 then
posix.wait(pid)
end
lines 1-37/37 (END)
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=PlcWtCDVHS&a=cc_unsubscribe
https://bugzilla.redhat.com/show_bug.cgi?id=156477
--- Comment #51 from Panu Matilainen <pmatilai(a)redhat.com> ---
There's no such thing as rpm.getconf() and never was, but rpm.expand() goes
back to at least RHEL-5 and does exactly what the name says - macro expansion.
For the full list of rpm lua extensions see
http://rpm.org/wiki/PackagerDocs/RpmLua
In the future if you need advise on rpm, I'd suggest asking the rpm maintainers
directly to avoid having to guess.
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=pGMNNSoq21&a=cc_unsubscribe
https://bugzilla.redhat.com/show_bug.cgi?id=156477
--- Comment #50 from Carlos O'Donell <codonell(a)redhat.com> ---
(In reply to Mike FABIAN from comment #49)
> I can try whether rpm.expand does the job.
Mike, Would you give it a try and see? That way we can unblock the copr rawhide
builds by removing the dependency on this apparently new feature. Otherwise
we'll have broken rawhide builds for a month or more before the copr upgrade
completes (to using F21 build hosts).
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=LM18683y9K&a=cc_unsubscribe
https://bugzilla.redhat.com/show_bug.cgi?id=156477
Mike FABIAN <mfabian(a)redhat.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Flags|needinfo?(mfabian(a)redhat.co |
|m) |
--- Comment #49 from Mike FABIAN <mfabian(a)redhat.com> ---
(In reply to Carlos O'Donell from comment #45)
> (In reply to Panu Matilainen from comment #44)
> > -%post common -e -p <lua>
> > +%post common -p <lua>
> > if posix.access("/etc/ld.so.cache") then
> > if posix.stat("%{_prefix}/lib/locale/locale-archive.tmpl", "size") > 0
> > then
> > pid = posix.fork()
> > if pid == 0 then
> > - posix.exec("%{_prefix}/sbin/build-locale-archive", "--install-langs",
> > "%%{_install_langs}")
> > + posix.exec("%{_prefix}/sbin/build-locale-archive", "--install-langs",
> > rpm.expand("%%{_install_langs}"))
> > elseif pid > 0 then
> > posix.wait(pid)
> > end
>
> I believe we tried this before but it had some negative consequences. I'm
> going to ask Fabian to comment and test it out.
>
> Mike,
>
> Do you remember if we talked about using rpm.expand and then didn't?
We didn’t talk about rpm.expand.
Florian Weimer mentioned rpm.getconf:
<fweimer> mfabian: I think you need to add something like:
[15年02月09日 14:11:35]
<fweimer> posix.exec("/usr/sbin/build-locale-archive",
rpm.getconf("_install_langs")) [15年02月09日 14:12:05]
But later said:
<fweimer> mfabian: Use -e if it works. I was mistaken, there is no way to
expand RPM macros from the current transaction using Lua.
[15年02月09日 15:48:02]
* fweimer sighs. [15年02月09日 15:48:12]
I can try whether rpm.expand does the job.
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=ZY7A5WMZds&a=cc_unsubscribe
https://bugzilla.redhat.com/show_bug.cgi?id=1160508
Leslie Satenstein <lsatenstein(a)yahoo.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Flags|needinfo?(lsatenstein@yahoo |
|.com) |
--- Comment #10 from Leslie Satenstein <lsatenstein(a)yahoo.com> ---
Please close, is fixed for Fedora22
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=4OuXx8gJGb&a=cc_unsubscribe