I-Dream-in-Code / kde-arch-update-plasmoid

30 stars 9 forks source link

[Feature request] Clean orphan packages #33

Closed navarroaxel closed 5 years ago

navarroaxel commented 6 years ago

Add a check option to clean no longer required packages (orphans) after upgrade.

I-Dream-in-Code commented 6 years ago

added

master and AUR package are updated

navarroaxel commented 6 years ago

Hi @I-Dream-in-Code, when I check the clean orphan packages checkbox the clean is not added as last step of the upgrade process. But a model opens requiring my password for made sudo changes.

screenshot_20180814_223358

I'm running the upgrade using yakuake. Which runs the same command as always

$ aurman -Syu --noconfirm --noedit ;  echo  ; echo ---------------- ; echo Update Finished
I-Dream-in-Code commented 6 years ago

That's intended

I tried adding it before the echo but it failed to start the upgrade

The current implementation is when the window closes it cleans orphans

Feel free to submit a patch to run in the same command.

Q process has a really nasty bug if stringing commands together via bash

I've tried various ways in other parts of the code stringing commands in one bash call and it fails to run any code in other functions

I-Dream-in-Code commented 5 years ago

@navarroaxel I updated the package to add confirm so it actually works

Also I asked the Qt Support about the long command bug and they said to put it in a shell script.

There's a branch called bashScript feel free to work on it.

The issue is that is runs fine in shell but when running from qprocess it unexpected EOF on the ' after -c.

Try making the stuff after -c a single string.

Got command not found due to environment variable issues.

If you can get that working then your #34 can work too

I-Dream-in-Code commented 5 years ago

@navarroaxel i got it working!! It was huge thing to do.

I don't want to push it until i can get it working with yakuake but maybe @naraesk can help out?

I-Dream-in-Code commented 5 years ago

@navarroaxel I got it all working now

@naraesk I figured out yakuake

master and AUR package are both updated

navarroaxel commented 5 years ago

Works awesome, thanks for your effort!