excalibur1234 / pacui

Bash script providing advanced Pacman and Yay/Pikaur/Aurman/Pakku/Trizen/Pacaur/Pamac-cli functionality in a simple UI
GNU General Public License v3.0
168 stars 13 forks source link

Making script executable #57

Closed freed00m closed 3 years ago

freed00m commented 3 years ago

Ok hear me out, I am thinking since the script is not allowed to live inside AUR, the next best thing is to get it from git straight away.

Making then a short symling somehere on PATH is easy so we could easily launch $ pacui instead of $ bash pacui

excalibur1234 commented 3 years ago

there are multiple ways of running pacui:

you are suggesting to change the file permissions of "pacui" file. sadly, i cannot see what the new file permissions are. i would suggest permission level 744. i have just pushed a commit, which does exactly that.

however, the problem probably continues, because curl, wget, etc do not copy/download a file (including file permissions), but only the content of the file. the permissions for this file are taken from umask as can be read here: https://unix.stackexchange.com/questions/82578/why-is-that-when-i-download-a-binary-from-the-web-it-doesnt-have-executable-per

this means that downloading the "pacui" file from github gives you an executable file, but downloading the "pacui" file with curl/wget does probably NOT give you an executable file.

i have not tested this behavior thoroughly, though.

do you think my latest commit solves the problem (partially) or do you recommend a different permission level?

freed00m commented 3 years ago

Yes

This was basically this commit and what I did :)

chmod +x pacui

The info is in inode and is visible in the commit patch and even on github.

Btw you can't install it from AUR, that was the whole point. Having pacui as a submodule somewhere :) It's not in Arch repository either.

Untitled