[pacman-dev] [PATCH 2/2] makepkg: add a config file option for the pacman binary to be used

[Cedric Staniewski]

Dan McGee wrote:
> On Fri, Nov 6, 2009 at 2:51 PM, Cedric Staniewski <cedric at gmx.ca> wrote:
>> Implements FS#13028.
>>
>> Signed-off-by: Cedric Staniewski <cedric at gmx.ca>
>> ---
>> Oops... forgot to adjust the documentation. -T support is not required
>> anymore for the pacman wrappers.
> 
> I think I'd rather see this as an environment variable- wouldn't that
> make more sense, as we wouldn't require it to be specified by the vast
> majority of people that just want to use the default?
> 
> PACMAN=${PACMAN:-pacman} or whatever it is.
> 
> -Dan

Sorry Dan, but I do not get your point. This patch makes it possible to specify a pacman command in one of the makepkg.conf files, but it is not mandatory because of the reason you mentioned. I already use your line in my patch:

+# set pacman command if not defined in config files
+PACMAN=${PACMAN:-pacman}
+

It is, however, not possible to provide a pacman command via environment variable, because I think it makes more sense to store it in a config file.
So do you mean it should be possible to use an environment variable as well?