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
Replace PackageAI
with Package
references
#820
Conversation
* They are not used in any YaST module.
3c5ac40
to
1964681
Compare
@@ -151,12 +151,14 @@ def Import(settings) | |||
to: "list <string>" | |||
) | |||
|
|||
PackageAI.toinstall = settings.fetch("packages", []) | |||
to_install = settings.fetch("packages", []) | |||
PackagesProposal.AddResolvables("autoyast", :package, to_install) unless to_install.empty? |
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.
Is it AddResolvables or SetResolvables? Is expected to have more there?
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.
I preferred to be conservative and use AddResolvables
, just in case for some reason we add more packages from a different place. In that case, the selection would be lost.
@kernel = settings.fetch("kernel", "") | ||
|
||
addPostPackages(settings.fetch("post-packages", [])) | ||
AutoinstData.post_patterns = settings.fetch("post-patterns", []) | ||
PackageAI.toremove = settings.fetch("remove-packages", []) | ||
to_remove = settings.fetch("remove-packages", []) | ||
PackagesProposal.AddTaboos("autoyast", :package, to_remove) unless to_remove.empty? |
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.
same here, Add or Set?
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.
Same than above :-)
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.
nothing critical
✔️ Public Jenkins job #222 successfully finished |
✔️ Internal Jenkins job #134 successfully finished |
Adapt AutoYaST to do not use the
PackageAI
module anymore. It should use the genericPackage
instead. See yast/yast-yast2#1219.NOTE: tests are expected to fail until the yast/yast-yast2#1219 is merged.