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

Add podman-based development support #84

Merged
merged 7 commits into from Feb 13, 2019

Conversation

dmnks
Copy link
Contributor

@dmnks dmnks commented Jan 17, 2019

Improves a developer's (my) life :)

@dmnks dmnks changed the title Add podman support Add podman-based development support Jan 17, 2019
@dmnks
Copy link
Contributor Author

dmnks commented Jan 17, 2019

@james-antill Hey, I don't feel like merging this without having involved another pair of eyes, so I'd like to ask you for a quick glance before I do. Note that this is just an upstream change that's not intended for Fedora or RHEL. Thanks!

@dmnks dmnks force-pushed the add-podman-support branch 5 times, most recently from 74c6908 to bdac7ba Compare January 17, 2019 15:10
This brings back old pygpgme support (when python2-gpg is not available)
removed by commit 2996c09, allowing the upstream version of yum to work
on CentOS/RHEL <= 7 again.
Source1 was never defined in the upstream spec file (so building an RPM
would fail).  In Fedora, we use it to override the upstream yum.conf
with our own.  As upstream should rather be distro-agnostic, let's just
stick with the defaults here.
@dmnks dmnks force-pushed the add-podman-support branch 2 times, most recently from 376347d to 9ac368c Compare January 18, 2019 12:16
@dmnks dmnks closed this Jan 18, 2019
@dmnks dmnks reopened this Jan 21, 2019
Mostly useful for development.
This allows for running the binary from the source tree (with PYTHONPATH
set to the tree).

Previously, the binary would try to import itself when it calls "import
yum" instead of the top-level package "yum" (the script directory is
inserted _before_ PYTHONPATH in sys.path).
After the newly migrated webserver VM, the original links for the
PLUGINS and FAQ files no longer work, making "make rpm" fail.  Let's
replace these with empty files (the upstream spec file is not used for
official releases anyway).
@dmnks dmnks force-pushed the add-podman-support branch 3 times, most recently from c5d0987 to 4507edf Compare February 12, 2019 19:24
Copy link
Member

@Conan-Kudo Conan-Kudo left a comment

Choose a reason for hiding this comment

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

I'm not @james-antill, but the changes look good to me.

@dmnks
Copy link
Contributor Author

dmnks commented Feb 13, 2019

I suppose it won't do any harm to have this merged, so I'll just do that :)

@dmnks dmnks merged commit d97a853 into rpm-software-management:master Feb 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants