From: "Ludovic Courtès" <ludo@gnu.org>
To: Ian Ressa <ian@eonn.xyz>
Cc: 53777@debbugs.gnu.org, Xinglu Chen <public@yoctocell.xyz>
Subject: [bug#53777] [PATCH] add python-posix-ipc
Date: Sun, 06 Mar 2022 22:37:25 +0100 [thread overview]
Message-ID: <871qzekbje.fsf_-_@gnu.org> (raw)
In-Reply-To: <87o83gw3gj.fsf@yoctocell.xyz> (Xinglu Chen's message of "Wed, 09 Feb 2022 14:54:04 +0100")
Hello Ian,
Did you have a chance to address Xinglu’s comments below?
Thanks in advance!
Ludo’.
Xinglu Chen <public@yoctocell.xyz> skribis:
> Hi,
>
> Ian schrieb am Freitag der 04. Februar 2022 um 04:53 GMT:
>
>> Adding python-posix-ipc to gnu/packages/python-xyz.scm
>>
>> ---
>> gnu/packages/python-xyz.scm | 32 ++++++++++++++++++++++++++++++++
>> 1 file changed, 32 insertions(+)
>>
>> diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
>> index 46419e8ace..53d5588564 100644
>> --- a/gnu/packages/python-xyz.scm
>> +++ b/gnu/packages/python-xyz.scm
>> @@ -116,6 +116,7 @@
>> ;;; Copyright © 2021 Greg Hogan <code@greghogan.com>
>> ;;; Copyright © 2022 John Kehayias <john.kehayias@protonmail.com>
>> ;;; Copyright © 2022 Aleksandr Vityazev <avityazev@posteo.org>
>> +;;; Copyright © 2022 Ian Ressa <ian@eonn.xyz>
>> ;;;
>> ;;; This file is part of GNU Guix.
>> ;;;
>> @@ -28926,3 +28927,34 @@ (define-public python-types-dataclasses
>> "This packages provides a collection of library stubs for Python, with
>> static types.")
>> (license license:asl2.0)))
>> +
>> +(define-public python-posix-ipc
>> + (package
>> + (name "python-posix-ipc")
>> + (version "1.0.5")
>> + (source (origin
>> + (method url-fetch)
>> + (uri (pypi-uri "posix_ipc" version))
>> + (sha256
>> + (base32 "15wbxf67y17g04i3f3vjw9zjbvlvqqk9q1x0ya1y7apl5k7b3pbc"))))
>> + (build-system python-build-system)
>> + (arguments
>> + `(#:modules ((guix build python-build-system)
>> + ((guix build gnu-build-system) #:prefix gnu:)
>> + (guix build utils))
>
> Nothing from (guix build gnu-build-system) seems to be used. I think
> you can drop the ‘#:modules’ arguments all together.
>
>> + #:phases
>> + (modify-phases %standard-phases
>> + (add-after 'unpack 'support-cross-compilation
>> + (lambda _
>> + (substitute* "prober.py"
>> + (("cc") ,(cc-for-target)))
>> + #t)))))
>
> Phases no longer have to return #f.
>
>
>> + (home-page "http://semanchuk.com/philip/posix_ipc/")
>
> The page supports HTTPS too, so it should be used instead of plain HTTP.
>
>> + (synopsis
>> + "POSIX IPC primitives (semaphores, shared memory and message
>> +queues) for Python")
>
> The things in parenthesis are also mentioned in the description, so I
> think you can drop them to make the synopsis a bit shorter.
>
>> + (description
>> + "The Python extension module posix_ipc gives Python access to
>> +POSIX inter-process semaphores, shared memory and message queues on systems
>> +that support the POSIX Realtime Extensions a.k.a. POSIX 1003.1b-1993.")
>> + (license license:bsd-3)))
>> --
>> 2.34.0
>
> The rest LGTM!
next prev parent reply other threads:[~2022-03-06 21:38 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-02-04 4:53 [bug#53777] [PATCH] add python-posix-ipc Ian Ressa
2022-02-09 13:54 ` Xinglu Chen
2022-03-06 21:37 ` Ludovic Courtès [this message]
2022-05-15 3:30 ` [bug#53777] [PATCH v2] gnu: Add python-posix-ipc Ian Ressa
2023-09-01 22:32 ` bug#53777: [PATCH] add python-posix-ipc Vagrant Cascadian
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
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=871qzekbje.fsf_-_@gnu.org \
--to=ludo@gnu.org \
--cc=53777@debbugs.gnu.org \
--cc=ian@eonn.xyz \
--cc=public@yoctocell.xyz \
/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 external index
https://git.savannah.gnu.org/cgit/guix.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.