-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
eldev doctor check for tar on MS-Windows #80
base: master
Are you sure you want to change the base?
Conversation
023ec6d
to
1cad700
Compare
This largely reads as "Eldev sucks". I would rephrase this to mention what's wrong about it, i.e. mishandling of drive letters in absolute Windows paths. More importantly, is there no way to convince GNU tar to behave sanely? E.g. an option, an environment variable, some sort of escaping in paths? Eldev already has a shitton of workarounds everywhere, why not add another one? Alternatively, maybe we could use a non-absolute path for tarring on Windows only if possible (i.e. if on the same drive). |
Sure, this is why code reviews are so important :)
There's this Let me try to summarise the problem to put any prospective solution into perspective. Problem: It is common for MS-Windows program to have a (graphical) installer that installs programs into a directory that the user should not be concerned about. Usually, this directory's bin folder is placed at the start of the Possible options to workaround both the general and specific GNU tar issue
What are you thoughts? As I said in one of my previous comments some time ago, I'm in favor of solution#1 since this is most likely to work under every circumstance, and it is most unlikely that the user has consciously installed a tar port on their MS-Windows machine (there is high chance this was done as a side effect as part of an unrelated program's support operation). Thanks |
Thanks for the detailed analysis, now I understand the issue much better. However, I still don't feel like overriding what I suggest going with 2 + 3. Implement whatever hacks are necessary to make Eldev work even with GNU tar (3). But still add a check to the doctor (2) that would complain about GNU tar being in path: even if Eldev itself is able to cope with it now, it would potentially be problematic for general use on Windows. Also:
Tests are nice to have, but not an absolute must. In this case a test can be skipped as long as you say that it works fine on your machine. |
Hi,
could you please consider patch to add a new doctor check for the compatibility of
tar
on MS-Windows. It addresses #50.It returns the following warn when it can't use tar
I have also added a test.
Thanks