[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Fix arandr.
From: |
Mark H Weaver |
Subject: |
Re: [PATCH] Fix arandr. |
Date: |
Sat, 27 Feb 2016 09:42:00 -0500 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.0.91 (gnu/linux) |
Ricardo Wurmus <address@hidden> writes:
> Mark H Weaver <address@hidden> writes:
>
>> Ricardo Wurmus <address@hidden> writes:
>>
>>> arandr doesn’t work as it needs “xrandr” at runtime. I tried to patch
>>> the sources to replace the path to the “xrandr” binary with a full store
>>> item path, but this wasn’t quite as pretty as I hoped as the arandr
>>> sources don’t define the path once but directly assemble command line
>>> strings in several places. Propagating “xrandr” is the cleaner solution
>>> in my opinion.
>>
>> How about making a wrapper for arandr that sets PATH to find xrandr?
>
> Patch attached:
>
> From 28f0dc229bdba3e53bd101e5acf5e93e4ff7907e Mon Sep 17 00:00:00 2001
> From: Ricardo Wurmus <address@hidden>
> Date: Sat, 27 Feb 2016 12:00:20 +0100
> Subject: [PATCH] gnu: arandr: Add "xrandr" to inputs.
>
> * gnu/packages/xdisorg.scm (arandr)[inputs]: Add "xrandr".
> [arguments]: Add build phase "make-xrandr-available" to ensure "xrandr"
> executable is found at runtime.
Looks good to me. Please push.
Thanks!
Mark