search

pediapress / mwlib

mediawiki parser library
103 stars 35 forks source link

mwlib provides top-level argv.py file, which can pollute the Python package namespace #8

Closed iliana closed 12 years ago

iliana commented 12 years ago

As part of the review process for getting mwlib into Fedora, the package reviewer stated the following:

Potential future issues:

* The package installs a toplevel argv.py file/module.  This seems to just be
  used for the command line scripts for parsing the command line.  Since this
  has the potential to pollute the python package namespace with a somewhat
  common name, suggest to upstream that they may want to restructure like
  this::
    mv argv.py mwlib/._argv.py
    sed s/import argv/from mwlib import _argv as argv/

Reference: https://bugzilla.redhat.com/show_bug.cgi?id=790154#c1