-
-
Notifications
You must be signed in to change notification settings - Fork 14
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
Bug on windows #117
Comments
Cannot see the change. I need to be able to reproduce it before looking at it. |
The file mentioned in coala/coala .moban.yaml refers to a symlink, and windows replaces the file contents with garbage from the symlink during the sync |
@PrajwalM2212 , what is this problem again? did you create a symlink file on windows? docs/netlify.toml link to ../../templates/docs/netlify.toml.jj2 and moban replace the target with junk. |
blocked because no way to reproduce it. |
Probably already fixed, if we have a test case which uses a symlink created on Unix, and that symlink test is active on Windows. |
According to this post: https://unix.stackexchange.com/questions/63172/does-windows-recognize-linuxs-symbolic-links, it seems Windows itself could distinguish linux symbolic link. Hence, I am not sure if os.path.islink would be able to find out this is linux symbolic link. |
please reference #251 , linux symbolic link is seen as a text file with the actual linked file inside it. hence, we would need some special utility so that moban can detect it as linux symlink on windows. |
* 🔬 let's start the test first. #117 * 🐛 fix import error * 🔬 show true failures on windows * 🐛fix samefile not available on windows. python-rope/ropevim#66 * 🐛 fix windows os path problem * 💚 make windows ci work * 💚 final touch on the windows ci failure * 💄 beautify the code using black and isort * 📰 show sub folder on windows * 🐛 jinja2 does not windows style file path * 💚 fix last failure...sigh.. * 📚 update change log
Nope, they are not resolve it. Both issues were found when I was looking at this issue. I do not think a unit test is possible for this issue either. |
What I can do is to insert a warning note in our doc: saying: Never put a symblic link in your templates if you want to use it in windows. |
moban is reported to be giving incorrect outputs in netlify.toml file on windows. https://github.com/coala/coala/pull/5881/files#diff-5db06fd2327543bbb72119cd1e5761cf
It would be great if the problem is identified and fixed.
It is replacing
The text was updated successfully, but these errors were encountered: