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

Skip to content

Conversation

@Therzok
Copy link
Contributor

@Therzok Therzok commented Sep 10, 2019

Fixes #16486 - This was only partially fixed until now

This is a continuation of #16539

Should we port all sources to corefx?

Fixes mono#16486 - This was only partially fixed until now
@Therzok
Copy link
Contributor Author

Therzok commented Sep 10, 2019

@monojenkins commit csproj
@monojenkins backport 2019-08
@monojenkins backport 2019-06
@monojenkins backport 2019-02

@Therzok
Copy link
Contributor Author

Therzok commented Sep 10, 2019

Also, where can I find the test suite to try and crop up a test case for this?

@steveisok steveisok requested a review from alexischr September 10, 2019 12:00
@Therzok
Copy link
Contributor Author

Therzok commented Sep 10, 2019

@monojenkins commit csproj

@Therzok
Copy link
Contributor Author

Therzok commented Sep 10, 2019

@monojenkins backport 2019-08
@monojenkins backport 2019-06
@monojenkins backport 2019-02

@monojenkins
Copy link
Contributor

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

Applying: Add RenamedEvent* to FSW sources from CoreFX
Applying: [csproj] Update project files
Using index info to reconstruct a base tree...
M	mcs/class/System/System.csproj
Falling back to patching base and 3-way merge...
Auto-merging mcs/class/System/System.csproj
CONFLICT (content): Merge conflict in mcs/class/System/System.csproj
error: Failed to merge in the changes.
Patch failed at 0002 [csproj] Update project files

Please backport manually!

@Therzok
Copy link
Contributor Author

Therzok commented Sep 10, 2019

I don't have push rights to the mono org to be able to modify the backport PRs.

@Therzok
Copy link
Contributor Author

Therzok commented Sep 11, 2019

@monojenkins build failed

@Therzok
Copy link
Contributor Author

Therzok commented Sep 11, 2019

@mononjenkins build failed

@Therzok
Copy link
Contributor Author

Therzok commented Sep 11, 2019

cc @lewing we're going to need a bump of mono once this is merged.

@steveisok
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@marek-safar
Copy link
Member

Could you please also remove the old files

@Therzok
Copy link
Contributor Author

Therzok commented Sep 18, 2019

Sorry, didn't see the notification. Pushed a new commit.

@marek-safar marek-safar merged commit 394d8d1 into mono:master Sep 24, 2019
@Therzok Therzok deleted the dev/therzok/fsw-renamed-sources branch September 25, 2019 02:59
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