Update: slim-1.3.6

2013-10-30 Thread David Coppa
Hi! Here's an update to slim-1.3.6. I've a positive report about the improved rc.d script. Ciao, David Index: Makefile === RCS file: /cvs/ports/x11/slim/Makefile,v retrieving revision 1.24 diff -u -p -u -p -r1.24 Makefile ---

Re: Update: slim-1.3.6

2013-10-30 Thread Robert Peichaer
On Wed, Oct 30, 2013 at 10:17:24AM +0100, David Coppa wrote: Hi! Here's an update to slim-1.3.6. I've a positive report about the improved rc.d script. Ciao, David works for me on amd64 ok rpe@ (comments about slim.rc shell style in my previous mail) Index: Makefile

Re: Update: slim-1.3.6

2013-10-30 Thread Stuart Henderson
On 2013/10/30 11:50, Robert Peichaer wrote: (comments about slim.rc shell style in my previous mail) I didn't see that (off-list?) but there's no need to call out to /bin/expr, we can use i=$((i+1)).

Re: Update: slim-1.3.6

2013-10-30 Thread David Coppa
On Wed, 30 Oct 2013, Stuart Henderson wrote: On 2013/10/30 11:50, Robert Peichaer wrote: (comments about slim.rc shell style in my previous mail) I didn't see that (off-list?) but there's no need to call out to /bin/expr, we can use i=$((i+1)). New diff, after feedback from rpe@ Index:

Re: Update: slim-1.3.6

2013-10-30 Thread Stuart Henderson
On 2013/10/30 12:24, David Coppa wrote: On Wed, 30 Oct 2013, Stuart Henderson wrote: On 2013/10/30 11:50, Robert Peichaer wrote: (comments about slim.rc shell style in my previous mail) I didn't see that (off-list?) but there's no need to call out to /bin/expr, we can use

Re: Update: slim-1.3.6

2013-10-30 Thread Alexander Hall
Bikeshedding on... Actually not sure if it's a ksh'ism or not, but I tend to use let i++ Either way, the backtick variant looked a bit too much. /Alexander Stuart Henderson st...@openbsd.org wrote: On 2013/10/30 11:50, Robert Peichaer wrote: (comments about slim.rc shell style in my

Re: Update: slim-1.3.6

2013-10-30 Thread Landry Breuil
On Wed, Oct 30, 2013 at 12:06:20PM +, Stuart Henderson wrote: On 2013/10/30 12:24, David Coppa wrote: On Wed, 30 Oct 2013, Stuart Henderson wrote: On 2013/10/30 11:50, Robert Peichaer wrote: (comments about slim.rc shell style in my previous mail) I didn't see that