Dan McGee wrote:
> On Fri, Nov 6, 2009 at 2:51 PM, Cedric Staniewski <[email protected]> wrote:
>> Implements FS#13028.
>>
>> Signed-off-by: Cedric Staniewski <[email protected]>
>> ---
>> 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?

Reply via email to