Re: NEW: lua-cliargs

2016-10-12 Thread Frederic Cambus
On Wed, Oct 12, 2016 at 03:47:35PM +0200, Jeremie Courreges-Anglas wrote:

> > A new tarball with the nitpickings fixed
> 
> ok jca@ to import
> 
> I plan to add explicit lua FLAVORS later.

Imported, thanks!



Re: NEW: lua-cliargs

2016-10-12 Thread Jeremie Courreges-Anglas
Florian Stinglmayr  writes:

> On Tue, Oct 11, 2016 at 10:32:12PM +0200, Frederic Cambus wrote:
>>
>> Works fine here, this is definitely handy. Some nitpicking:
>>
>> -COMMENT =  command line parser for lua
>> +COMMENT =  command line parser for Lua
>>
>> -#MIT
>> +# MIT
>>
>> Also, portcheck warns about 2 line(s) longer than 80 chars in Makefile.
>> Could you wrap the lines in the 'do-install' target?
>>
>
> A new tarball with the nitpickings fixed

ok jca@ to import

I plan to add explicit lua FLAVORS later.

-- 
jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF  DDCC 0DFA 74AE 1524 E7EE



Re: NEW: lua-cliargs

2016-10-11 Thread Florian Stinglmayr
On Tue, Oct 11, 2016 at 10:32:12PM +0200, Frederic Cambus wrote:
>
> Works fine here, this is definitely handy. Some nitpicking:
>
> -COMMENT =  command line parser for lua
> +COMMENT =  command line parser for Lua
>
> -#MIT
> +# MIT
>
> Also, portcheck warns about 2 line(s) longer than 80 chars in Makefile.
> Could you wrap the lines in the 'do-install' target?
>

A new tarball with the nitpickings fixed

Regards,
Florian


lua-cliargs.tar.gz
Description: application/tar-gz


Re: NEW: lua-cliargs

2016-10-11 Thread Frederic Cambus
On Sun, Jun 05, 2016 at 10:15:37AM +0200, Florian Stinglmayr wrote:

> attached is a port for lua-cliargs. A command line parsing library for
> Lua. It works with all versions, and is really handy. It is also a
> dependency for some bigger ports I have line up.

Works fine here, this is definitely handy. Some nitpicking:

-COMMENT =  command line parser for lua
+COMMENT =  command line parser for Lua

-#MIT
+# MIT

Also, portcheck warns about 2 line(s) longer than 80 chars in Makefile.
Could you wrap the lines in the 'do-install' target?