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

Mass rebuild packages with Python 3.11 #746

Merged

Conversation

Odilhao
Copy link
Member

@Odilhao Odilhao commented Nov 11, 2023

No description provided.

@Odilhao Odilhao force-pushed the rpm/develop-insert-macro-into-specs branch from 2c6102f to 94672d3 Compare November 13, 2023 01:52
packages/ansible-lint/ansible-lint.spec Show resolved Hide resolved
packages/ansible-lint/ansible-lint.spec Show resolved Hide resolved
packages/ansible-lint/ansible-lint.spec Show resolved Hide resolved
BuildRequires: python%{python3_pkgversion}-setuptools_scm_git_archive
BuildRequires: pyproject-rpm-macros

%{?python_provide:%python_provide python%{python3_pkgversion}-%{pypi_name}}
# This should be
Copy link
Member

Choose a reason for hiding this comment

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

We can now support rich dependencies so you can update it.

Copy link
Member Author

Choose a reason for hiding this comment

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

How would be the best to change this?

Copy link
Member

Choose a reason for hiding this comment

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

I think the suggestion is in the file itself. @evgeni any thoughts?

Copy link
Member Author

Choose a reason for hiding this comment

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

pyproject-rpm-macros is a project we build for EL8, it's present on EL9 by default, but we need this for PEP-517, we build this internally

Copy link
Member

Choose a reason for hiding this comment

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

I'm not talking about the BuildRequires, but about the comment (and the lines that follow).

Copy link
Member Author

Choose a reason for hiding this comment

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

Oh ok, we would need to work in all packages, because we use the same line for all packages.

packages/createrepo_c/createrepo_c.spec Outdated Show resolved Hide resolved
packages/libsolv/libsolv.spec Outdated Show resolved Hide resolved
packages/libsolv/libsolv.spec Show resolved Hide resolved
packages/libsolv/libsolv.spec Show resolved Hide resolved
packages/libsolv/libsolv.spec Show resolved Hide resolved
packages/libsolv/libsolv.spec Show resolved Hide resolved
@Odilhao
Copy link
Member Author

Odilhao commented Nov 13, 2023

Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

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

We declare the module dependencies here:

- buildrequires:
platform: ['el8']
python39: ['3.9']
requires:
platform: ['el8']
python39: ['3.9']

At the least, we should drop that. But if we no longer need modularity for Python 3.11, does it still make sense to have a pulpcore module?

Python 3.11 don't ship with modules on EL8
@Odilhao
Copy link
Member Author

Odilhao commented Nov 13, 2023

Modules dropped with 240f0b8

@Odilhao
Copy link
Member Author

Odilhao commented Nov 14, 2023

@ekohl can we merge this? I want to address the other pulp bits today, and later this week work on the obsoletes that we need for upgrades from python39 to 3.11

@pcreech
Copy link
Member

pcreech commented Nov 14, 2023

So, reviewing this pr, and the trade-offs of needing something functional to keep other trains moving smoothly, I'm going ahead and merging this PR. It appears most everything should be mostly functional to a degree good enough to iterate on top of to fix issues that crop up.

Anything that is felt should have been addressed before merging this PR, let's identify them and create action items to follow up on asap after.

@pcreech pcreech merged commit d95f72e into theforeman:rpm/develop Nov 14, 2023
1 of 3 checks passed
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.

None yet

3 participants