[pacman-dev] [PATCH 2/2] pacscripts: No need to run as root.
Signed-off-by: Karol Błażewicz <karol.blazewicz@gmail.com> --- contrib/pacscripts.sh.in | 4 ++-- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/contrib/pacscripts.sh.in b/contrib/pacscripts.sh.in index 4ecebde..2844f19 100644 --- a/contrib/pacscripts.sh.in +++ b/contrib/pacscripts.sh.in @@ -113,7 +113,7 @@ print_scriptlet() { error "Package $1 not found" return 1 fi - url=$(spacman -Sdp $1 | tail -n1) + url=$(pacman -Sdp $1 | tail -n1) filename=$(basename $url) if [ ! -f "$pac_cache/$filename" ]; then if ! spacman -Sdw --noconfirm $1 >&2; then -- 1.8.3.1
On 12/06/13 07:16, Karol Błażewicz wrote:
Signed-off-by: Karol Błażewicz <karol.blazewicz@gmail.com> --- contrib/pacscripts.sh.in | 4 ++-- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/contrib/pacscripts.sh.in b/contrib/pacscripts.sh.in index 4ecebde..2844f19 100644 --- a/contrib/pacscripts.sh.in +++ b/contrib/pacscripts.sh.in @@ -113,7 +113,7 @@ print_scriptlet() { error "Package $1 not found" return 1 fi - url=$(spacman -Sdp $1 | tail -n1) + url=$(pacman -Sdp $1 | tail -n1)
This should be a -Sddp these days and remove the tail.
filename=$(basename $url) if [ ! -f "$pac_cache/$filename" ]; then if ! spacman -Sdw --noconfirm $1 >&2; then
-Sddw here too. Allan
On Sun, Jun 16, 2013 at 8:04 AM, Allan McRae <allan@archlinux.org> wrote:
On 12/06/13 07:16, Karol Błażewicz wrote:
Signed-off-by: Karol Błażewicz <karol.blazewicz@gmail.com> --- contrib/pacscripts.sh.in | 4 ++-- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/contrib/pacscripts.sh.in b/contrib/pacscripts.sh.in index 4ecebde..2844f19 100644 --- a/contrib/pacscripts.sh.in +++ b/contrib/pacscripts.sh.in @@ -113,7 +113,7 @@ print_scriptlet() { error "Package $1 not found" return 1 fi - url=$(spacman -Sdp $1 | tail -n1) + url=$(pacman -Sdp $1 | tail -n1)
This should be a -Sddp these days and remove the tail.
filename=$(basename $url) if [ ! -f "$pac_cache/$filename" ]; then if ! spacman -Sdw --noconfirm $1 >&2; then
-Sddw here too.
Allan
I did both '-d' -> '-dd' changes in a single patch, but I'm not sure if the patch summary + commit message make sense.
On 18/06/13 07:49, Karol Blazewicz wrote:
On Sun, Jun 16, 2013 at 8:04 AM, Allan McRae <allan@archlinux.org> wrote:
On 12/06/13 07:16, Karol Błażewicz wrote:
Signed-off-by: Karol Błażewicz <karol.blazewicz@gmail.com> --- contrib/pacscripts.sh.in | 4 ++-- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/contrib/pacscripts.sh.in b/contrib/pacscripts.sh.in index 4ecebde..2844f19 100644 --- a/contrib/pacscripts.sh.in +++ b/contrib/pacscripts.sh.in @@ -113,7 +113,7 @@ print_scriptlet() { error "Package $1 not found" return 1 fi - url=$(spacman -Sdp $1 | tail -n1) + url=$(pacman -Sdp $1 | tail -n1)
This should be a -Sddp these days and remove the tail.
filename=$(basename $url) if [ ! -f "$pac_cache/$filename" ]; then if ! spacman -Sdw --noconfirm $1 >&2; then
-Sddw here too.
Allan
I did both '-d' -> '-dd' changes in a single patch, but I'm not sure if the patch summary + commit message make sense.
Thanks for the patches. I adjusted the commit message to this: [PATCH] pacscripts: Update for pacman changes Changes to pacman mean that -Sp can be called without root permissions and '-d' needs passed twice to completely ignore dependencies. Allan
participants (3)
-
Allan McRae
-
Karol Blazewicz
-
Karol Błażewicz