-
Notifications
You must be signed in to change notification settings - Fork 139
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
goal: return reason user if transaction was assembled without resolving #213
Conversation
…ng (RhBug:1393595) This is required by dnf history cmd as it doesn't resolve undo/redo transactions and get_reason is consequently called in the transaction output.
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.
Looks good to me, it fixes mga#19744 / rhbz#1393595 for me.
@@ -1179,8 +1179,8 @@ int | |||
hy_goal_get_reason(HyGoal goal, DnfPackage *pkg) | |||
{ | |||
//solver_get_recommendations | |||
|
|||
assert(goal->solv); | |||
if (!goal->solv) |
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.
let's use existing @hughsie's explicit style if (goal->solv == NULL)
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.
ok
…resolving (RhBug:1393595)
1 similar comment
📌 Commit 2fb3c11 has been approved by |
…ng (RhBug:1393595) This is required by dnf history cmd as it doesn't resolve undo/redo transactions and get_reason is consequently called in the transaction output. Closes: #213 Approved by: ignatenkobrain
💥 Test timed out |
@ignatenkobrain The bot exploded, can someone merge this manually? |
@dnf-bot r+ |
📌 Commit 2fb3c11 has been approved by |
@dnf-bot retry |
☀️ Test successful - status-jenkins |
This is required by dnf history cmd as it doesn't resolve undo/redo transactions
and get_reason is consequently called in the transaction output.