diff mbox

Why was "gnu: wrap-python3: Create more symlinks." reverted?

Message ID cf7108a4-654a-9acc-b3fe-adbfb75c0dd3@goebel-consult.de
State New
Headers show

Commit Message

Hartmut Goebel Sept. 4, 2016, 6:35 p.m. UTC
Hello Leo,

why did you revert this patch? Having these aliases helps a lot when
packaging python (extension) modules. Please reapply!

commit 187e9094f7fa79e52820b154bbac855f357553ce
Author: Leo Famulari <leo@famulari.name>
Date:   Fri Jul 1 17:03:29 2016 -0400

    Revert "gnu: wrap-python3: Create more symlinks."
   
    This reverts commit 478e75b53bdf754cdf911226d37e517387f73c09.

Python@tie{}3.x such

Comments

Leo Famulari Sept. 4, 2016, 7:31 p.m. UTC | #1
On Sun, Sep 04, 2016 at 08:35:11PM +0200, Hartmut Goebel wrote:
> Hello Leo,
> 
> why did you revert this patch? Having these aliases helps a lot when
> packaging python (extension) modules. Please reapply!

Hi,

I reverted the commit in response to this request from Andreas:
http://lists.gnu.org/archive/html/guix-devel/2016-07/msg00011.html

I am really sorry that I did not add you to the email thread when I
replied to Andreas confirming the reversion. I should have done that but
it must have slipped my mind.

I'm not sure if the problems he described would still exist. Do you want
to look into it?
Hartmut Goebel Sept. 5, 2016, 9:30 a.m. UTC | #2
Hi Leo,
> I reverted the commit in response to this request from Andreas:
> http://lists.gnu.org/archive/html/guix-devel/2016-07/msg00011.html

IC. (I would not even have recognised if you'd CC me, since neither the
subject nor the content strikes me.)

So we have a dilemma now: not adding `python-config` (at al.) to the
wrapper will make packaging more complicated for those Python packages
not caring about python resp. python3 in their Makeifles (or what ever
they use). OTOH, as these problems with cflags show, this may not work
anyway.

I *may* have a look into it, but this will not happen any-time soon.
diff mbox

Patch

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index b92746a..8bba7ca 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -370,8 +370,8 @@  data types.")
                   (lambda (old new)
                     (symlink (string-append python old)
                              (string-append bin "/" new)))
-                  '("python3" "pydoc3" "idle3" "pip3" "python3-config")
-                  '("python"  "pydoc"  "idle" "pip" "python-config"))))))
+                  '("python3" "pydoc3" "idle3")
+                  '("python"  "pydoc"  "idle"))))))
     (synopsis "Wrapper for the Python 3 commands")
     (description
      "This package provides wrappers for the commands of