From: ng0 <ng0@we.make.ritual.n0.is>
To: Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH 3/4] gnu: Add psyclpc.
Date: Sun, 30 Oct 2016 12:46:55 +0000 [thread overview]
Message-ID: <87lgx657i8.fsf@we.make.ritual.n0.is> (raw)
In-Reply-To: <87d1ii9xvf.fsf@mdc-berlin.de> (Ricardo Wurmus's message of "Sun, 30 Oct 2016 07:03:48 +0100")
Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de> writes:
> ng0 <ngillmann@runbox.com> writes:
>
>> * gnu/packages/psyc.scm (psyclpc): New variable.
>> ---
>> gnu/packages/psyc.scm | 79 +++++++++++++++++++++++++++++++++++++++++++++++++++
>> 1 file changed, 79 insertions(+)
>>
>> diff --git a/gnu/packages/psyc.scm b/gnu/packages/psyc.scm
>
> As before, please add this to “messaging”.
Okay. I will move the already added module psyc.scm in master to
messaging then. It's not exclusive messaging, but I don't really
care where it ends up. psyc.scm was just a suggestion I made
(which can be read in the very first message for this patch set).
>> +
>> +;; This commit removes the historic bundled pcre, not released as a tarball so far.
>> +(define-public psyclpc
>> + (let* ((commit "8bd51f2a4847860ba8b82dc79348ab37d516011e")
>> + (revision "1"))
>
> Okay.
>
>> + (package
>> + (name "psyclpc")
>> + (version (string-append "20160821-" revision "." (string-take commit 7)))
>> + (source (origin
>> + (method git-fetch)
>> + (uri (git-reference
>> + (url "git://git.psyced.org/git/psyclpc")
>> + (commit commit)))
>> + (sha256
>> + (base32
>> + "10w4kx9ygcv1lcmd7j4knvjiy8dac1y3hjfv3lhp67jpv6w3iagz"))))
>> + (build-system gnu-build-system)
>> + (arguments
>> + `(#:tests? #f ; There are no tests/checks.
>> + #:configure-flags
>> + ;; If you have questions about this part, look at
>> + ;; "src/settings/psyced" and the ebuild.
>
> Please don’t do this. Guix is not a derivative of Gentoo. This should
> be explained right here.
>
>> + (list
>> + "--enable-use-tls=yes"
>> + "--enable-use-mccp" ; Mud Client Compression Protocol, leave this enabled.
>> + (string-append "--prefix="
>> + (assoc-ref %outputs "out"))
>> + ;; src/Makefile: Set MUD_LIB to the directory which contains
>> + ;; the mud data. defaults to MUD_LIB = @libdir@
>> + (string-append "--libdir="
>> + (assoc-ref %outputs "out")
>> + "/opt/psyced/world")
>> + (string-append "--bindir="
>> + (assoc-ref %outputs "out")
>> + "/opt/psyced/bin")
>> + ;; src/Makefile: Set ERQ_DIR to directory which contains the
>> + ;; stuff which ERQ can execute (hopefully) savely. Was formerly
>> + ;; defined in config.h. defaults to ERQ_DIR= @libexecdir@
>> + (string-append "--libexecdir="
>> + (assoc-ref %outputs "out")
>> + "/opt/psyced/run"))
>
> I don’t think it’s wise to override the target directories to have a
> prefix of “/opt/psyced”. We already have a prefix:
> “/gnu/store/<hash>-<name>-<version>/”. If this is needed at runtime for
> some reason it should be done by a service.
>
>> + #:phases
>> + (modify-phases %standard-phases
>> + (add-before 'configure 'chdir-to-src
>> + ;; We need to pass this as env variables
>> + ;; and manually change the directory.
>
> What is “this”?
>
>> + (lambda _
>> + (chdir "src")
>> + (setenv "CONFIG_SHELL" (which "sh"))
>> + (setenv "SHELL" (which "sh")))))
>
> CONFIG_SHELL and SHELL are usually set for autoconf. Why does this not
> work in this case?
>
> Please end the phase with #t.
>
>> + #:make-flags (list "install-all")))
>> + (inputs
>> + `(("zlib" ,zlib)
>> + ("openssl" ,openssl)
>> + ("pcre" ,pcre)))
>> + (native-inputs
>> + `(("pkg-config" ,pkg-config)
>> + ("bison" ,bison)
>> + ("gnu-gettext" ,gnu-gettext)
>> + ("help2man" ,help2man)
>> + ("autoconf" ,autoconf)
>> + ("automake" ,automake)))
>> + (home-page "http://lpc.psyc.eu/")
>> + (synopsis "psycLPC is a multi-user network server programming
>> language")
>
> s/psycLPC is a//
>
>> + (description
>> + "LPC is a bytecode language, invented to specifically implement
>> +multi user virtual environments on the internet. This technology is used for
>> +MUDs and also the psyced implementation of the Protocol for SYnchronous Conferencing (PSYC). psycLPC is a fork of LDMud with some new features and
>> +many bug fixes.")
>
> Please fix the line lengths.
As far as I know you picked an old thread and I fixed this in a
newer one. If I'm wrong about this, there'll be a new thread
soon enough.
>> + (license license:gpl2))))
>
> GPLv2 only?
Yes, but since 2010 there's also a different license, as this was
originally up to someone else to review, I'll add again: I have
to send in a new patch revisision. psyclpc also needs a fix for
reproducibility which I will pass on directly to one of us with
push access. After psyclpc is fixed at our end, I will send new
versions to guix.
>
> ~~ Ricardo
>
Thanks for review
next prev parent reply other threads:[~2016-10-30 12:47 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-09-29 10:38 [PATCH 1/4] gnu: Add perl-net-psyc ng0
2016-09-29 10:38 ` [PATCH 2/4] gnu: Add libpsyc ng0
2016-10-30 5:57 ` Ricardo Wurmus
2016-09-29 10:38 ` [PATCH 3/4] gnu: Add psyclpc ng0
2016-10-30 6:03 ` Ricardo Wurmus
2016-10-30 12:46 ` ng0 [this message]
2016-09-29 10:38 ` [PATCH 4/4] gnu: Add psyced ng0
2016-09-29 10:44 ` ng0
2016-10-30 5:42 ` Ricardo Wurmus
2016-10-30 5:48 ` Ricardo Wurmus
2016-10-02 1:46 ` [PATCH 1/4] gnu: Add perl-net-psyc Leo Famulari
2016-10-02 10:35 ` ng0
2016-10-04 2:24 ` Leo Famulari
2016-10-04 8:23 ` ng0
2016-10-04 8:34 ` psyc.scm: new patch series ng0
2016-10-04 8:34 ` [PATCH 1/4] gnu: Add perl-net-psyc ng0
2016-10-04 8:34 ` [PATCH 2/4] gnu: Add libpsyc ng0
2016-10-04 8:34 ` [PATCH 3/4] gnu: Add psyclpc ng0
2016-10-04 8:34 ` [PATCH 4/4] gnu: Add psyced ng0
2016-10-26 11:37 ` Efraim Flashner
2016-10-28 10:04 ` ng0
2016-10-30 5:54 ` [PATCH 1/4] gnu: Add perl-net-psyc Ricardo Wurmus
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87lgx657i8.fsf@we.make.ritual.n0.is \
--to=ng0@we.make.ritual.n0.is \
--cc=guix-devel@gnu.org \
--cc=ricardo.wurmus@mdc-berlin.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).