Bug 1279091 (openra-eluant) - Review Request: openra-eluant - Lua C# bindings
Summary: Review Request: openra-eluant - Lua C# bindings
Status: CLOSED NOTABUG
Alias: openra-eluant
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Keywords:
Depends On:
Blocks: openra
TreeView+ depends on / blocked
 
Reported: 2015-11-07 20:10 UTC by Raphael Groner
Modified: 2016-11-05 22:11 UTC (History)
3 users (show)

(edit)
Clone Of:
(edit)
Last Closed: 2016-11-05 22:11:43 UTC


Attachments (Terms of Use)

Description Raphael Groner 2015-11-07 20:10:35 UTC
Spec URL: eluant.spec
SRPM URL: <srpm info here>
Description: C# bindings for Lua with a specific focus on building sandboxes
Fedora Account System Username: raphgro

This is just a place holder for an upcoming package request.

URL: https://github.com/OpenRA/Eluant

Comment 1 Matthias Mailänder 2015-11-24 13:16:24 UTC
See https://build.opensuse.org/package/show/games:openra/eluant

Comment 2 Raphael Groner 2016-05-16 15:51:34 UTC
Spec URL: https://raphgro.fedorapeople.org/review/openra/openra-eluant.spec
SRPM URL: https://raphgro.fedorapeople.org/review/openra/openra-eluant-20160124-1.src.rpm
Description: Lua C# bindings
Fedora Account System Username: raphgro

Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=14110187

Comment 3 Timotheus Pokorra 2016-05-17 08:40:49 UTC
Raphael notified me of a problem: System.DllNotFoundException : lua51.dll

upstream says: "If you have issues with DllNotFoundException being thrown, make sure that liblua5.1.so or lua5.1.dll are in your dynamic library search path."

The solution would be to include a line  
<dllmap dll="lua51" target="liblua5.1.so" os="!windows"/> 
in file /etc/mono/config, similar to how it works with libgdiplus.

I have not tested that.

If that works, please file a new bug for the mono package, and I will add that line to /etc/mono/config.

Comment 4 Matthias Mailänder 2016-05-17 20:40:50 UTC
Eluant is not part of the Mono project so I am not sure if that would be best practice. Keeping the .dll.config file in the eluant package might be an easy and straightforward solution. Upstream just didn't have cross platform distribution of the library in mind and sadly there is no standardized .so name for Lua 5.1 libraries across distributions.

<dllmap os="linux" dll="lua51.dll" target="liblua5.1.so" /> should work as well (for Fedora).

Comment 5 gil cattaneo 2016-08-15 20:59:27 UTC
Build fail: No matching package to install: 'nunit = 2.6.4'

Comment 6 Timotheus Pokorra 2016-08-15 21:36:42 UTC
I have upgraded nunit to version 3.4.1 in Rawhide.
Please try if you can patch the package to use NUnit3, otherwise there is a review request for nunit2 package for compatibility reasons: https://bugzilla.redhat.com/show_bug.cgi?id=1366713

Comment 7 Raphael Groner 2016-08-28 16:29:24 UTC
(In reply to Timotheus Pokorra from comment #6)
> I have upgraded nunit to version 3.4.1 in Rawhide.
> Please try if you can patch the package to use NUnit3, otherwise there is a
> review request for nunit2 package for compatibility reasons:
> https://bugzilla.redhat.com/show_bug.cgi?id=1366713

@Timotheus Can you please try to also rebuild sharpziplib, restsharp and newtonsoft-json with the new nunit, I could give you commit rights to update the packages. All three packages fail in koschei due to the broken nunit dependency. Sorry, I do not have the time currently to do it by myself.

Comment 8 Raphael Groner 2016-09-29 19:08:11 UTC
Easy fix for lua51:
sed -i s:lua51.dll:liblua-5.1.so: %{libname}/LuaApi.cs


Note You need to log in before you can comment on or make changes to this bug.