Hi Hisham,

>> I compared the rockspec I submitted to the rockspec in the *.rock file
>> and it appears that the values in external_dependencies.platforms.unix
>> were replaced with values from external_dependencies.platforms.macosx.
>> Did you do that on purpose? Because it breaks Debian/Ubuntu installs...
> 
> Ouch, sorry. I had to do that for testing here (that's actually the
> default for a vanilla build of apr-util on Linux). But I'll take the
> pragmatic road and leave the Ubuntu setup as the default for Linux.
> 
> I never wanted to do distro detection in LuaRocks but it seems like an
> inevitable direction...
> 
> As for lua-apr, I'll reupload the rock as you sent it (with an updated
> revision number to avoid further confusion), sorry about the mess up.

Thanks to your explanation I found a bug in the Lua/APR makefile which
finally resolves an issue that's been bothering me for quite a while:
The makefile dependency detection didn't work when the makefile was
executed by LuaRocks. This problem is now fixed [1].

Because the makefile has over time gained support for Debian, Ubuntu,
Arch Linux, Mac OS X and FreeBSD I don't see the point in wrestling
LuaRocks about a problem that's already been solved. Because of this
I've decided to remove the "external_dependencies" and "platforms"
sections from the rockspec and just have the makefile figure it out.

If you agree this is a reasonable approach, I've uploaded a new rockspec
[2] and ZIP archive [3]. Hopefully the new release will compile on your
machine without having to tweak the rockspec or makefile :-)

 - Peter

PS. If my "portable makefile experiment" succeeds then IMHO LuaRocks
doesn't need to support distro detection since there's a workaround.

[1] https://github.com/xolox/lua-apr/commit/68b7c5a
[2] http://peterodding.com/code/lua/apr/downloads/lua-apr-0.18-4.rockspec
[3] http://peterodding.com/code/lua/apr/downloads/lua-apr-0.18-4.zip

------------------------------------------------------------------------------
EditLive Enterprise is the world's most technically advanced content
authoring tool. Experience the power of Track Changes, Inline Image
Editing and ensure content is compliant with Accessibility Checking.
http://p.sf.net/sfu/ephox-dev2dev
_______________________________________________
Luarocks-developers mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/luarocks-developers

Reply via email to