Re: [PATCH 1/2] gnu: Add perl-net-psyc.

[ng0]

Leo Famulari <address@hidden> writes:

> On Tue, Sep 13, 2016 at 11:32:36AM +0000, ng0 wrote:
>> * gnu/packages/psyc.scm: New file.
>> * gnu/local.mk (GNU_SYSTEM_MODULES): Add it.
>
> This package looks pretty good to me!
>
>> +         (replace 'install
>> +           (lambda* (#:key outputs #:allow-other-keys)
>> +             (let* ((out (assoc-ref outputs "out"))
>> +                    (doc (string-append out "/share/doc/perl-net-psyc"))
>> +                    (man (string-append out "/share/man"))
>> +                    (bin (string-append out "/bin"))
>> +                    (libpsyc (string-append out "/lib/psyc/ion"))
>> +                    (libperl (string-append out "/lib/perl5/site_perl/"
>> +                                            ,(package-version perl))))
>> +
>> +               (copy-recursively "lib/perl5" libperl)
>> +               (copy-recursively "lib/psycion" libpsyc)
>> +               (copy-recursively "bin" bin)
>> +               (install-file "cgi/psycpager" (string-append doc "/cgi"))
>> +               (copy-recursively "contrib" (string-append doc "/contrib"))
>> +               (copy-recursively "hooks" (string-append doc "/hooks"))
>> +               (copy-recursively "sdj" (string-append doc "/sdj"))
>> +               (install-file "README.txt" doc)
>> +               (install-file "TODO.txt" doc)
>> +               (copy-recursively "share/man/man1" (string-append man 
>> "/man1"))
>> +               (copy-recursively "share/man/man3" (string-append man 
>> "/man3"))
>> +               #t)))
>
> But, there aren't any man pages in the built package.

Oh? Then the change I applied before submitting it broke it. There are
manpages, I just thought I could save a line by using
(let (man (string-append out "/share/man"))
  (copy-recursively "share/man/man1" (string-append man "/man1"))
  (copy-recursively "share/man/man3" (string-append man "/man3"))

and not 

(let (man1 (string-append out "/share/man/man1"))
     (man3 (string-append out "/share/man/man3"))
  (copy-recursively "share/man/man1" man1))
  (copy-recursively "share/man/man3" man3))

>> +    (description
>> +     "@code{Net::PSYC} with support for TCP, UDP, Event.pm, 
>> @code{IO::Select} and
>> +Gtk2 event loops.  This package includes 12 applications and additional 
>> scripts:
>> +psycion, a @uref{http://about.psyc.eu,PSYC} chat client, remotor, a control 
>> console
>> +for @uref{https://torproject.org,tor} router, and many more.")
>
> Also, I don't know this word "remotor". Is it related to PSYC?

Then I need to restructur my sentence. If you take a look at the website
you will see that it is one of the many applications perlpsyc has.

-- 
              ng0