Skip to content

CMake fixes#22

Merged
robUx4 merged 3 commits intoMatroska-Org:masterfrom
evpobr:cmake
May 19, 2019
Merged

CMake fixes#22
robUx4 merged 3 commits intoMatroska-Org:masterfrom
evpobr:cmake

Conversation

@evpobr
Copy link
Copy Markdown
Contributor

@evpobr evpobr commented May 5, 2019

  • Use Matroska:: namespace as recommended
  • Use EBML::ebml instead of ebml exported target
  • Properly create exported target (search EBML library in config script)
  • Properly set Matroska_FOUND is Matroska package is found
  • Use uniform ${CMAKE_INSTALL_PACKAGEDIR} path

Matroska-Org/libebml#44 needs to be merged first.

evpobr added 2 commits May 5, 2019 11:19
Classes derived from KaxInternalBlock are exported,  but KaxInternalBlock is not.

Fixes compiler warning.
Copy link
Copy Markdown
Contributor

@robUx4 robUx4 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@evpobr
Copy link
Copy Markdown
Contributor Author

evpobr commented May 19, 2019

Well, if you have merged Matroska-Org/libebml#44, this should be merged too. Then i will update Vcpkg ports

@robUx4 robUx4 merged commit 51648bb into Matroska-Org:master May 19, 2019
@robUx4
Copy link
Copy Markdown
Contributor

robUx4 commented May 19, 2019

Done, thanks a lot !

@evpobr
Copy link
Copy Markdown
Contributor Author

evpobr commented May 19, 2019

Vpkg ports update: microsoft/vcpkg#6526

@evpobr evpobr deleted the cmake branch May 19, 2019 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants