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
using xml:base with microdnf and packagekit #833
Conversation
libdnf/dnf-package.cpp
Outdated
* | ||
* Returns: %TRUE if the pkg is a pkg on local or media filesystem | ||
* | ||
* Since: 0.37.1 |
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.
Can you please bump the versions here, since 0.37.2
was already released...
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.
Will do.
…865) Repository can have remote and local packages at the same time using xml:base, we have to ask the pkg if it is local. The package needs to have specified repository otherwise we cannot tell with certainty if it is local or remote. https://bugzilla.redhat.com/show_bug.cgi?id=1734350 https://bugzilla.redhat.com/show_bug.cgi?id=1717865
Repos packages attribute is solely used for path to packages cache, path to packages of a local repository is calculated using repos attribute location anyway. https://bugzilla.redhat.com/show_bug.cgi?id=1734350 https://bugzilla.redhat.com/show_bug.cgi?id=1717865
I managed to simplify the change. With this approach there is no need for a new attribute for packages cache or 2 new methods for controlling it. |
Congratulations! The build has finished successfully. 🍾 You can install the built RPMs by following these steps:
Please note that the RPMs should be used only in a testing environment. |
@kontura So this is ready for review now? |
ah, sorry, yes I think it is 👍 |
📌 Commit 394edbb has been approved by |
…865) Repository can have remote and local packages at the same time using xml:base, we have to ask the pkg if it is local. The package needs to have specified repository otherwise we cannot tell with certainty if it is local or remote. https://bugzilla.redhat.com/show_bug.cgi?id=1734350 https://bugzilla.redhat.com/show_bug.cgi?id=1717865 Closes: #833 Approved by: Conan-Kudo
Repos packages attribute is solely used for path to packages cache, path to packages of a local repository is calculated using repos attribute location anyway. https://bugzilla.redhat.com/show_bug.cgi?id=1734350 https://bugzilla.redhat.com/show_bug.cgi?id=1717865 Closes: #833 Approved by: Conan-Kudo
💥 Test timed out |
Since the changes have been approved and the tests pass locally I am merging this. |
This adds 3 functions to the API.
https://bugzilla.redhat.com/show_bug.cgi?id=1734350
https://bugzilla.redhat.com/show_bug.cgi?id=1717865