Skip to content

[FIX] packaging: fix rpm package and stop using bdist_rpm#84665

Closed
d-fence wants to merge 1 commit intoodoo:masterfrom
odoo-dev:master-fix-rpmpack-moc
Closed

[FIX] packaging: fix rpm package and stop using bdist_rpm#84665
d-fence wants to merge 1 commit intoodoo:masterfrom
odoo-dev:master-fix-rpmpack-moc

Conversation

@d-fence
Copy link
Copy Markdown
Contributor

@d-fence d-fence commented Feb 15, 2022

The rpm package failed recently on a file name containing a space.
This issue is fixed by protecting file names with double quotes in the
installed files list.

Also, during this fix, it was discovered that the bdist_rpm is
deprecated in setuptools [0].

For that reason, the rpm is now generated from the package.py script.

[0] pypa/setuptools#2780

@robodoo
Copy link
Copy Markdown
Contributor

robodoo commented Feb 15, 2022

Pull request status dashboard

@C3POdoo C3POdoo added the RD research & development, internal work label Feb 15, 2022
@d-fence d-fence force-pushed the master-fix-rpmpack-moc branch 3 times, most recently from 1ebde89 to 0e96878 Compare February 16, 2022 12:38
@d-fence d-fence force-pushed the master-fix-rpmpack-moc branch from 0e96878 to 5f22fc8 Compare February 16, 2022 13:20
The rpm package failed recently on a file name containing a space.
This issue is fixed by protecting file names with double quotes in the
installed files list.

Also, during this fix, it was discovered that the bdist_rpm is
deprecated in setuptools [0].

For that reason, the rpm is now generated from the package.py script.

[0] pypa/setuptools#2780
@d-fence d-fence force-pushed the master-fix-rpmpack-moc branch from 5f22fc8 to 8d47e1d Compare February 16, 2022 17:17
@d-fence
Copy link
Copy Markdown
Contributor Author

d-fence commented Feb 16, 2022

robodoo r+

robodoo pushed a commit that referenced this pull request Feb 16, 2022
The rpm package failed recently on a file name containing a space.
This issue is fixed by protecting file names with double quotes in the
installed files list.

Also, during this fix, it was discovered that the bdist_rpm is
deprecated in setuptools [0].

For that reason, the rpm is now generated from the package.py script.

[0] pypa/setuptools#2780

closes #84665

Signed-off-by: Christophe Monniez (moc) <moc@odoo.com>
@robodoo robodoo closed this Feb 16, 2022
@robodoo robodoo temporarily deployed to merge February 16, 2022 20:21 Inactive
@robodoo robodoo added the 15.3 label Feb 16, 2022
@d-fence d-fence deleted the master-fix-rpmpack-moc branch February 16, 2022 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

15.3 RD research & development, internal work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants