From: "Michał Górny" <mgorny@gentoo.org>
To: Mike Gilbert <floppym@gentoo.org>
Cc: gentoo-python@lists.gentoo.org
Subject: Re: [gentoo-python] distutils-r1 -- support for out-of-source builds?
Date: Sun, 28 Oct 2012 17:11:15 +0100 [thread overview]
Message-ID: <20121028171115.26599f0d@pomiocik.lan> (raw)
In-Reply-To: <CAJ0EP400yViKCpaTVPgDkcUJkhsH+uHVS4Zy9_hmrfX8qLRifg@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 2774 bytes --]
On Sun, 28 Oct 2012 12:04:04 -0400
Mike Gilbert <floppym@gentoo.org> wrote:
> On Sun, Oct 28, 2012 at 11:55 AM, Michał Górny <mgorny@gentoo.org> wrote:
> > On Sun, 28 Oct 2012 08:57:05 +0100
> > Michał Górny <mgorny@gentoo.org> wrote:
> >
> >> On the other hand, we would introduce a tiny bit of additional code.
> >
> > I did a bit research on the topic, and the code introduced is circa 100
> > lines, including docs and esetuppy() which would be probably added
> > anyway. There are other potential problems however.
> >
> > The main issue is how distutils is designed. The only command accepting
> > build-dir is the 'build' command. Other commands which need to access
> > the build tree just use some weird method of getting parameters for
> > other commands. Effectively, if you need to provide a build-dir, you
> > always have to run 'build'.
> >
> > So, the install command would grow from:
> >
> > ./setup.py install ...
> >
> > to:
> >
> > ./setup.py build -b ... install ...
> >
> > Of course, all other commands accessing the build-dir would need to
> > be prepended with the 'build -b ...' as well. For that reason,
> > the patches I'm going to attach add that thing to esetuppy() directly.
> >
> > This has the following implications:
> >
> > 1) all calls to esetuppy() involve running build,
> >
> > 2) if build takes additional arguments, they may need to be passed
> > consistently to all esetuppy calls (like: 'build --foo' in args).
> > Otherwise, a rebuild with different options may happen (but that's
> > probably a case already anyway).
> >
> > In any case, I wouldn't say it's very elegant of Python. It will work
> > flawlessly for the most of Python packages, and likely most
> > of the remaining ones would require running in-source builds anyway
> > (and that can be enabled with a single variable).
> >
> > Well, best just look at the code and tell me what you think.
> >
>
> I don't really see any need for out-of-source builds, but if you want
> to write the code for it I certainly don't mind having it there.
>
> I will say that doing out-of-source builds occasionally makes writing
> src_test a bit tricky; you have to play with the working directory and
> PYTHONPATH.
Yes, I have fixed gentoopm to make it compile Python modules (in tests)
in build-dir rather than sourcedir.
https://bitbucket.org/mgorny/gentoopm/changeset/f46c4057d76
https://bitbucket.org/mgorny/gentoopm/changeset/975fd2dde55
The first commit should be enough for most packages out there.
The second one is necessary if any of the modules were loaded before
(e.g. in gentoopm to get PV). But probably getting the PV like that was
a bad idea anyway...
--
Best regards,
Michał Górny
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 316 bytes --]
prev parent reply other threads:[~2012-10-28 16:10 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-28 7:57 [gentoo-python] distutils-r1 -- support for out-of-source builds? Michał Górny
2012-10-28 15:55 ` Michał Górny
2012-10-28 16:04 ` Mike Gilbert
2012-10-28 16:11 ` Michał Górny [this message]
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=20121028171115.26599f0d@pomiocik.lan \
--to=mgorny@gentoo.org \
--cc=floppym@gentoo.org \
--cc=gentoo-python@lists.gentoo.org \
/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.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox