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
[merged] This variable starts out so unsubstituted. Until the end. #274
Conversation
A future commit is going to change our parsing of the "ref" member in treefiles, so ensure we only load it once early on in compose-tree. We already looked up the previous commit there, so just pass it down rather than reloading the ref.
This comes from the tradition of yum repo files. It's significantly easier for rpm-ostree users building multiple architectures if some core templating for the treefiles is built in. Otherwise, everyone needs to learn about a new wrapper tool for rpm-ostree, and said tool would need to re-do the same "basearch" evaluation that is already occuring inside libhif. This commit also paves the way for introducing `${releasever}` substitution. NOTE: This depends on pending changes to libhif git.
This depends on rpm-software-management/libdnf#112 |
@cgwalters-bot retry |
bot, retest this please |
@@ -32,6 +32,7 @@ | |||
#include <libhif/hif-utils.h> | |||
#include <libhif/hif-package.h> | |||
#include <librepo/librepo.h> | |||
#include <librepo/librepo.h> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Extraneous?
Can't we use |
Hmm, right, we don't really have a HifRepo to begin with here. |
I was trying to think of a way to leverage what libhif already does for this, but there's nothing that really jumps out. |
This comes from the tradition of yum repo files. It's significantly easier for rpm-ostree users building multiple architectures if some core templating for the treefiles is built in. Otherwise, everyone needs to learn about a new wrapper tool for rpm-ostree, and said tool would need to re-do the same "basearch" evaluation that is already occuring inside libhif. This commit also paves the way for introducing `${releasever}` substitution. NOTE: This depends on pending changes to libhif git. Closes: #274 Approved by: jlebon
☀️ Test successful - status-atomicjenkins |
No description provided.