Add a script to restart fixes during the package upgrade #2

Closed
eSh wants to merge 3 commits from refs/pull/2/head into main
eSh commented 2022-09-17 23:36:38 +02:00 (Migrated from git.webape.site)
No description provided.
rokosun (Migrated from git.webape.site) reviewed 2022-09-18 12:35:57 +02:00
@ -12,3 +2,1 @@
find /usr/lib/systemd/user/ -name "fix-*.service" -type f -printf "%f\0" | xargs -0 systemctl --global disable
echo "Warning: A restart is required to take effect"
}
killall -r 'fix-(flatpaks-theming|theming|tweaked-desktop-files)'
rokosun (Migrated from git.webape.site) commented 2022-09-18 12:35:57 +02:00

Here it might be better to use this command instead:

killall -r 'fix-(flatpaks-theming|theming|tweaked-desktop-files)'
Here it might be better to use this command instead: ```bash killall -r 'fix-(flatpaks-theming|theming|tweaked-desktop-files)' ```
rokosun (Migrated from git.webape.site) reviewed 2022-09-18 12:52:22 +02:00
@ -15,0 +2,4 @@
killall -r 'fix-(flatpaks-theming|theming|tweaked-desktop-files)'
sudo -u "$SUDO_USER" setsid -f fix-tweaked-desktop-files
sudo -u "$SUDO_USER" setsid -f fix-theming
sudo -u "$SUDO_USER" setsid -f fix-flatpaks-theming
rokosun (Migrated from git.webape.site) commented 2022-09-18 12:52:22 +02:00

If the scripts don't work running as root then try running it like this:

sudo -u "$SUDO_USER" fix-tweaked-desktop-files &
sudo -u "$SUDO_USER" fix-theming &
sudo -u "$SUDO_USER" fix-flatpaks-theming &

Maybe it creates a sub-process under pacman and that causes some issues, so in that case its good to try this one too:

sudo -u "$SUDO_USER" setsid -f fix-tweaked-desktop-files
sudo -u "$SUDO_USER" setsid -f fix-theming
sudo -u "$SUDO_USER" setsid -f fix-flatpaks-theming
If the scripts don't work running as root then try running it like this: ```bash sudo -u "$SUDO_USER" fix-tweaked-desktop-files & sudo -u "$SUDO_USER" fix-theming & sudo -u "$SUDO_USER" fix-flatpaks-theming & ``` Maybe it creates a sub-process under pacman and that causes some issues, so in that case its good to try this one too: ```bash sudo -u "$SUDO_USER" setsid -f fix-tweaked-desktop-files sudo -u "$SUDO_USER" setsid -f fix-theming sudo -u "$SUDO_USER" setsid -f fix-flatpaks-theming ```
rokosun (Migrated from git.webape.site) approved these changes 2022-09-18 12:59:57 +02:00
rokosun commented 2022-09-18 13:00:40 +02:00 (Migrated from git.webape.site)

I'm still learning how these git features work, lol 😄

I'm still learning how these git features work, lol 😄
eSh commented 2022-09-18 13:05:29 +02:00 (Migrated from git.webape.site)

I'm still learning how these git features work, lol 😄

You're doing very good 😄

> I'm still learning how these git features work, lol 😄 You're doing very good 😄
rokosun commented 2022-09-18 13:09:15 +02:00 (Migrated from git.webape.site)

Does it work now? Or it still doesn't restart the scripts?

Does it work now? Or it still doesn't restart the scripts?
eSh commented 2022-09-18 14:36:18 +02:00 (Migrated from git.webape.site)

Does it work now? Or it still doesn't restart the scripts?

Yeah, unfortunately, it doesn't 😞 We got some sudo usage issues while testing, @tio shared an output in the TROMjaro chat.

> Does it work now? Or it still doesn't restart the scripts? Yeah, unfortunately, it doesn't 😞 We got some `sudo` usage issues while testing, @tio shared an output in the TROMjaro chat.
eSh commented 2022-09-18 17:56:22 +02:00 (Migrated from git.webape.site)

Unfortunately, this approach most probably won't work 😞 The reason is it's hard to control the scripts running on the user level from the root user or on the system level. I couldn't find any info about that so far, but we can return to this later. Closing for now.

Unfortunately, this approach most probably won't work 😞 The reason is it's hard to control the scripts running on the user level from the root user or on the system level. I couldn't find any info about that so far, but we can return to this later. Closing for now.

Pull request closed

Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: TROMjaro/fixes-package#2
No description provided.