From mboxrd@z Thu Jan 1 00:00:00 1970 From: ng0 Subject: Re: [PATCH] gnu: perl-net-psyc Date: Thu, 16 Jun 2016 06:31:52 +0000 Message-ID: <20160616063152.GA4304@khazad-dum> References: <20160615163930.GA6612@khazad-dum> <20160615164148.GA6667@khazad-dum> <20160615165334.636551b4@openmailbox.org> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="LZvS9be/3tNcYl/X" Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:51511) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bDQqS-0000nu-8Y for guix-devel@gnu.org; Thu, 16 Jun 2016 02:32:07 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bDQqO-0006j7-F9 for guix-devel@gnu.org; Thu, 16 Jun 2016 02:32:04 -0400 Received: from 93-95-228-168.1984.is ([93.95.228.168]:45973 helo=beleriand.n0.is) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bDQqO-0006ik-2C for guix-devel@gnu.org; Thu, 16 Jun 2016 02:32:00 -0400 Received: by beleriand.n0.is (OpenSMTPD) with ESMTPSA id 34693c92 TLS version=TLSv1/SSLv3 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NO for ; Thu, 16 Jun 2016 06:31:55 +0000 (UTC) Content-Disposition: inline In-Reply-To: <20160615165334.636551b4@openmailbox.org> List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: "Guix-devel" To: guix-devel@gnu.org --LZvS9be/3tNcYl/X Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On 2016-06-15(04:53:34-0500), Eric Bavier wrote: > On Wed, 15 Jun 2016 16:41:48 +0000 > ng0 wrote: > > > Optional dependencies incoming in the future, Perl Curses being the > > first one. > > They are optional and need to explanation as the binaries themselves > > complaint (and give you motivation to package them ;) ) > > > > > > On 2016-06-15(04:39:30PM+0000), ng0 wrote: > > > From 10eeb85733c67c4fdd5cd4df84457a565e45971e Mon Sep 17 00:00:00 2001 > > > From: ng0 > > > Date: Wed, 15 Jun 2016 16:30:30 +0000 > > > Subject: [PATCH] gnu: Add perl-net-psyc. > > > > > > * gnu/packages/perl.scm (perl-net-psyc): New Variable. > > > --- > > > gnu/packages/perl.scm | 71 +++++++++++++++++++++++++++++++++++++++++= ++++++++++ > > > 1 file changed, 71 insertions(+) > > > > > > diff --git a/gnu/packages/perl.scm b/gnu/packages/perl.scm > > > index e7e7967..8772d0a 100644 > > > --- a/gnu/packages/perl.scm > > > +++ b/gnu/packages/perl.scm > > > @@ -7,6 +7,7 @@ > > > ;;; Copyright =C2=A9 2016 Mark H Weaver > > > ;;; Copyright =C2=A9 2016 Jochem Raat > > > ;;; Copyright =C2=A9 2016 Efraim Flashner > > > +;;; Coypright =C2=A9 2016 ng0 > > > ;;; > > > ;;; This file is part of GNU Guix. > > > ;;; > > > @@ -28,6 +29,7 @@ > > > #:use-module (gnu packages) > > > #:use-module (guix packages) > > > #:use-module (guix download) > > > + #:use-module (guix git-download) > > > #:use-module (guix build-system gnu) > > > #:use-module (guix build-system perl)) > > > > > > @@ -3865,6 +3867,75 @@ cycle. Functions called in the package itself= will still be bound by their > > > name, but they won't show up as methods on your class or instances.") > > > (license (package-license perl)))) > > > > > > +;; highly optional dependencies: rxaudio (mp3 binary from 1996) > > > +;; opt. perl: MP3::List (psycmp3), Curses (psycion), pwd.pl > > > +;; (psyccmd), SGI::FAM module (psycfilemonitor); for full > > > +;; functionality package all except MP3::List. > > > +(define-public perl-net-psyc > > Is it not possible to use the zip at > http://perlpsyc.psyc.eu/perlpsyc-1.0.zip ? If there are bug fixes in > the git repo that we cannot do without, could you give a brief summary > in a comment? Adding to the previous email, basically commits happen often (today one of the contribs was updated) and release snapshot happen very rarely, as 1.0 was the first in very long time (years). > > > + (let ((commit "4176f5236e73e99391c0877152d5596ab0a9b154")) > > > + (package > > > + (name "perl-net-psyc") > > > + (version (string-append "1.0" "-" > > > + (string-take commit 8))) > > > + (source (origin > > > + (method git-fetch) > > > + (uri (git-reference > > > + (url "git://git.psyced.org/git/perlpsyc") > > > + (commit commit))) > > > + (file-name (string-append name "-" version)) > > > + (sha256 > > > + (base32 > > > + "19msvr1kriqa6n8cnk8z96xd5i0v5cy2y52jsj08vaj79b4m7= sph")))) > > > + (build-system perl-build-system) > > > + (arguments > > > + `(#:phases > > > + (modify-phases %standard-phases > > > + (delete 'configure) > > > + (delete 'build) > > > + (replace 'install > > Could you add a comment saying that the source distribution's > INSTALL.txt says to roll-your-own installation? > > > > + (lambda* (#:key outputs #:allow-other-keys) > > > + (let* ((out (assoc-ref outputs "out")) > > > + (doc (string-append out "/share/doc/perl-net-p= syc")) > > > + (libpsyc (string-append out "/lib/psyc/ion")) > > > + (libperl (string-append out "/lib/perl5/site_p= erl/" > > > + ,(package-version perl= ))) > > > + (bin (string-append out "/bin"))) > > > + (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 "/co= ntrib")) > > I doesn't seem clear to me that the contents of "contrib" are covered > under the same license as the rest of the package. > > > > + (copy-recursively "hooks" (string-append doc "/hook= s")) > > > + (copy-recursively "sdj" (string-append doc "/sdj")) > > > + (install-file "README.txt" doc) > > > + (install-file "TODO.txt" doc) > > > + #t))) > > > + (add-after 'install 'wrap-programs > > > + (lambda* (#:key outputs #:allow-other-keys) > > > + ;; Make sure all executables in "bin" find the Perl m= odules > > > + ;; provided by this package at runtime. > > > + (let* ((out (assoc-ref outputs "out")) > > > + (bin (string-append out "/bin/")) > > > + (path (string-append out "/lib/perl5/site_perl= "))) > > > + (for-each (lambda (file) > > > + (wrap-program file > > > + `("PERL5LIB" ":" prefix (,path)))) > > > + (find-files bin "\\.*$")) > > > + ;; XXX: hooks/dpa2psyc,examples/{recvtest,sendtest}, > > > + ;; cgi/psycpager,contrib/{makenoise,keefchat > > > + ;; contrib/MovableTypeBlog/plugins/psyc.pl > > Should this XXX be removed? > > > > + #t)))))) > > > + (description > > > + "Net::PSYC with support for TCP, UDP, Event.pm, IO::Select and > > > +Gtk2 event loops. This package includes 12 applications and > > Perhaps "@code{Net::PSYC} is a Perl implementation of the PSYC protocol > with support for ..."? > > > > +additional scripts: psycion, a @uref{http://about.psyc.eu, PSYC} cha= t client, > > > +remotor, a control console for @uref{https://torproject.org, tor} ro= uter, and many more. > > > +Further information is included in the package source and on the web= site.") > > > + (synopsis "Perl implementation of PSYC protocol") > > > + (home-page "http://perlpsyc.psyc.eu/") > > > + ;;dual licensed: gpl2+ Artistic > > > + (license (list gpl2+ (package-license perl)))))) > > LICENSE.txt says "GPL-2", and I cannot see an "or later" in the source > files themselves. In fact, the few files I see that mention a license > have just the "under the same terms as perl itself" verbiage. So > maybe leave out the "gpl2+"? > > Could you send an updated patch? > > Thanks, > `~Eric -- =E2=99=A5=E2=92=B6 ng0 For non-prism friendly talk find me on psyced.org / loupsycedyglgamf.onion --LZvS9be/3tNcYl/X Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- iF0EARYKAAYFAldiR9cACgkQhhoAchyzrCD/5AEA1PdIWo9T4VMGOKhBYcoVr2Vb KTB+nCgnEgI16QBZQG0A+Osg3J2/UD61DBvnsJRp9rWgpRMOpk3QoX5uQn+WAQo= =ljdx -----END PGP SIGNATURE----- --LZvS9be/3tNcYl/X--