Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Conversation

@gkhanna79
Copy link
Member

@jkotas PTAL

@jkotas
Copy link
Member

jkotas commented May 20, 2016

LGTM

@gkhanna79
Copy link
Member Author

@dotnet-bot test Windows_NT x86 legacy_backend Checked Build and Test please

@gkhanna79
Copy link
Member Author

@dotnet-bot test CentOS7.1 x64 Debug Build and Test please

@gkhanna79
Copy link
Member Author

@gkhanna79
Copy link
Member Author

@janvorli Thoughts?

@janvorli
Copy link
Member

@gkhanna79 this is a new test that @kouvel has added yesterday when checking in the cross process named mutexes stuff. He has a PR pending with a fix for some issues he has discovered later. @kouvel - is the error in the test something that you'd expect to be fixed by your pending change?

@kouvel
Copy link

kouvel commented May 20, 2016

This seems like a new issue - basic named mutex creation is failing. I suspect it's an intermittent issue, I'll take a look.

@gkhanna79
Copy link
Member Author

@kouvel Thank you. I would like to merge this PR today, so please let me know if you think this test failure would be a blocker.

@kouvel
Copy link

kouvel commented May 20, 2016

I don't think it's related to your change, so it shouldn't block your change. It may pass on rerun, this may be something special about that machine.

@kouvel
Copy link

kouvel commented May 20, 2016

I can continue to investigate this separately

@gkhanna79
Copy link
Member Author

Thank you. I will merge the PR.

@gkhanna79
Copy link
Member Author

@gkhanna79 gkhanna79 changed the title [WIP - Do not merge] Throw InvalidProgramException when VarArgs are encountered Throw InvalidProgramException when VarArgs are encountered May 20, 2016
@gkhanna79 gkhanna79 merged commit 18110e9 into dotnet:master May 20, 2016
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
Throw InvalidProgramException when VarArgs are encountered

Commit migrated from dotnet/coreclr@18110e9
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
Add same exclusions as added for other platforms in dotnet/coreclr#5124.  These were
missed because we don't have JIT32 testing on dotnet-ci.


Commit migrated from dotnet/coreclr@ac81aaf
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants