Re: Heads up: lua-lpeg 1.0.1 in Rawhide; lua no longer in mock root by default?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 01/15/2017 05:44 PM, Neal Gompa wrote:
On Sun, Jan 15, 2017 at 12:24 AM, Michel Alexandre Salim
<michel@xxxxxxxxxxxxxxx> wrote:
Hi all,

Just a heads-up: I just built lua-lpeg 1.0.1 for Rawhide. The release notes
does not seem to suggest any significant API change, but just to be sure we
should probably let it sit in Rawhide for a while.

On the packaging side, turns out Lua is not available by default in the mock
root, and a lot of our Lua packages try to detect the version of Lua used
during the build by using this expansion:


  %{!?luaver: %global luaver %(lua -e "print(string.sub(_VERSION, 5))")}

This now fails because the lua binary is not available when mock rebuilds
the SRPM. For anyone updating other Lua packages the following works:

  %{!?luaver: %global luaver %(test -x /usr/bin/lua && lua -e
"print(string.sub(_VERSION, 5))" || echo 5.3)}

(I'll update my draft Lua packaging guidelines with this, if it looks good
to others)

The lua package was recently refactored to split the libraries out
into lua-libs, so you need to add "BuildRequires: lua" or
"BuildRequires: /usr/bin/lua" to get it specifically.

Note that Lua being special to rpm, and as long as there's just one version of Lua in Fedora, you don't actually need the lua executable or buildrequires just for that:

%{!?luaver: %global luaver %{lua:print(string.sub(_VERSION, 5))}}


	- Panu -
_______________________________________________
devel mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [Fedora Kernel]     [Fedora Testing]     [Fedora Formulas]     [Fedora PHP Devel]     [Kernel Development]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Gimp]     [Yosemite News]
  Powered by Linux