-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
Incorrect Filename and FullPath from FileSystemWatcher under Win32 #21677
Comments
This was referenced Jul 16, 2023
klightspeed
added a commit
to EDDiscovery/EliteDangerousCore
that referenced
this issue
Jul 16, 2023
Mono running under Windows or Wine sets the `Name` in the `FileSystemEventArgs` to the absolute path of the file rather than the relative path, resulting in `FullPath` being an invalid path. Work around this by using `Name` if it's an absolute path. This has been reported as mono/mono#21677
robbyxp1
pushed a commit
to EDDiscovery/EliteDangerousCore
that referenced
this issue
Aug 7, 2023
Mono running under Windows or Wine sets the `Name` in the `FileSystemEventArgs` to the absolute path of the file rather than the relative path, resulting in `FullPath` being an invalid path. Work around this by using `Name` if it's an absolute path. This has been reported as mono/mono#21677
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When running under Windows or Wine (and likely other systems where mono does not have native file change notifications), FileSystemWatcher is returning the full path and not the relative path in the
FileSystemEventArgs.Filename
property, and that is being concatenated with the FileSystemWatcher's pathSteps to Reproduce
Current Behavior
Expected Behavior
On which platforms did you notice this
[ ] macOS
[ ] Linux
[x] Windows
[x] Wine
Version Used:
6.12.0.199
edit: 5.20.1.34 was the last version to correctly combine the watcher path with the filename into the
FullPath
; 6.0 onwards concatenates the watcher path with the filename into theFullPath
.The text was updated successfully, but these errors were encountered: