-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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] packaging, adapt documentation #9116
Conversation
ecd1ba9
to
17ccd34
Compare
17ccd34
to
b332392
Compare
robodoo r+ |
lxml.hml.clean was moved to a separate package starting in lxml 5.2 The requirements and debian control needs to be adapted to install the new package in ubuntu noble (and after ubuntu noble only) The solution was to use fallback to try to install lxml_html_clean if it exists (noble) and fallback on lxml if not. The script use in the documentation and available in more recent version will take all first packge of the debian conrtol, meaning that after this change it won't work in distribution not having lxml_html_clean. The script is adapted to use fallback in apt-install command Debian control: font1 | font2, package_1, package_noble | package_fallback, Before: apt install font1 package package_noble After: apt install font1|font2 package package_noble|package_fallback closes #163788 Related: odoo/documentation#9116 Signed-off-by: Christophe Monniez (moc) <moc@odoo.com>
closes #9116 Related: odoo/odoo#163788 Signed-off-by: Xavier Dollé (xdo) <xdo@odoo.com>
lxml.hml.clean was moved to a separate package starting in lxml 5.2 The requirements and debian control needs to be adapted to install the new package in ubuntu noble (and after ubuntu noble only) The solution was to use fallback to try to install lxml_html_clean if it exists (noble) and fallback on lxml if not. The script use in the documentation and available in more recent version will take all first packge of the debian conrtol, meaning that after this change it won't work in distribution not having lxml_html_clean. The script is adapted to use fallback in apt-install command Debian control: font1 | font2, package_1, package_noble | package_fallback, Before: apt install font1 package package_noble After: apt install font1|font2 package package_noble|package_fallback closes #163788 Related: odoo/documentation#9116 Signed-off-by: Christophe Monniez (moc) <moc@odoo.com>
closes #9116 Related: odoo/odoo#163788 Signed-off-by: Xavier Dollé (xdo) <xdo@odoo.com>
On Debian/Ubuntu, the following commands should install the required packages: | ||
|
||
.. code-block:: console | ||
|
||
$ cd /CommunityPath | ||
$ sed -n -e '/^Depends:/,/^Pre/ s/ python3-\(.*\),/python3-\1/p' debian/control | sudo xargs apt-get install -y | ||
$ cd odoo #CommunityPath |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The CommunityPath placeholder is explained above and presented in the /CommunityPath
format in commands in many other places on this page; let's keep it.
$ cd odoo #CommunityPath | |
$ cd /CommunityPath |
It could maybe be made clearer, but then it should be done for the entire page at once, not just this one command block.
|
||
Using pip may lead to security issue and broken dependencies, only do this if you | ||
know what you rare doing. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- removed extra line
- issues
- , -> ;
- rare -> are
Using pip may lead to security issue and broken dependencies, only do this if you | |
know what you rare doing. | |
Using pip may lead to security issues and broken dependencies; only do this if you | |
know what you are doing. |
@Xavier-Do this pull request has forward-port PRs awaiting action (not merged or closed): |
@Xavier-Do this pull request has forward-port PRs awaiting action (not merged or closed): |
lxml.hml.clean was moved to a separate package starting in lxml 5.2 The requirements and debian control needs to be adapted to install the new package in ubuntu noble (and after ubuntu noble only) The solution was to use fallback to try to install lxml_html_clean if it exists (noble) and fallback on lxml if not. The script use in the documentation and available in more recent version will take all first packge of the debian conrtol, meaning that after this change it won't work in distribution not having lxml_html_clean. The script is adapted to use fallback in apt-install command Debian control: font1 | font2, package_1, package_noble | package_fallback, Before: apt install font1 package package_noble After: apt install font1|font2 package package_noble|package_fallback closes odoo#163788 Related: odoo/documentation#9116 Signed-off-by: Christophe Monniez (moc) <moc@odoo.com>
@Xavier-Do this pull request has forward-port PRs awaiting action (not merged or closed): |
The debian/controll changed making the previous sed outdated, adapt to use a script for an easier onboarding and mainteance.
Also, warn user that pip option is not adviced.