[Discover] [Bug 394327] Discover fails to warn about reverse dependences to be removed

2018-05-28 Thread Aleix Pol
https://bugs.kde.org/show_bug.cgi?id=394327

Aleix Pol  changed:

   What|Removed |Added

  Latest Commit|https://commits.kde.org/dis |https://commits.kde.org/dis
   |cover/ee8e49422118895f0c75b |cover/c0277f806a6eab8e9850e
   |5ea67308ac15d69a476 |4d022ebc0419dd8efc1

--- Comment #4 from Aleix Pol  ---
Git commit c0277f806a6eab8e9850e4d022ebc0419dd8efc1 by Aleix Pol, on behalf of
Antonio Rojas.
Committed on 28/05/2018 at 16:05.
Pushed by apol into branch 'Plasma/5.12'.

Abort transaction on PackageKit errors

If PackageKit throws an error, abort the transaction instead of finishing it
anyway (potentially breaking the user's installation).
This is far from ideal, but prevents Discover from performing destructive
operations while the PackageKit issues are fixed
Differential Revision: https://phabricator.kde.org/D13119

M  +6-2libdiscover/backends/PackageKitBackend/PKTransaction.cpp

https://commits.kde.org/discover/c0277f806a6eab8e9850e4d022ebc0419dd8efc1

-- 
You are receiving this mail because:
You are watching all bug changes.

[Discover] [Bug 394327] Discover fails to warn about reverse dependences to be removed

2018-05-26 Thread Antonio Rojas
https://bugs.kde.org/show_bug.cgi?id=394327

Antonio Rojas  changed:

   What|Removed |Added

   Version Fixed In||5.13.0

-- 
You are receiving this mail because:
You are watching all bug changes.

[Discover] [Bug 394327] Discover fails to warn about reverse dependences to be removed

2018-05-26 Thread Antonio Rojas
https://bugs.kde.org/show_bug.cgi?id=394327

Antonio Rojas  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |FIXED
  Latest Commit||https://commits.kde.org/dis
   ||cover/ee8e49422118895f0c75b
   ||5ea67308ac15d69a476

--- Comment #3 from Antonio Rojas  ---
Git commit ee8e49422118895f0c75b5ea67308ac15d69a476 by Antonio Rojas.
Committed on 26/05/2018 at 07:13.
Pushed by arojas into branch 'Plasma/5.13'.

Abort transaction on PackageKit errors

If PackageKit throws an error, abort the transaction instead of finishing it
anyway (potentially breaking the user's installation).
This is far from ideal, but prevents Discover from performing destructive
operations while the PackageKit issues are fixed
Differential Revision: https://phabricator.kde.org/D13119

M  +6-2libdiscover/backends/PackageKitBackend/PKTransaction.cpp

https://commits.kde.org/discover/ee8e49422118895f0c75b5ea67308ac15d69a476

-- 
You are receiving this mail because:
You are watching all bug changes.

[Discover] [Bug 394327] Discover fails to warn about reverse dependences to be removed

2018-05-25 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=394327

Nate Graham  changed:

   What|Removed |Added

 CC||n...@kde.org

-- 
You are receiving this mail because:
You are watching all bug changes.

[Discover] [Bug 394327] Discover fails to warn about reverse dependences to be removed

2018-05-18 Thread Antonio Rojas
https://bugs.kde.org/show_bug.cgi?id=394327

Antonio Rojas  changed:

   What|Removed |Added

 CC||aro...@archlinux.org

--- Comment #2 from Antonio Rojas  ---
This can be reproduced by trying to remove a package which is an *optional*
dependency of another one. Packagekit will then throw an error since it can't
handle the pacman warning:

PackageKit error: PackageKit::Transaction::Error(ErrorDepResolutionFailed)
"Dependency resolution failed" "calligra\noptionally requires kirigami2: for
Calligra Gemini\n"

and then discover will proceed to uninstall the package and all reverse
dependencies anyway!

-- 
You are receiving this mail because:
You are watching all bug changes.

[Discover] [Bug 394327] Discover fails to warn about reverse dependences to be removed

2018-05-18 Thread Aleix Pol
https://bugs.kde.org/show_bug.cgi?id=394327

--- Comment #1 from Aleix Pol  ---
Created attachment 112734
  --> https://bugs.kde.org/attachment.cgi?id=112734&action=edit
Proof

I've tried it and I do get a confirmation dialog. Can you please try again or
give some more information?

-- 
You are receiving this mail because:
You are watching all bug changes.