search

cat-master21 / pacstall-gui

A gui for Pacstall (pacstall.dev)
GNU General Public License v3.0
4 stars 1 forks source link

Installing/Removing pacscripts hide password windows #6

Open Xdavius opened 11 months ago

Xdavius commented 11 months ago

The window for installing/removing block access on the password windows.

Screenshot_20231103_132155

cat-master21 commented 11 months ago

@Xdavius this issue is because pacstall-gui installs pacscripts with a slightly patched pacstall to allow it to run as root but this means that pacscripts may not install. Currently I am thinking about replacing the infos and polkit authority by opening a terminal window and running the appropriate command closing #7. Let me know what you think of this.

Xdavius commented 11 months ago

Well,

I had to go around this issue too for my yad app.

I don't know how to intercept a prompt.

For example, I have a script for secureboot which use mokutil and it ask you to input twice a 1 use password to enroll the look key.

I haven't find a solution. I have chosen opening a terminal (and you need to check what is the terminal installed on the machine because of KDE/xfce/gnome, and all WM, it's not the same), and execute the script inside.

I have seen an option in to permit editing inside a text box. I use the log function to have a loading bar so, maybe is different.

I have tried too patching pacstall to permit root but it's not a good practice. And I wasn't able to ensure the update of pacstall too.

Opening a terminal is the good solution. But it could be very good to find a way to intercept the ask of the password while pacstall run inside a subprocess.

Hope my response is not too long and comprehensible.

cat-master21 commented 11 months ago

Sorry for the long response time @Xdavius but yes I have had these issues and plan on using a terminal it is just that Pacstall does not use Polkit at all and even if, pacscripts may require the use of sudo. Maybe adding a directory to PATH with a link of sudo to Polkit may work but I would need to try it.

Xdavius commented 11 months ago

Hi @cat-master21

No hurry for replying. No fire on the lagoon.

If you need, I can perform tests too to help before merging to main on an other branch.

Ask me when you want.