all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* core-updates!
@ 2016-07-01  9:02 Ludovic Courtès
  2016-07-01 11:33 ` python-pycairo in core-updates (Re: core-updates!) 宋文武
  2016-07-01 20:59 ` core-updates! Leo Famulari
  0 siblings, 2 replies; 9+ messages in thread
From: Ludovic Courtès @ 2016-07-01  9:02 UTC (permalink / raw)
  To: guix-devel

Hello!

Here’s a kind invitation for you to look at ‘core-updates’ and make sure
your favorite packages build there.  :-)

  https://hydra.gnu.org/jobset/gnu/core-updates

It seems that the bigger issues have been fixed, so now it will be
mostly leaf packages and/or mips64el/arm needing love.

Ludo’.

^ permalink raw reply	[flat|nested] 9+ messages in thread

* python-pycairo in core-updates (Re: core-updates!)
  2016-07-01  9:02 core-updates! Ludovic Courtès
@ 2016-07-01 11:33 ` 宋文武
  2016-07-01 15:33   ` Andreas Enge
  2016-07-02 13:12   ` Ludovic Courtès
  2016-07-01 20:59 ` core-updates! Leo Famulari
  1 sibling, 2 replies; 9+ messages in thread
From: 宋文武 @ 2016-07-01 11:33 UTC (permalink / raw)
  To: Ludovic Courtès, Leo Famulari; +Cc: guix-devel

Hi, python-wrapper has a broken `python3-config', which
output cflags to the wrapper instead of the origin python3,
leading the build failure of python-pycairo.


It was introduced in commit 478e75b53.  Does it really needed?

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: python-pycairo in core-updates (Re: core-updates!)
  2016-07-01 11:33 ` python-pycairo in core-updates (Re: core-updates!) 宋文武
@ 2016-07-01 15:33   ` Andreas Enge
  2016-07-01 21:06     ` Leo Famulari
  2016-07-02 13:12   ` Ludovic Courtès
  1 sibling, 1 reply; 9+ messages in thread
From: Andreas Enge @ 2016-07-01 15:33 UTC (permalink / raw)
  To: 宋文武; +Cc: guix-devel

On Fri, Jul 01, 2016 at 07:33:52PM +0800, 宋文武 wrote:
> Hi, python-wrapper has a broken `python3-config', which
> output cflags to the wrapper instead of the origin python3,
> leading the build failure of python-pycairo.
> 
> It was introduced in commit 478e75b53.  Does it really needed?

Unless someone suggests a different solution very soon, I would suggest
to revert the commit.

Andreas

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: core-updates!
  2016-07-01  9:02 core-updates! Ludovic Courtès
  2016-07-01 11:33 ` python-pycairo in core-updates (Re: core-updates!) 宋文武
@ 2016-07-01 20:59 ` Leo Famulari
  1 sibling, 0 replies; 9+ messages in thread
From: Leo Famulari @ 2016-07-01 20:59 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: guix-devel

On Fri, Jul 01, 2016 at 11:02:13AM +0200, Ludovic Courtès wrote:
> Here’s a kind invitation for you to look at ‘core-updates’ and make sure
> your favorite packages build there.  :-)
> 
>   https://hydra.gnu.org/jobset/gnu/core-updates
> 
> It seems that the bigger issues have been fixed, so now it will be
> mostly leaf packages and/or mips64el/arm needing love.

Mozjs is failing, which breaks GNOME:
<http://hydra.gnu.org/eval/109000?filter=mozjs&compare=108999&full=>
<http://hydra.gnu.org/eval/109000?filter=gnome&compare=108999&full=>

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: python-pycairo in core-updates (Re: core-updates!)
  2016-07-01 15:33   ` Andreas Enge
@ 2016-07-01 21:06     ` Leo Famulari
  0 siblings, 0 replies; 9+ messages in thread
From: Leo Famulari @ 2016-07-01 21:06 UTC (permalink / raw)
  To: Andreas Enge; +Cc: guix-devel

On Fri, Jul 01, 2016 at 05:33:33PM +0200, Andreas Enge wrote:
> On Fri, Jul 01, 2016 at 07:33:52PM +0800, 宋文武 wrote:
> > Hi, python-wrapper has a broken `python3-config', which
> > output cflags to the wrapper instead of the origin python3,
> > leading the build failure of python-pycairo.
> > 
> > It was introduced in commit 478e75b53.  Does it really needed?
> 
> Unless someone suggests a different solution very soon, I would suggest
> to revert the commit.

Reverted with 187e9094f7

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: python-pycairo in core-updates (Re: core-updates!)
  2016-07-01 11:33 ` python-pycairo in core-updates (Re: core-updates!) 宋文武
  2016-07-01 15:33   ` Andreas Enge
@ 2016-07-02 13:12   ` Ludovic Courtès
  2016-10-16  3:21     ` [PATCH] gnu: wrap-python3: Use a bash wrapper for 'python3-config'. (was: python-pycairo in core-updates) 宋文武
  1 sibling, 1 reply; 9+ messages in thread
From: Ludovic Courtès @ 2016-07-02 13:12 UTC (permalink / raw)
  To: 宋文武; +Cc: guix-devel

Hi,

iyzsong@member.fsf.org (宋文武) skribis:

> Hi, python-wrapper has a broken `python3-config', which
> output cflags to the wrapper instead of the origin python3,
> leading the build failure of python-pycairo.

So it appears to resolve the CFLAGS it returns relative to its own
argv[0] or /proc/self/exe, right?

Eventually we should fix this and reinstate this commit, but reverting
it was the right thing in the short term.

Thank you!

Ludo’.

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [PATCH] gnu: wrap-python3: Use a bash wrapper for 'python3-config'. (was: python-pycairo in core-updates)
  2016-07-02 13:12   ` Ludovic Courtès
@ 2016-10-16  3:21     ` 宋文武
  2016-10-16 18:14       ` Leo Famulari
  0 siblings, 1 reply; 9+ messages in thread
From: 宋文武 @ 2016-10-16  3:21 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: guix-devel

[-- Attachment #1: Type: text/plain, Size: 626 bytes --]


The revert isn't in 'core-updates', python-pycairo is broken again for
the same reason.

ludo@gnu.org (Ludovic Courtès) writes:

> Hi,
>
> iyzsong@member.fsf.org (宋文武) skribis:
>
>> Hi, python-wrapper has a broken `python3-config', which
>> output cflags to the wrapper instead of the origin python3,
>> leading the build failure of python-pycairo.
>
> So it appears to resolve the CFLAGS it returns relative to its own
> argv[0] or /proc/self/exe, right?
>
> Eventually we should fix this and reinstate this commit, but reverting
> it was the right thing in the short term.
>

Here is a fix: 


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-wrap-python3-Use-a-bash-wrapper-for-python3-conf.patch --]
[-- Type: text/x-patch, Size: 2558 bytes --]

From b7badcf695fcb245224ffdf51cdb016b45f9a2a5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E5=AE=8B=E6=96=87=E6=AD=A6?= <iyzsong@gmail.com>
Date: Sun, 16 Oct 2016 10:44:29 +0800
Subject: [PATCH] gnu: wrap-python3:  Use a bash wrapper for 'python3-config'.

See <http://lists.gnu.org/archive/html/guix-devel/2016-07/msg00008.html>
for details.

* gnu/packages/python.scm (wrap-python3)[arguments]: Use a bash wrapper
instead of a symlink for python3-config.
---
 gnu/packages/python.scm | 18 ++++++++++++++++--
 1 file changed, 16 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index 02ec731..9aa704b 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -52,6 +52,7 @@
   #:use-module (gnu packages adns)
   #:use-module (gnu packages attr)
   #:use-module (gnu packages backup)
+  #:use-module (gnu packages bash)
   #:use-module (gnu packages compression)
   #:use-module (gnu packages databases)
   #:use-module (gnu packages django)
@@ -379,6 +380,7 @@ data types.")
     (source #f)
     (build-system trivial-build-system)
     (outputs '("out"))
+    (inputs `(("bash" ,bash)))
     (propagated-inputs `(("python" ,python)))
     (arguments
      `(#:modules ((guix build utils))
@@ -392,8 +394,20 @@ 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" ,"pip3")
+                  `("python"  ,"pydoc"  ,"idle"  ,"pip"))
+                ;; python-config outputs search paths based upon its location,
+                ;; use a bash wrapper to avoid changing its behavior.
+                (let ((bash (string-append (assoc-ref %build-inputs "bash")
+                                           "/bin/bash"))
+                      (old  (string-append python "python3-config"))
+                      (new  (string-append bin "/python-config")))
+                  (with-output-to-file new
+                    (lambda ()
+                      (format #t "#!~a~%" bash)
+                      (format #t "exec \"~a\" \"$@\"~%" old)
+                      (chmod new #o755)
+                      #t)))))))
     (synopsis "Wrapper for the Python 3 commands")
     (description
      "This package provides wrappers for the commands of Python@tie{}3.x such
-- 
2.10.0


^ permalink raw reply related	[flat|nested] 9+ messages in thread

* Re: [PATCH] gnu: wrap-python3: Use a bash wrapper for 'python3-config'. (was: python-pycairo in core-updates)
  2016-10-16  3:21     ` [PATCH] gnu: wrap-python3: Use a bash wrapper for 'python3-config'. (was: python-pycairo in core-updates) 宋文武
@ 2016-10-16 18:14       ` Leo Famulari
  2016-10-17 10:58         ` [PATCH] gnu: wrap-python3: Use a bash wrapper for 'python3-config' 宋文武
  0 siblings, 1 reply; 9+ messages in thread
From: Leo Famulari @ 2016-10-16 18:14 UTC (permalink / raw)
  To: 宋文武; +Cc: guix-devel

On Sun, Oct 16, 2016 at 11:21:13AM +0800, 宋文武 wrote:
> >> Hi, python-wrapper has a broken `python3-config', which
> >> output cflags to the wrapper instead of the origin python3,
> >> leading the build failure of python-pycairo.
> >
> > So it appears to resolve the CFLAGS it returns relative to its own
> > argv[0] or /proc/self/exe, right?
> >
> > Eventually we should fix this and reinstate this commit, but reverting
> > it was the right thing in the short term.
> 
> Here is a fix: 
>
> From b7badcf695fcb245224ffdf51cdb016b45f9a2a5 Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?=E5=AE=8B=E6=96=87=E6=AD=A6?= <iyzsong@gmail.com>
> Date: Sun, 16 Oct 2016 10:44:29 +0800
> Subject: [PATCH] gnu: wrap-python3:  Use a bash wrapper for 'python3-config'.
> 
> See <http://lists.gnu.org/archive/html/guix-devel/2016-07/msg00008.html>
> for details.
> 
> * gnu/packages/python.scm (wrap-python3)[arguments]: Use a bash wrapper
> instead of a symlink for python3-config.

I can confirm that python-pycairo builds with this patch applied to
core-updates.

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] gnu: wrap-python3: Use a bash wrapper for 'python3-config'.
  2016-10-16 18:14       ` Leo Famulari
@ 2016-10-17 10:58         ` 宋文武
  0 siblings, 0 replies; 9+ messages in thread
From: 宋文武 @ 2016-10-17 10:58 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel

Leo Famulari <leo@famulari.name> writes:

> On Sun, Oct 16, 2016 at 11:21:13AM +0800, 宋文武 wrote:
>> >> Hi, python-wrapper has a broken `python3-config', which
>> >> output cflags to the wrapper instead of the origin python3,
>> >> leading the build failure of python-pycairo.
>> >
>> > So it appears to resolve the CFLAGS it returns relative to its own
>> > argv[0] or /proc/self/exe, right?
>> >
>> > Eventually we should fix this and reinstate this commit, but reverting
>> > it was the right thing in the short term.
>> 
>> Here is a fix: 
>>
>> From b7badcf695fcb245224ffdf51cdb016b45f9a2a5 Mon Sep 17 00:00:00 2001
>> From: =?UTF-8?q?=E5=AE=8B=E6=96=87=E6=AD=A6?= <iyzsong@gmail.com>
>> Date: Sun, 16 Oct 2016 10:44:29 +0800
>> Subject: [PATCH] gnu: wrap-python3:  Use a bash wrapper for 'python3-config'.
>> 
>> See <http://lists.gnu.org/archive/html/guix-devel/2016-07/msg00008.html>
>> for details.
>> 
>> * gnu/packages/python.scm (wrap-python3)[arguments]: Use a bash wrapper
>> instead of a symlink for python3-config.
>
> I can confirm that python-pycairo builds with this patch applied to
> core-updates.

Pushed, thanks for the confirming :-)

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2016-10-17 10:58 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-07-01  9:02 core-updates! Ludovic Courtès
2016-07-01 11:33 ` python-pycairo in core-updates (Re: core-updates!) 宋文武
2016-07-01 15:33   ` Andreas Enge
2016-07-01 21:06     ` Leo Famulari
2016-07-02 13:12   ` Ludovic Courtès
2016-10-16  3:21     ` [PATCH] gnu: wrap-python3: Use a bash wrapper for 'python3-config'. (was: python-pycairo in core-updates) 宋文武
2016-10-16 18:14       ` Leo Famulari
2016-10-17 10:58         ` [PATCH] gnu: wrap-python3: Use a bash wrapper for 'python3-config' 宋文武
2016-07-01 20:59 ` core-updates! Leo Famulari

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.