Skip to content
This repository was archived by the owner on Dec 17, 2025. It is now read-only.

remove duplicated line "psutil" (whatever it is)#44

Open
lepistone wants to merge 1 commit intocamptocamp:masterfrom
lepistone:dedupe-psutil
Open

remove duplicated line "psutil" (whatever it is)#44
lepistone wants to merge 1 commit intocamptocamp:masterfrom
lepistone:dedupe-psutil

Conversation

@lepistone
Copy link
Contributor

No description provided.

@yvaucher
Copy link
Contributor

Any reason to keep the old version ?

@lepistone
Copy link
Contributor Author

I checked in odoo:

  • 8.0 has 2.1.1 in requirements.txt
  • 9.0 has 2.2.0 in requirements.txt

and actually the dependency seems optional:

https://github.com/odoo/odoo/blob/e1a17b433d87d871e1e215b7dfd182faa274eb5f/openerp/netsvc.py#L38-L41

I think there is no reason to keep the old one.

@yvaucher
Copy link
Contributor

yvaucher commented Apr 26, 2016

Anyway it is defined in most projects if not all. We can set it to 2.1.1.

@guewen
Copy link
Contributor

guewen commented Apr 26, 2016

for the record, ever psutil > 4.0 should work on the most recent revisions: odoo/odoo#11459

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants