public inbox for gentoo-dev@lists.gentoo.org
 help / color / mirror / Atom feed
From: Brian Harring <ferringb@gmail.com>
To: Arfrever Frehtes Taifersar Arahesis <Arfrever@gentoo.org>
Cc: gentoo-dev@lists.gentoo.org
Subject: Re: [gentoo-dev] Versioning of Python scripts
Date: Mon, 21 Dec 2009 02:50:14 -0800	[thread overview]
Message-ID: <20091221105014.GA6292@hrair> (raw)
In-Reply-To: <200912191624.55244.Arfrever@gentoo.org>

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

On Sat, Dec 19, 2009 at 04:24:49PM +0100, Arfrever Frehtes Taifersar Arahesis wrote:
> Distutils/Setuptools/Distribute modify shebangs of installed Python scripts, so that they
> contain path of Python interpreter with version included (e.g. "#!/usr/bin/python3.2").
> This behavior has both advantage and disadvantages:
>   - Scripts of packages supporting only e.g. Python 2 can be executed (without necessity
>     of using of e.g. "python2 /usr/bin/${script}") after activating of e.g. Python 3.
>   - Scripts of packages supporting multiple Python versions ignore active Python version.
>   - Scripts of packages supporting multiple Python versions cannot be easily (without
>     necessity of using of e.g. "python3.1 /usr/bin/${script}") executed with a Python
>     version different than active Python version.
> The best solution, which removes these 2 disadvantages and preserves the advantage, seems
> to be to rename Python scripts to include Python version [1] in filenames, and create wrapper
> scripts, which call appropriate target scripts [2]. Some files sometimes try to execute
> e.g. "/usr/bin/python /usr/bin/${script}", so wrapper scripts must be implemented in Python.
> Wrapper scripts try to execute "${wrapper_script}-${PYTHON_ABI}" files (e.g. "py.test" will
> execute "py.test-3.1", when Python 3.1 is set as active Python version).
> 
> distutils.eclass will automatically rename some scripts [3] in "${D}usr/bin" and call
> the function, which generates wrapper scripts. In case somebody is interested in reading of
> source code of python_generate_wrapper_scripts() function and potential suggesting of
> improvements, I'm attaching this function and 2 example wrapper scripts. I'm planning to
> commit addition of this function in next week.

Not really a huge fan of the EPYTHON var... can you clarify it's real 
world usage?  I can see that causing all sorts of mayhem as it passes 
it's way down through python scripts invoking other scripts- 
specifically thinking of a py3k only script being forced to 3.1, then 
invoking a py2k script.

Beyond that, please provide a way to *disable* this for a pkg.  At 
least for pkgcore, I've already been looking at ways to deal with 
this and would rather solve it at the pkg level (since EPYTHON let 
alone eselect may not exist for certain target deployments of 
pkgcore itself).

Basically, no point in having wrapper scripts if the target already 
can do it's own version of this, hence wanting a way to disable it in 
the ebuild- that's just for the script mangling, library installing 
for multiple python abis is a seperate thing.

Aside from that, punting on the re import might be nice primarily for 
speed reasons (no it's not a huge import in cost, but this is an extra 
~.025 per python script invoked, ignoring the ~.3 to ~.02 for eselect 
dependant on cache status).

~harring

[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]

  reply	other threads:[~2009-12-21 12:09 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-12-19 15:24 [gentoo-dev] Versioning of Python scripts Arfrever Frehtes Taifersar Arahesis
2009-12-21 10:50 ` Brian Harring [this message]
2009-12-23  0:50   ` Arfrever Frehtes Taifersar Arahesis
2009-12-22  9:04 ` Peter Volkov

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=20091221105014.GA6292@hrair \
    --to=ferringb@gmail.com \
    --cc=Arfrever@gentoo.org \
    --cc=gentoo-dev@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