← Back to team overview

openerp-india team mailing list archive

[Bug 1017120] Re: import babel in setup.py breaks linux installs

 

Your suggested fix was applied in trunk at server r.4643 rev-id:
odo@xxxxxxxxxxx-20121206135212-dt5hfcb35uy23k4

Thanks for reporting!

** Changed in: openobject-server
       Status: Confirmed => Fix Released

** Changed in: openobject-server
    Milestone: None => 7.0

-- 
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:
  Fix Released

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