Skip to content
This repository has been archived by the owner on Aug 19, 2020. It is now read-only.

[RFC] Fedora 30: Install gobject via pip for no system-wide packages #16

Closed
wants to merge 1 commit into from

Conversation

mikedep333
Copy link
Member

@mikedep333 mikedep333 commented Oct 10, 2019

See email thread on pulp-dev "Dependency on gi / GObject"

@fao89
Copy link
Member

fao89 commented Jan 16, 2020

this seems to solve this issue:
https://pulp.plan.io/issues/5992

@dralley
Copy link
Contributor

dralley commented Jan 16, 2020

Getting rid of all of the build failures in provisioning would be enormously helpful, thank you

- python3-libmodulemd
- rpm-devel
- openssl-devel
- sqlite-devel
- xz-devel
- zchunk-devel
- zlib-devel

rpm_prereq_pip_packages:
- PyGObject
Copy link
Contributor

Choose a reason for hiding this comment

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

This is probably unnecessary as long as it's also listed in pulp_rpm's setup.py

@mikedep333
Copy link
Member Author

I recall that I decided against this because it made the install process larger & longer.

#14 is the preferred implementation.

@mikedep333 mikedep333 closed this Jan 17, 2020
@dralley
Copy link
Contributor

dralley commented Jan 17, 2020

@mikedep333 I don't think that patch fully fixes the issue, because I still can't vagrant up properly with the RPM plugin selected for installation.

@mikedep333
Copy link
Member Author

@dralley Do you have pulp_use_system_wide_pkgs set to true?

I'm going to submit a PR for pulplift right now to update to the latest ansible-pulp also.

@dralley
Copy link
Contributor

dralley commented Jan 17, 2020

I checked and I do not. Is it possible to set that variable as true automatically in the prerequisites plugin?

@mikedep333
Copy link
Member Author

@dralley I want to look how to do that properly and robustly (handles the different ways users are calling the roles, like static roles list vs dynamic includes_roles tasks.)

We'll soon have CI on ansible-pulp for dynamic include_roles. That will help us made a decision.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants