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

pyrpmostreetoolbox: New Python library #70

Closed
wants to merge 5 commits into from
Closed

Conversation

cgwalters
Copy link
Member

This can be used by other projects to do rpm-ostree related rel-eng
stuff, since rpm-ostree isn't a shared library right now.

Change the treecompose command to use it.

This can be used by other projects to do rpm-ostree related rel-eng
stuff, since rpm-ostree isn't a shared library right now.

Change the treecompose command to use it.
@mbarnes
Copy link
Contributor

mbarnes commented Mar 12, 2015

What kind of stability guarantee are we making with this new library?

@cgwalters
Copy link
Member Author

I think it has to be "forever".

This can be used by scripts to generate reports and such.
This can be used by release engineering tools.
@cgwalters
Copy link
Member Author

I think doing much more is going to block a bit on test cases. Which in turn gets messy as rpm-ostree compose tree needs to run as root because rpm does. This is related to coreos/rpm-ostree#90

@cgwalters
Copy link
Member Author

I'm thinking that we should make a new git repository dedicated to python-rpmostree releng bits, and change -toolbox to dep on that. Any objections?

@cgwalters
Copy link
Member Author

Right now -toolbox i s a big bag of stuff, pulling in lorax/imagefactory/docker, plus a big pile of Perl, etc., whereas not every releng task needs all of that.

@mbarnes
Copy link
Contributor

mbarnes commented Mar 16, 2015

Splitting it out seems sensible. Lugging around a whole toolbox when you just need a screwdriver is suboptimal.

Though it kinda feels like those bits should somehow, someday end up in rpm-ostree proper.

@cgwalters
Copy link
Member Author

This is obsoleted by coreos/rpm-ostree#139

@cgwalters cgwalters closed this Jun 15, 2015
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