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 |
next prev 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.