On Sat, Oct 29, 2011 at 3:03 PM, Peter Odding <[email protected]> wrote:
> Hi Hisham and Fabio,
>
> Would you please add the latest Lua/APR rockspec to the repository? It's
> available at the following location:
>
> http://peterodding.com/code/lua/apr/downloads/lua-apr-0.20.3-1.rockspec
>
> One notable change since the last rockspec that was included in the
> repository is that the rockspec no longer references any external
> dependencies, instead the makefile uses pkg-config to automatically
> detect external dependencies.
>
> The main rule in the makefile now also runs a Lua script that knows how
> to check for and install missing but required system packages on Debian,
> Ubuntu, Arch Linux, Red Hat, Fedora Core, Suse Linux and CentOS.

I don't think distro maintainers would be too happy with this. Suppose
I wanted to build a Fedora package of your project. It shouldn't
install other Fedora packages by itself. Instead, the dependencies
should be in your package's .spec file.

I understand your effort to make everything automatic with a single
command, but I don't think users will expect running a makefile
through luarocks to go and install, say, systemwide Debian packages on
their system.

If you want to provide the automagic functionality for users that
desire it, it's your call, but please provide a make target that skips
those steps and constrains itself to installing your module, so we can
use it in LuaRocks. Installing a rock must not cause the installation
of anything outside the rock tree.

-- Hisham

------------------------------------------------------------------------------
Get your Android app more play: Bring it to the BlackBerry PlayBook 
in minutes. BlackBerry App World&#153; now supports Android&#153; Apps 
for the BlackBerry&reg; PlayBook&#153;. Discover just how easy and simple 
it is! http://p.sf.net/sfu/android-dev2dev
_______________________________________________
Luarocks-developers mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/luarocks-developers

Reply via email to