[PATCH 3 of 3] extensions: also search for extension in the 'hgext3rd' package

Pierre-Yves David pierre-yves.david at ens-lyon.org
Wed Mar 16 15:19:22 EDT 2016

On 03/14/2016 07:24 AM, Ryan McElroy wrote:
> On 3/14/2016 1:49 PM, Pierre-Yves David wrote:
>> On 03/14/2016 01:41 PM, timeless wrote:
>>> It isn't useful.
>>> If we want to have a --debug which lists the search path, that's fine.
>>> But for normal "did load" cases, we shouldn't show failures to find
>>> extensions in earlier parts of the path.
>>> When I run `echo hi` with a PATH of:
>>> /usr/local/bin:/usr/bin:/usr/local/sbin:/usr/sbin
>>> I don't get:
>>>   -bash: /usr/local/bin/echo: command not found
>>> hi
>>> I just get:
>>> hi
>> That's perpendicular to my patch, there is already such output without
>> it.
> Doesn't this patch make things strictly worse by adding another place to
> look?

Not sure worth apply. This provide with useful debug information and 
I've never heard a complain about them until someone spotted them in 
this patch. So I'm not eager to consider them as a bad things and even 
less a blocker since it seems fairly out of scope here.


Pierre-Yves David

More information about the Mercurial-devel mailing list