[P5P] [perl #126834] Cygwin cygdrive prefix test
Dec 7, 2015 at 6:48 pm
Tony Cook via RT writes:
Can you supply this patch as a "git format-patch" style patch?
: Just for symmetry reasons, but I have no problem with keeping it named just $prefix. Can do. Regards, Achim. -- +<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+ Samples for the Waldorf Blofeld: http://Synth.Stromeko.net/Downloads.html#BlofeldSamplesExtra
: `cygpath C:\' isn't completely fool-proof as C:\ might be mounted independently. (you decide if it's fool-proof enough for the case at hand) `readlink /proc/cygdrive' does look like the best solution. So, the best way I see is to use it and fall back to the heuristics if it's missing. "Shelling out" can't be avoided unless you wish to check the registry by hand: initially (in 2000), there was no dedicated utility to read the prefix at all, the docs pointed directly to the registry
: In which case the former code would fail just the same. If you wanted, you could loop through all 26 drive letters and see which prefix comes up the most often and pick that as the cygdrive prefix, but it could still fail in some contrived situations. Well, I can get behind that idea; thanks for mentioning. How about the attached?
: Sorry, the patch still had a change in I made to test the fallback… Here's the correct one.
Net::SNMP vs. 5.10.1 snapshot
Test failure when building with stupid prefix
Re: Change 29324: Patch by Yves Orton to fix the regression reported in :
MakeMaker PREFIX vs. PERL5LIB
Oh no, not PREFIX again!
[PATCH] ExtUtils::Installed canonicalization
[PATCH] Module/Build/t/compat.t failure
Re: Perl 5.21.10 is now available!
[perl #78502] ext/XS-APItest/t/call_checker.t fails under Cygwin
4 of 8
Dec 6, '15 at 5:58p
Dec 9, '15 at 4:29a
3 users in discussion
Achim Gratz (4)
Tony Cook via RT (3)
Ivan Pozdeev (1)
Groups & Organizations
site design / logo © 2021 Grokbase