-
Notifications
You must be signed in to change notification settings - Fork 50
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
TRFP still fails for symlinked input #180
Comments
Hi @bernt-matthias, How do you get the release of TRFP? |
I install via conda. I also had troubles to reproduce. The point seems to be that the extension of the link target matters. A symlink |
I looked a bit further into it. I can confirm that mono 5.14 raises an error when processing symlinked files, although the dll is in place. I will need to investigate further, but maybe the solution is to require It looks like vendor libraries require I am afraid, there isn't much I can do about it. You can try to open an issue with https://github.com/thermofisherlsms/RawFileReader to remove this restriction. Similarly, when having a directory as an input, TRFP will only try to open the files/links with the case-insensitive |
What would happen if you feed the link (with the |
Mono 6 and Windows .NET FrameworkThe vendor library cannot process links directly (that was the initial issue #108). After the fix, TRFP checks if the provided path is a link, resolves its target (using WinAPI on Windows and Mono.Unix library on other platforms), and feeds the target path to the vendor library. Mono 5The vendor library can process symlinks directly. After the fix, Mono.Unix raises |
Just noticed that 1.4.3 still fails for input that is symlinked:
With
mono
6.12.0.90-h780b84a_0 (from conda-forge) I getIf I use a normal file it works.
And for
mono
5.14.0.177-he1b5a44_1001 I getThe text was updated successfully, but these errors were encountered: