Skip to content

Export YAML_CPP_DLL define on Windows#29

Closed
Ace314159 wants to merge 1 commit intoros2:masterfrom
Ace314159:master
Closed

Export YAML_CPP_DLL define on Windows#29
Ace314159 wants to merge 1 commit intoros2:masterfrom
Ace314159:master

Conversation

@Ace314159
Copy link
Copy Markdown
Contributor

This will fix #10.

I saw that #25 will also fix #10. I agree that is the better long term solution and this can be reverted once that is merged. However, until the CI issues in the PR are fixed, I think this is the best alternative.

Signed-off-by: Akash Munagala <akash.munagala@gmail.com>
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.

[Windows] It is nice YAML_CPP_DLL is exported by default.

1 participant