Skip to content

[mono] Add eglib_objects cmake object target for eglib#50186

Closed
lambdageek wants to merge 1 commit intodotnet:mainfrom
lambdageek:cmake-eglib-object-library
Closed

[mono] Add eglib_objects cmake object target for eglib#50186
lambdageek wants to merge 1 commit intodotnet:mainfrom
lambdageek:cmake-eglib-object-library

Conversation

@lambdageek
Copy link
Copy Markdown
Member

@lambdageek lambdageek commented Mar 24, 2021

Link it into monosgen-shared and monosgen-static.

This is needed by the "mono components" prototype #49913 so that each dynamic component could statically link its own version of eglib operations (and we will do followup work to move the ones that use global state, like the logging operations, to be in monosgen-{shared,static} only)

Link it into monosgen-shared and monosgen-static.

This is needed by the "mono components" prototype
dotnet#49913 so that each dynamic component
could statically link its own version of eglib operations (and we will do followup work
to move the ones that use global state, like the logging operations, to be in
monosgen-{shared,static} only)
@lambdageek lambdageek force-pushed the cmake-eglib-object-library branch from f7b4c40 to d19b86e Compare March 24, 2021 17:16
@lambdageek
Copy link
Copy Markdown
Member Author

/cc @lateralusX

@lambdageek
Copy link
Copy Markdown
Member Author

I think the main risk here is if someone sets properties or cflags on monosgen-shared or monosgen-static, we won't necessarily get them on eglib_objects which may lead to difficult bugs.

@lambdageek
Copy link
Copy Markdown
Member Author

Subsumed by #50837

@lambdageek lambdageek closed this Apr 9, 2021
@ghost ghost locked as resolved and limited conversation to collaborators May 9, 2021
@karelz karelz added this to the 6.0.0 milestone May 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants