openerp-india team mailing list archive
-
openerp-india team
-
Mailing list archive
-
Message #14058
[Bug 1017120] Re: import babel in setup.py breaks linux installs
Hello Chrisophe,
Yes, you are right. Import babel only require on windows version at
setup.py and I agreed with your solution.
So I am assigning this as a "Low" priority.
If you want to contribute on OpenERP the would you please create a stack branch for server project and commit your changes/fixes. Then put the merge purposal(MP) for trunk branch and our core team will review it.
That's better if you providing a MP instead of giving solution on bug
which is also easy for us.
Hope you are understand and start to contribute on OpenERP.
Thank you!
** Changed in: openobject-server
Importance: Undecided => Low
** Changed in: openobject-server
Status: New => Confirmed
** Changed in: openobject-server
Assignee: (unassigned) => OpenERP's Framework R&D (openerp-dev-framework)
--
You received this bug notification because you are a member of OpenERP
Indian Team, which is subscribed to OpenERP Server.
https://bugs.launchpad.net/bugs/1017120
Title:
import babel in setup.py breaks linux installs
Status in OpenERP Server:
Confirmed
Bug description:
The setup.py file contains both an "import babel" and a babel requirement.
When running the setup, babel should be installed as a dependency and should not be required by the setup itself.
This seems to be done to workaround a bug in windows with py2exe.
However on Linux we don't need this workaround : the workaround for windows is turned into a bug for linux.
I suggest to move the "import babel" in the "if" block for windows
(as it was probably intended to be done) :
setup.py:41 :
if os.name == 'nt':
r.append(("Microsoft.VC90.CRT", glob.glob('C:\Microsoft.VC90.CRT\*.*')))
import babel
r.append(("localedata",
glob.glob(os.path.join(os.path.dirname(babel.__file__), "localedata" , '*'))))
To manage notifications about this bug go to:
https://bugs.launchpad.net/openobject-server/+bug/1017120/+subscriptions
References