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] base: do not crash if rtlcss.cmd is not found #27926
Conversation
rtlcss = misc.find_in_path('rtlcss.cmd') | ||
try: | ||
rtlcss = misc.find_in_path('rtlcss.cmd') | ||
except IOError: |
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.
Shouldn't we put a warning to give an hint to the user ?
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.
It will still actually display the warning 'You need https://rtlcss.com/ to convert...'
robodoo r+ |
Merged, thanks! |
Changes in default can be bothersome, embed a full baseline configuration for reliability. closes #27926 Signed-off-by: Pierre Masereel <pim@odoo.com>
Changes in default can be bothersome, embed a full baseline configuration for reliability. closes #27926 Signed-off-by: Pierre Masereel <pim@odoo.com>
Changes in default can be bothersome, embed a full baseline configuration for reliability. closes #27926 Signed-off-by: Pierre Masereel <pim@odoo.com>
Changes in default can be bothersome, embed a full baseline configuration for reliability. closes #27926 Signed-off-by: Pierre Masereel <pim@odoo.com>
Changes in default can be bothersome, embed a full baseline configuration for reliability. closes #27926 Signed-off-by: Pierre Masereel <pim@odoo.com>
Changes in default can be bothersome, embed a full baseline configuration for reliability. closes #27926 Signed-off-by: Pierre Masereel <pim@odoo.com>
Changes in default can be bothersome, embed a full baseline configuration for reliability. closes #27926 Signed-off-by: Pierre Masereel <pim@odoo.com>
Changes in default can be bothersome, embed a full baseline configuration for reliability. closes #27926 Signed-off-by: Pierre Masereel <pim@odoo.com>
Changes in default can be bothersome, embed a full baseline configuration for reliability. closes #27926 Signed-off-by: Pierre Masereel <pim@odoo.com>
Changes in default can be bothersome, embed a full baseline configuration for reliability. closes odoo/odoo#27926 Signed-off-by: Pierre Masereel <pim@odoo.com>
Description of the issue/feature this PR addresses:
Current behavior before PR:
Desired behavior after PR is merged:
--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr