[pacman-dev] [PATCH 2/2] pacscripts: No need to run as root.

Karol Blazewicz karol.blazewicz at gmail.com
Mon Jun 17 17:49:56 EDT 2013


On Sun, Jun 16, 2013 at 8:04 AM, Allan McRae <allan at archlinux.org> wrote:
> On 12/06/13 07:16, Karol Błażewicz wrote:
>> Signed-off-by: Karol Błażewicz <karol.blazewicz at 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.


More information about the pacman-dev mailing list