Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 315335 - net-mail/mailman: Verify and potentially improve Python-related code
Summary: net-mail/mailman: Verify and potentially improve Python-related code
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All All
: High normal (vote)
Assignee: Hanno Böck
URL:
Whiteboard:
Keywords:
: 374293 (view as bug list)
Depends on:
Blocks: 308257
  Show dependency tree
 
Reported: 2010-04-14 21:10 UTC by Arfrever Frehtes Taifersar Arahesis (RETIRED)
Modified: 2012-08-08 17:38 UTC (History)
6 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
ebuild patch (mailman.patch,1.60 KB, patch)
2011-11-15 19:57 UTC, Ian Delaney (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Arfrever Frehtes Taifersar Arahesis (RETIRED) gentoo-dev 2010-04-14 21:10:52 UTC
- If this package can be used as a library and installs Python modules
  (*.so or *.py) into site-packages directories, then consider supporting
  installation for multiple Python versions.
  Please read section "Types of packages" in documentation [1] to decide
  if this package can support installation for multiple Python versions.

- Ensure that the ebuilds do not use deprecated functions or variables.

- Please check if Python 3 is supported by this package. You can temporarily
  set Python 3 as main active version of Python to properly test if this
  package supports Python 3.

- If this package does not support Python 3:
  - Specify dependency on Python 2.
    You can use PYTHON_DEPEND helper variable, which should be set before
    inheriting of python eclass.
    Please read section "Specification of dependency on Python" in
    documentation [1].

  - If this package cannot support installation for multiple versions of
    Python, then set active version of Python using
    python_set_active_version().

  - Ensure that shebangs in installed scripts specify correct version of
    Python. If shebangs are too generic (e.g. '#!/usr/bin/python'), then you
    can use python_convert_shebangs() to convert shebangs.
    (Wrapper scripts generated by python_generate_wrapper_scripts() do not
    require any changes.)
    Please read section "Shebangs in installed scripts" in documentation [1].

  - To ensure that changes applied to the ebuilds are sufficient, please
    temporarily set Python 3 as main active version of Python and test if
    this package can be properly installed and if it works at run time.

Please see documentation [1] for more details.
[1] http://www.gentoo.org/proj/en/Python/developersguide.xml
Comment 1 Nathan Phillip Brink (binki) (RETIRED) gentoo-dev 2010-04-15 04:25:02 UTC
Mailman fails on python3. This should be documented and shebangs need to be properly mangled. This includes the crontab, I believe, which calls ``/usr/bin/python -S''.

You also get to have the fun of making sure that the direct calls to /usr/bin/python inside of the compiled portions of mailman call python2 ;-)

ohnopublishing portage # grep -a -o  '.\{20\}/bin/python.\{20\}' /usr/lib64/mailman/mail/mailman 
r/lib64/mailman/usr/bin/pythonPYTHONPATH=PYTHONH
ohnopublishing portage # file /usr/lib64/mailman/mail/mailman 
/usr/lib64/mailman/mail/mailman: ELF 64-bit LSB executable, x86-64, version 1 (SYSV), dynamically linked (uses shared libs), for GNU/Linux 2.6.9, stripped

/me attempts to get his unimportant ML back up... no time for patches :-(
Comment 2 Ian Delaney (RETIRED) gentoo-dev 2011-11-15 19:57:12 UTC
Created attachment 292681 [details, diff]
ebuild patch
Comment 3 Arfrever Frehtes Taifersar Arahesis 2011-11-16 19:13:34 UTC
(In reply to comment #2)

Read comment #1.
Comment 4 Ian Delaney (RETIRED) gentoo-dev 2011-11-19 12:47:53 UTC
I think I shall await a plan of attack from you in irc; this looks kind of, hard
Comment 5 Hanno Böck gentoo-dev 2012-08-08 17:31:42 UTC
*** Bug 374293 has been marked as a duplicate of this bug. ***
Comment 6 Hanno Böck gentoo-dev 2012-08-08 17:38:52 UTC
fixed with 2.1.15