On Fri, 5 Aug 2016 16:20:49 +0200 Julien Lepiller <jul...@lepiller.eu> wrote:
> On Fri, 05 Aug 2016 15:47:50 +0200 > Andy Wingo <wi...@igalia.com> wrote: > > > On Fri 05 Aug 2016 14:18, Julien Lepiller <jul...@lepiller.eu> > > writes: > > > here is a patch that adds a service definition for openssh. > > > > Very nice! > > > > > + (let ((pid (primitive-fork))) > > > + (case pid > > > + ((0) > > > + (execl (string-append #$openssh "/bin/ssh-keygen") > > > + "ssh-keygen" "-A") > > > + (else > > > + (zero? (cdr (waitpid pid))))))) > > > > I guess you could system* this one instead; would be easier I think. > > Dunno. > > I modified my patch, so you can choose whichever feels better. > > > > > Other than that looks all good to me. Would need an addition to the > > manual eventually though. > > > > > Andy > Hi, It's been a bit of time since I posted the patch, but didn't hear any news about it. What's wrong with it? Is it that I misunderstood the "Would need an addition to the manual eventually though"? Does that mean I need to add an entry to the manual myself (if so, could you tell me what file to edit?)