Re: [PATCH] gnu: Add ircii.

2016-08-13 Thread ng0
Leo Famulari writes: > On Tue, Aug 09, 2016 at 09:38:55PM +0200, Ricardo Wurmus wrote: > > I actually applied a tweaked version of this patch with c744f5c0ad. > > But, the improvements you suggest should be applied. > >> > + (lambda* (#:key inputs #:allow-other-keys) >> > +

Re: [PATCH] gnu: Add ircii.

2016-08-09 Thread Leo Famulari
On Tue, Aug 09, 2016 at 09:38:55PM +0200, Ricardo Wurmus wrote: I actually applied a tweaked version of this patch with c744f5c0ad. But, the improvements you suggest should be applied. > > + (lambda* (#:key inputs #:allow-other-keys) > > + (substitute* "bsdinstall" > > +

Re: [PATCH] gnu: Add ircii.

2016-08-09 Thread Ricardo Wurmus
ng0 writes: > From 1438e5654cfd5f4eb4141a42580bd73c716e7134 Mon Sep 17 00:00:00 2001 > From: ng0 > Date: Mon, 1 Aug 2016 19:31:23 + > Subject: [PATCH] gnu: Add ircii. > * gnu/packages/irc.scm (ircii): New variable. > --- > gnu/packages/irc.scm | 46

Re: [PATCH] gnu: Add ircii.

2016-08-05 Thread Leo Famulari
On Fri, Aug 05, 2016 at 06:27:17PM +, ng0 wrote: > Leo Famulari writes: > > So the package is ready to merge? > > From my perspective yes. We have to add a smaller socks server/library > than ghc-socks, when that is done I can add --with-socks. This patch > clarifies this in a comment. Great

Re: [PATCH] gnu: Add ircii.

2016-08-05 Thread ng0
Leo Famulari writes: > On Fri, Aug 05, 2016 at 04:11:38PM +, ng0 wrote: >> Leo Famulari writes: >> >> > On Fri, Aug 05, 2016 at 03:48:53PM +, ng0 wrote: >> >> It looks like this is not complete yet: >> >> >> >> ng0@shadowwalker ~$ strace irc >> > >> > What should we be looking for in t

Re: [PATCH] gnu: Add ircii.

2016-08-05 Thread Leo Famulari
On Fri, Aug 05, 2016 at 04:11:38PM +, ng0 wrote: > Leo Famulari writes: > > > On Fri, Aug 05, 2016 at 03:48:53PM +, ng0 wrote: > >> It looks like this is not complete yet: > >> > >> ng0@shadowwalker ~$ strace irc > > > > What should we be looking for in this trace? > > Apparently nothin

Re: [PATCH] gnu: Add ircii.

2016-08-05 Thread ng0
Leo Famulari writes: > On Fri, Aug 05, 2016 at 03:48:53PM +, ng0 wrote: >> It looks like this is not complete yet: >> >> ng0@shadowwalker ~$ strace irc > > What should we be looking for in this trace? Apparently nothing because I did not rtfm enough. ... export IRCSERVER="SSLIRC/chat.freen

Re: [PATCH] gnu: Add ircii.

2016-08-05 Thread Leo Famulari
On Fri, Aug 05, 2016 at 03:48:53PM +, ng0 wrote: > It looks like this is not complete yet: > > ng0@shadowwalker ~$ strace irc What should we be looking for in this trace?

Re: [PATCH] gnu: Add ircii.

2016-08-05 Thread ng0
ng0 writes: > ng0 writes: > >> Indeed, not needed. Fixed: > > Well.. now it is fixed. Last patch was wrong. Blame multi-multitasking. > > From 9b383fff3ea117f5de5a096fdef1dc22cca4dc5c Mon Sep 17 00:00:00 2001 > From: ng0 > Date: Mon, 1 Aug 2016 19:31:23 + > Subject: [PATCH] gnu: Add ircii.

Re: [PATCH] gnu: Add ircii.

2016-08-05 Thread ng0
ng0 writes: > Indeed, not needed. Fixed: Well.. now it is fixed. Last patch was wrong. Blame multi-multitasking. >From 9b383fff3ea117f5de5a096fdef1dc22cca4dc5c Mon Sep 17 00:00:00 2001 From: ng0 Date: Mon, 1 Aug 2016 19:31:23 + Subject: [PATCH] gnu: Add ircii. * gnu/packages/irc.scm (irci

Re: [PATCH] gnu: Add ircii.

2016-08-05 Thread ng0
Indeed, not needed. Fixed: >From 1438e5654cfd5f4eb4141a42580bd73c716e7134 Mon Sep 17 00:00:00 2001 From: ng0 Date: Mon, 1 Aug 2016 19:31:23 + Subject: [PATCH] gnu: Add ircii. * gnu/packages/irc.scm (ircii): New variable. --- gnu/packages/irc.scm | 46

Re: [PATCH] gnu: Add ircii.

2016-08-05 Thread ng0
ng0 writes: > Ricardo Wurmus writes: > >> Why is this needed? Did you try using #:configure-flags instead of >> replacing the configure phase? You can refer to inputs with >> “%build-inputs” there. >> >> ~~ Ricardo >> > > Thanks, README made me try with ./configure --switches first. > Fixed in

Re: [PATCH] gnu: Add ircii.

2016-08-05 Thread ng0
Ricardo Wurmus writes: > Why is this needed? Did you try using #:configure-flags instead of > replacing the configure phase? You can refer to inputs with > “%build-inputs” there. > > ~~ Ricardo > Thanks, README made me try with ./configure --switches first. Fixed in this version. I'm still tes

Re: [PATCH] gnu: Add ircii.

2016-08-05 Thread Ricardo Wurmus
ng0 writes: > Ricardo Wurmus writes: > >> ng0 writes: >> >>> Thanks, in the new revision of the patch both issues you two found are >>> fixed. >> >>> + (replace 'configure >>> + (lambda* (#:key inputs outputs #:allow-other-keys) >>> + (let ((out (assoc-ref outputs

Re: [PATCH] gnu: Add ircii.

2016-08-05 Thread ng0
Ricardo Wurmus writes: > ng0 writes: > >> Thanks, in the new revision of the patch both issues you two found are >> fixed. > >> + (replace 'configure >> + (lambda* (#:key inputs outputs #:allow-other-keys) >> + (let ((out (assoc-ref outputs "out")) >> +

Re: [PATCH] gnu: Add ircii.

2016-08-04 Thread Ricardo Wurmus
ng0 writes: > Thanks, in the new revision of the patch both issues you two found are > fixed. > + (replace 'configure > + (lambda* (#:key inputs outputs #:allow-other-keys) > + (let ((out (assoc-ref outputs "out")) > + (ncurses (assoc-ref inputs "

Re: [PATCH] gnu: Add ircii.

2016-08-04 Thread ng0
Hi, ng0 writes: > 宋文武 writes: > >> Leo Famulari writes: >> >>> On Mon, Aug 01, 2016 at 07:34:49PM +, ng0 wrote: * gnu/packages/irc.scm (ircii): New variable. >>> >>> Cool! >>> + (snippet + '(begin + (substitute* "bsdinstall"

Re: [PATCH] gnu: Add ircii.

2016-08-02 Thread ng0
宋文武 writes: > Leo Famulari writes: > >> On Mon, Aug 01, 2016 at 07:34:49PM +, ng0 wrote: >>> * gnu/packages/irc.scm (ircii): New variable. >> >> Cool! >> >>> + (snippet >>> + '(begin >>> + (substitute* "bsdinstall" >>> + (("/bin/stri

Re: [PATCH] gnu: Add ircii.

2016-08-02 Thread 宋文武
Leo Famulari writes: > On Mon, Aug 01, 2016 at 07:34:49PM +, ng0 wrote: >> * gnu/packages/irc.scm (ircii): New variable. > > Cool! > >> + (snippet >> + '(begin >> + (substitute* "bsdinstall" >> + (("/bin/strip") "strip") >> +

Re: [PATCH] gnu: Add ircii.

2016-08-02 Thread ng0
Hi, Andreas Enge writes: > On Mon, Aug 01, 2016 at 08:54:13PM -0400, Leo Famulari wrote: >> > + (snippet >> > + '(begin >> > + (substitute* "bsdinstall" >> > + (("/bin/strip") "strip") >> > + (("/bin/cp") "cp") >> > +

Re: [PATCH] gnu: Add ircii.

2016-08-02 Thread Andreas Enge
On Mon, Aug 01, 2016 at 08:54:13PM -0400, Leo Famulari wrote: > > + (snippet > > + '(begin > > + (substitute* "bsdinstall" > > + (("/bin/strip") "strip") > > + (("/bin/cp") "cp") > > + (("/bin/chmod") "ch

Re: [PATCH] gnu: Add ircii.

2016-08-01 Thread Leo Famulari
On Mon, Aug 01, 2016 at 07:34:49PM +, ng0 wrote: > * gnu/packages/irc.scm (ircii): New variable. Cool! > + (snippet > + '(begin > + (substitute* "bsdinstall" > + (("/bin/strip") "strip") > + (("/bin/cp") "cp") > +