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

[Allan McRae]

On 18/06/13 07:49, Karol Blazewicz wrote:
> 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.
> 

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