Skip to content
This repository has been archived by the owner on May 16, 2024. It is now read-only.

Commit

Permalink
[REF] travis_install_nightly: Installing pyyaml compatible with py3.7
Browse files Browse the repository at this point in the history
  • Loading branch information
@moylop260
moylop260 committed Jul 20, 2018
1 parent 9494026 commit 01d0816
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions travis/travis_install_nightly
View file
Original file line number Diff line number Diff line change
Expand Up @@ -107,6 +107,8 @@ sed -i '/^python-ldap\=\=/d' ${ODOO_PATH}/requirements.txt
# Use requests with [security] suffix to fix [Errno 111] Connection refused for old python2.7 versions.
sed -i 's/^requests\=\=/requests[security]\=\=/g' ${ODOO_PATH}/requirements.txt


sed -i "s/PyYAML==3.12/PyYAML==3.13/g" ${ODOO_PATH}/requirements.txt
pip install -q --no-binary pycparser -r ${ODOO_PATH}/requirements.txt
pip install -q QUnitSuite coveralls codecov

Expand Down

3 comments on commit 01d0816

@sbidoul
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That should not be necessary. We must not freeze requirements in OCA repos.

@moylop260
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @sbidoul
You are right!
It is not necessary.

It is just to justify my following comment:

@sbidoul
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@moylop260 sorry, I did not notice it was for Odoo requirements. Sorry for the noise.

Please sign in to comment.