[valtam]

Relevant :)

[ralphy]

Jerry,

I had a good laugh at that image  ;D

Best of all, after quite some debugging I did figure out a way to make it run properly. Now I have to code my way through to give you a good working sample.

1- I need for you not to make changes to liteupdatesnotify master for now (if possible)
2- Is it a fact that you are gonna use deb's postinst to create the /etc/sudoers.d/lun file or would you prefer to create the file and assign permissions when the user launches Lite Updates Notify to make changes?

With those two things cleared up I believe we can get liteupdatesnotify to update properly in my next code refresh (still a work in progress) :P

[valtam]

1 - Will do.
2 - I'll let you make the call on that one.

[valtam]

I noticed that we didn't have a check to kill package managers in lite-tweaks-super. I have committed the check to master: https://github.com/linuxlite/litetweaks/...dfa48818e2

But in the same process, while double checking code, I noticed that we have package managers kill switch in liteupdatesnotify:

https://github.com/linuxlite/liteupdates...datenotify

Code:

...
# Kill off any package managers that may be running
if [ "$(pidof synaptic)" ]
then
   killall -9 synaptic
fi

if [ ! -z "$(pgrep gdebi-gtk)" ]
then
   killall -9 gdebi-gtk
fi
...

There is no interference between what we are executing in liteupdatesnotify and a lock from package managers that may cause the script to error out. Do we really need the kill switch there?

Ran some more tests, seems this is not needed. Feel free to delete that code in your next push.

[ralphy]

Ran some more tests, seems this is not needed. Feel free to delete that code in your next push.

Agreed.

For liteupdatenotify, we need to add an additional file: /usr/local/sbin/updatenotify-launcher

So the folder should have two files instead of just one:

updatenotify
updatenotify-launcher

updatenotify-launcher will call updatenotify and that's how we are going to be able to run apt-get update without being prompted for the sudoer password. Can you create that file on master for us please?

[valtam]

Done :)

[ralphy]

Thank you!

My turn  :D

[ralphy]

Ok Jerry, your turn next! I've committed changes to master. Feel free to abuse it :)

Note: You no longer need to create the lun file in postinst. The file is now created by the application and the correct permissions are set. In the same way, the lun file is removed when Lite Updates Notification is disabled.

Cheers!

ps: /usr/local/sbin/updatenotify & /usr/local/sbin/updatenotify-launcher should be owned by root:root with 755 permissions. If the execute bit is not set the task won't run.

[valtam]

Awesome Ralphy. I'm at work currently so I'll take a look tomorrow. Thank you.

Sent from my Phone using Tapatalk

[ralphy]

A little extra compliment