Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@Therzok
Copy link
Contributor

@Therzok Therzok commented Aug 28, 2019

The corefx code for FileSystemEventArgs has some changes to support
constructing one with a null filename.

Fixes #16486

The corefx code for FileSystemEventArgs has some changes to support
constructing one with a null filename.

Fixes mono#16486
@Therzok
Copy link
Contributor Author

Therzok commented Aug 28, 2019

We need this backported down to 2019-06 at least.

@Therzok
Copy link
Contributor Author

Therzok commented Aug 28, 2019

@steveisok @alexischr

@Therzok
Copy link
Contributor Author

Therzok commented Aug 28, 2019

@monojenkins commit csproj

@steveisok steveisok requested a review from alexischr August 28, 2019 16:37
@steveisok
Copy link
Contributor

@alexischr I added you as a reviewer. Please have a look

@steveisok
Copy link
Contributor

@monojenkins build failed

@Therzok
Copy link
Contributor Author

Therzok commented Aug 28, 2019

Based on https://dev.azure.com/devdiv/DevDiv/_workitems/edit/972836, 2019-02 is the target

@steveisok steveisok merged commit 1d23ca1 into mono:master Aug 28, 2019
@steveisok
Copy link
Contributor

@monojenkins backport 2019-08

@steveisok
Copy link
Contributor

@monojenkins backport 2019-06

@steveisok
Copy link
Contributor

@monojenkins backport 2019-02

@monojenkins
Copy link
Contributor

@steveisok backporting to 2019-02 failed, the patch results in conflicts:

Applying: Use the right FileSystemWatcher sources
Using index info to reconstruct a base tree...
M	mcs/class/System/common.sources
Falling back to patching base and 3-way merge...
Auto-merging mcs/class/System/common.sources
CONFLICT (content): Merge conflict in mcs/class/System/common.sources
error: Failed to merge in the changes.
Patch failed at 0001 Use the right FileSystemWatcher sources

Please backport manually!

@Therzok
Copy link
Contributor Author

Therzok commented Aug 29, 2019

@steveisok should I do a manual backport to 2019-02?

@akoeplinger
Copy link
Member

@Therzok yes, please do

@Therzok
Copy link
Contributor Author

Therzok commented Aug 29, 2019

#16560

UnityAlex pushed a commit to Unity-Technologies/mono that referenced this pull request Jun 2, 2022
* Use the right FileSystemWatcher sources

The corefx code for FileSystemEventArgs has some changes to support
constructing one with a null filename.

Fixes mono#16486

* Remove the mono sources for the newly replaced files

* [csproj] Update project files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FileSystemEventArgs.FullPath throws ArgumentNullException

5 participants