all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Hartmut Goebel <h.goebel@crazy-compilers.com>
To: Marius Bakke <mbakke@fastmail.com>, guix-devel@gnu.org
Subject: Re: [PATCH 00/14] Change python-build-system (fixes bug 20765)
Date: Tue, 4 Oct 2016 23:39:49 +0200	[thread overview]
Message-ID: <149f0d8c-94a2-122e-9fb1-d55a34767f04@crazy-compilers.com> (raw)
In-Reply-To: <87bmz5ih9u.fsf@ike.i-did-not-set--mail-host-address--so-tickle-me>

Am 30.09.2016 um 16:39 schrieb Marius Bakke:

> Also some packages are missing a dependency on "python-py"[1].

FYI: I found an example: python-pytest-cov fails in the "check" phase.
This is an undocumented requirement for the tests, resp. this is an
install-requirement for pytest, but pytest has "python-py" defined as a
normal input.

I investigated this and found that the existing build-system was kind of
broken :-( Prior to my changes, a .pth file was created in
in many cases - while now it is never creates.

(For those not common to Python: .pth-files are an old way to specify
additional directories to be added to sys.path (PYTHONPATH). .pth-file
were rarely used until setuptools and easy-install came up. And now
since pip is the recommended install method, they are bound to vanish
again.)

This .pth file did not only point to the .egg-directory (or file, in
case of a zipped .egg), but also to the .eggs of the requirement of the
current package. In the case of pytest, the .pth file looked something
like thus:

   ./pytest-1.2.3.egg
   /gnu/store/…-python-py-3.2.1/lib/python3.4/site-packages/py-3.2.1.egg

Thus the .pth-file did "propagate" the inputs.

Due to my change the .pth-file is gone now, and thus all required inputs
need to be declared as propageded-inputs as uix.

I'm working on this, but it requires to have a look at many of the
packages' source to do it right and will take some days to finish.

-- 
Regards
Hartmut Goebel

| Hartmut Goebel          | h.goebel@crazy-compilers.com               |
| www.crazy-compilers.com | compilers which you thought are impossible |

  parent reply	other threads:[~2016-10-04 21:39 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-28 13:58 [PATCH 00/14] Change python-build-system (fixes bug 20765) Hartmut Goebel
2016-09-28 13:58 ` [PATCH 01/14] gnu: ensure pip and setuptools are installed even for Python 2 Hartmut Goebel
2016-09-28 13:58 ` [PATCH 02/14] guix: build all Python packages with --single-version-externally-managed Hartmut Goebel
2016-09-28 13:58 ` [PATCH 03/14] guix: Add lint-checker for packages which should be no inputs at all Hartmut Goebel
2016-09-28 13:58 ` [PATCH 04/14] gnu: python-setuptools: remove pre-built binaries Hartmut Goebel
2016-09-28 13:58 ` [PATCH 05/14] gnu: Remove python-setuptools and python2-setuptools from inputs (part 1) Hartmut Goebel
2016-09-28 13:58 ` [PATCH 06/14] gnu: Remove python-setuptools and python2-setuptools from inputs (part 2) Hartmut Goebel
2016-09-28 13:58 ` [PATCH 07/14] gnu: Remove python-setuptools and python2-setuptools from inputs (part 3) Hartmut Goebel
2016-09-28 13:58 ` [PATCH 08/14] gnu: Remove python-setuptools and python2-setuptools from inputs (part 4) Hartmut Goebel
2016-09-28 13:58 ` [PATCH 09/14] gnu: Remove python-setuptools and python2-setuptools from inputs (part 5a) Hartmut Goebel
2016-09-28 13:58 ` [PATCH 10/14] gnu: Remove python-setuptools and python2-setuptools from inputs (part 5b) Hartmut Goebel
2016-09-28 13:58 ` [PATCH 11/14] gnu: Remove needless inputs python-pip and python2-pip Hartmut Goebel
2016-09-28 13:58 ` [PATCH 12/14] lint: 'check-inputs-should-not-be-an-input-at-all' checks for python-pip Hartmut Goebel
2016-09-28 13:58 ` [PATCH 13/14] gnu: python-h5py: Remove needless "python2-variant" property Hartmut Goebel
2016-09-28 13:58 ` [PATCH 14/14] gnu: Remove work-arounds for bug 20765 (ensure uncompressed eggs) Hartmut Goebel
2016-09-28 14:03 ` [PATCH 00/14] Change python-build-system (fixes bug 20765) Hartmut Goebel
2016-09-28 15:54   ` Marius Bakke
2016-09-28 17:33     ` Hartmut Goebel
2016-09-28 17:40     ` Hartmut Goebel
2016-09-30 14:39       ` Marius Bakke
2016-10-02  9:05         ` Hartmut Goebel
2016-10-02 12:14           ` Hartmut Goebel
2016-10-02 14:24         ` Ludovic Courtès
2016-10-02 17:31           ` Hartmut Goebel
2016-10-02 18:01             ` Leo Famulari
2016-10-02 19:25               ` Marius Bakke
2016-10-10 13:24               ` Please set up Hydra channel for new python-build-system (was: [PATCH 00/14] Change python-build-system (fixes bug 20765)) Hartmut Goebel
2016-10-12 14:29                 ` Leo Famulari
2016-10-12 14:37                   ` Andreas Enge
2016-10-12 14:51                     ` Leo Famulari
2016-10-13  8:17                       ` Vincent Legoll
2016-10-03  9:31         ` [PATCH 00/14] Change python-build-system (fixes bug 20765) Hartmut Goebel
2016-10-04 21:39         ` Hartmut Goebel [this message]
2016-10-02  2:27 ` Leo Famulari
2016-10-02  2:46   ` Leo Famulari
2016-10-02  9:19     ` Hartmut Goebel
2016-10-02  9:18   ` Hartmut Goebel

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=149f0d8c-94a2-122e-9fb1-d55a34767f04@crazy-compilers.com \
    --to=h.goebel@crazy-compilers.com \
    --cc=guix-devel@gnu.org \
    --cc=mbakke@fastmail.com \
    /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.