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

Make --forcearch a global argument #941

Merged

Conversation

evan-goode
Copy link
Member

Many commands respect --forcearch (distro-sync, download, install, list, makecache, repoquery, search, etc.) so it makes sense to make --forcearch a global argument that can be used with any command.

Resolves #872

@evan-goode evan-goode self-assigned this Oct 4, 2023
@evan-goode evan-goode force-pushed the evan-goode/global-forcearch branch from 01abef0 to 7626ecc Compare October 6, 2023 15:27
@evan-goode evan-goode requested a review from j-mracek October 9, 2023 17:25
@j-mracek j-mracek self-assigned this Oct 11, 2023
dnf5/shared_options.cpp Outdated Show resolved Hide resolved
@j-mracek
Copy link
Contributor

If I am not mistaken, we agreed with the team to use deprivation warning raised during compilation, and to keep the same functionality for deprecated method even if it is unused API and there is low chance that is in use from other project.

@evan-goode evan-goode force-pushed the evan-goode/global-forcearch branch from 7626ecc to 24cf087 Compare October 16, 2023 16:26
@evan-goode
Copy link
Member Author

OK, I've kept the original create_forcearch_option there with the [deprecated] attribute. I've tested that it does not compile if you try to reference create_forcearch_option.

@evan-goode evan-goode requested a review from j-mracek October 16, 2023 16:27
@j-mracek
Copy link
Contributor

I was just on way to merge it and I discovered that there is a merge conflict. @evan-goode May I ask you for a rebase?

@evan-goode evan-goode force-pushed the evan-goode/global-forcearch branch from 24cf087 to 7a36a61 Compare October 17, 2023 15:40
Many commands respect --forcearch (distro-sync, download, install, list,
makecache, repoquery, search, etc.) so it makes sense to make
--forcearch a global argument that can be used with any command.

Resolves rpm-software-management#872
@evan-goode evan-goode force-pushed the evan-goode/global-forcearch branch from 7a36a61 to a8947f7 Compare October 17, 2023 15:47
@j-mracek j-mracek added this pull request to the merge queue Oct 18, 2023
Merged via the queue into rpm-software-management:main with commit 8b515b7 Oct 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Make --forcearch option a global one
2 participants