Skip to content
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

[Severe bug] Triggering "Unapply all patches" in Patchmanager2's top level pulley menu merely unapplies the first 10 Patches! #21

Closed
Olf0 opened this issue Aug 13, 2017 · 7 comments

Comments

@Olf0
Copy link
Contributor

Olf0 commented Aug 13, 2017

Observation: Triggering "Unapply all patches" in Patchmanager2's top level pulley menu merely unapplies the first 10 Patches!

NB: Hopefully Patchmanager2's built-in patch "Unapply all patches on Upgrade" behaves differently and really unapplies all Patches! (I still had Patchmanager1 installed when upgrading from SFOS 2.0.5.6 to 2.1.0.11, so I have not observed this situation, yet.) Edit: See TMO post 2276 by coderus for an answer.

Maybe this is related to issue #15 (or vice versa).

NB: To be on the safe side, there is https://openrepos.net/content/schturman/patch-unapply-all-patches-system-update-manually

Tested with Patchmanager 2.3.1-10.39.1.jolla on a Jolla 1 phone under SFOS 2.1.0.11

Originally reported on TMO as issue 12. (filing this here as promised).

@CODeRUS
Copy link
Contributor

CODeRUS commented Oct 3, 2017

@Olf0
Copy link
Contributor Author

Olf0 commented Oct 4, 2017

Well, this seems to be working now (I had the impression all active Patches were unapplied when triggering "Unapply all patches" in the pulley menu, at least many more than ten), although I cannot check at the GUI, due to patchmanager-2.4.0-10.42.1 failing to list the installed Patches (see my comment at issue #14).

But when I tried to revert to Patchmanager 2.3.3 by uninstalling Patchmanager 2.4.0 in Storeman / Warehouse, I see that a severe regression was introduced:

  • Storeman: screenshot_20171004_007
  • Warehouse: screenshot_20171004_004

So I tried doing this at the CLI: patchmanager -u (I did patchmanager -u sailfishos-browser-patch-right-handed-close-icon > pm2.4.0-u.txt 2>&1) always segfaults (i.e. stops with Segmentation fault).

Thus I checked this at the GUI and CLI with Patchmanager 2.3.3 (with and without the Patch "Right-handed tabs" applied): Working well.

Note that the output of patchmanager -u at the CLI is quite different with Patchmanager 2.4.0 (before it segfaults) from what Patchmanager 2.3.3 writes to stderr (all tests were carried out under SFOS 2.1.1.26):

  • Patchmanager 2.4.0 (no Patches applied anymore, due to testing "Unapply all patches" successfully, before discovering this regression): pm2.4.0-u.txt
  • Patchmanager 2.3.3 (please ignore the many Patches applied; IMHO the interesting fact is, that there is no output at all, besides the listing of applied Patches, in contrast to Patchmanager 2.4.0): pm2.3.3-u.txt

HTH & thanks for your efforts.

@Olf0
Copy link
Contributor Author

Olf0 commented Oct 4, 2017

After going back to Patchmanager 2.3.3 with rpm --oldpackage -U patchmanager-2.3.3-10.41.1.jolla_.armv7hl.rpm, I can see that all Patches were indeed successfully unapplied: So this works now, but aforementioned, freshly introduced regression is quite bad.

@CODeRUS
Copy link
Contributor

CODeRUS commented Oct 4, 2017

can you give me link to that patch please?

@Olf0
Copy link
Contributor Author

Olf0 commented Oct 4, 2017

https://openrepos.net/content/markkyboy/patch-right-handed-close-icon-tabs-sailfish-browser, but I have the impression that it is just the first Patch Patchmanager 2.4.0 is trying to unapply (as said, unapplying it with Patchmanager 2.3.3 works fine).

@Olf0
Copy link
Contributor Author

Olf0 commented Oct 14, 2017

If you want me to do some more debugging, I am absolutely willing to: Just tell me what to do, and I will reinstall the test-version and post the requested output here.

@Olf0
Copy link
Contributor Author

Olf0 commented Jun 14, 2018

Retested with Patchmanager 3.0.36 (beta) under SFOS 2.2.0 on a Xperia X: Works flawlessly (and quick!).

Thanks!

@Olf0 Olf0 closed this as completed Jun 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants