-
Notifications
You must be signed in to change notification settings - Fork 7.7k
[ace] Simplified port file #11464
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
Merged
Merged
[ace] Simplified port file #11464
Changes from 28 commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
974ae7b
[ace] Update to 6.5.8
jwillemsen 0f88c1a
Address review comments
jwillemsen 411981f
Put back copy step
jwillemsen 0c31b88
Fixed error
jwillemsen aeec218
ARM should work, let us try it
jwillemsen d3d5f66
ARM doesn't work yet, so disable it again
jwillemsen 6b22b90
Merge branch 'master' of https://github.com/Microsoft/vcpkg
jwillemsen 4bc796b
Merge branch 'master' of https://github.com/Microsoft/vcpkg
jwillemsen 1162f9a
Add support for vcpkg of ace on MacOSX
jwillemsen 42ed50c
Merge branch 'master' of https://github.com/Microsoft/vcpkg
jwillemsen 751fb81
Merge branch 'master' of https://github.com/Microsoft/vcpkg
jwillemsen 0b96ccf
Merge branch 'master' of https://github.com/Microsoft/vcpkg
jwillemsen 158f002
Merge branch 'master' of https://github.com/Microsoft/vcpkg
jwillemsen 77ab4e6
Merge branch 'master' of https://github.com/Microsoft/vcpkg
jwillemsen 642a4e1
[ace] Add patch to fix Visual Studio 2019 internal compiler error
jwillemsen 0bcf649
Attempt to fix apply patches
jwillemsen 73985a0
[ace] Updated baseline for ace, 4 configurations work again with the …
jwillemsen 5f8ad22
Address review comments
jwillemsen 0ea23d3
Revised patch for ACE
jwillemsen a22f3e1
Merge branch 'master' of https://github.com/Microsoft/vcpkg
jwillemsen 506ebd5
Merge branch 'master' into jwi-ace-659patch
jwillemsen e870128
[ace] Add support for uwp
jwillemsen b264978
Merge branch 'jwi-ace-659patch'
jwillemsen 77d7c93
ace now works in all configurations
jwillemsen d583340
Removed !uwp and update version
jwillemsen 6f46b29
Simplified install of copyright file and on windows we always have a …
jwillemsen 152adfb
[ace] Simplified the port file by using an array for directories and …
jwillemsen 541f5cc
Update ports/ace/portfile.cmake
jwillemsen 9712200
[ace] resolve merge conflicts
jwillemsen dc2d034
Merge branch 'master' into jwi-ace-helpermethods
jwillemsen 61ec737
Fixed merge conflict
jwillemsen ad64f9f
Merge branch 'master' of https://github.com/Microsoft/vcpkg
jwillemsen 7618358
Merge branch 'master' into jwi-ace-helpermethods
jwillemsen File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,57 @@ | ||
| diff --git a/ace/Process_Manager.cpp b/ace/Process_Manager.cpp | ||
| index ee0418b8fca..da9d9741e28 100644 | ||
| --- a/ace/Process_Manager.cpp | ||
| +++ b/ace/Process_Manager.cpp | ||
| @@ -26,6 +26,9 @@ ACE_Process_Manager_cleanup (void *instance, void *arg) | ||
| { | ||
| ACE_Process_Manager::cleanup (instance, arg); | ||
| } | ||
| +#define ACE_PROCESS_MANAGER_CLEANUP_FUNCTION ACE_Process_Manager_cleanup | ||
| +#else | ||
| +#define ACE_PROCESS_MANAGER_CLEANUP_FUNCTION ACE_Process_Manager::cleanup | ||
| #endif | ||
|
|
||
| ACE_BEGIN_VERSIONED_NAMESPACE_DECL | ||
| @@ -124,19 +127,10 @@ ACE_Process_Manager::instance (void) | ||
| // Register with the Object_Manager so that the wrapper to | ||
| // delete the proactor will be called when Object_Manager is | ||
| // being terminated. | ||
| - | ||
| -#if defined ACE_HAS_SIG_C_FUNC | ||
| - ACE_Object_Manager::at_exit (ACE_Process_Manager::instance_, | ||
| - ACE_Process_Manager_cleanup, | ||
| - 0, | ||
| - typeid (*ACE_Process_Manager::instance_).name ()); | ||
| -#else | ||
| ACE_Object_Manager::at_exit (ACE_Process_Manager::instance_, | ||
| - ACE_Process_Manager::cleanup, | ||
| + ACE_PROCESS_MANAGER_CLEANUP_FUNCTION, | ||
| 0, | ||
| - typeid (*ACE_Process_Manager::instance_).name ()); | ||
| -#endif /* ACE_HAS_SIG_C_FUNC */ | ||
| - | ||
| + typeid (ACE_Process_Manager).name ()); | ||
| } | ||
| } | ||
|
|
||
| @@ -157,18 +151,10 @@ ACE_Process_Manager::instance (ACE_Process_Manager *tm) | ||
| // Register with the Object_Manager so that the wrapper to | ||
| // delete the proactor will be called when Object_Manager is | ||
| // being terminated. | ||
| - | ||
| -#if defined ACE_HAS_SIG_C_FUNC | ||
|
JackBoosY marked this conversation as resolved.
|
||
| - ACE_Object_Manager::at_exit (ACE_Process_Manager::instance_, | ||
| - ACE_Process_Manager_cleanup, | ||
| - 0, | ||
| - typeid (*ACE_Process_Manager::instance_).name ()); | ||
| -#else | ||
| ACE_Object_Manager::at_exit (ACE_Process_Manager::instance_, | ||
| - ACE_Process_Manager::cleanup, | ||
| + ACE_PROCESS_MANAGER_CLEANUP_FUNCTION, | ||
| 0, | ||
| - typeid (*ACE_Process_Manager::instance_).name ()); | ||
| -#endif /* ACE_HAS_SIG_C_FUNC */ | ||
| + typeid (*t).name ()); | ||
|
|
||
| ACE_Process_Manager::instance_ = tm; | ||
| return t; | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.