Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FIX] support psutils >= 4.0. #471

Merged
merged 1 commit into from
Jul 4, 2016
Merged

[FIX] support psutils >= 4.0. #471

merged 1 commit into from
Jul 4, 2016

Conversation

NL66278
Copy link

@NL66278 NL66278 commented Apr 6, 2016

Description of the issue/feature this PR addresses:
Odoo does not run in virtual env due to incompatible psutils.

Current behavior before PR:
Server does not start properly, unless workers=0

Desired behavior after PR is merged:

Server starts properly. No matter the value for option workers.

I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

backport of 8.0 fix created by stephen144.

backport of 8.0 fix createded by stephen144.
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.004%) to 30.915% when pulling 1aaa49e on NL66278:7.0-update-psutil into f97227f on OCA:7.0.

@hbrunn hbrunn added this to the 7.0 milestone Apr 6, 2016
@hbrunn
Copy link
Member

hbrunn commented Apr 6, 2016

👍

@StefanRijnhart
Copy link
Member

You could have cherry picked 8562e11 but I guess final credit goes to one of the reviewers of that PR. I guess odoo#11459 can count as the upstream PR, so 👍

@NL66278
Copy link
Author

NL66278 commented Apr 6, 2016

@StefanRijnhart I considered cherry picking, but the code is in a different file in 8.0.

@StefanRijnhart
Copy link
Member

I see, thanks for clarifying.

@hbrunn hbrunn merged commit 94d50f1 into OCA:7.0 Jul 4, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants