Skip to content
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

[libogg] Add package config module #5384

Merged
merged 1 commit into from
Feb 26, 2019

Conversation

evpobr
Copy link
Contributor

@evpobr evpobr commented Feb 16, 2019

PS D:\vcpkg> vcpkg install libogg
The following packages will be built and installed:
    libogg[core]:x64-windows
Starting package 1/1: libogg:x64-windows
Building package libogg[core]:x64-windows...
-- Using cached D:/vcpkg/downloads/xiph-ogg-v1.3.3.tar.gz
-- Using source at D:/vcpkg/buildtrees/libogg/src/v1.3.3-8dc28d69e8
-- Configuring x64-windows
-- Building x64-windows-dbg
-- Building x64-windows-rel
-- Installing: D:/vcpkg/packages/libogg_x64-windows/share/libogg/copyright
-- Performing post-build validation
-- Performing post-build validation done
Building package libogg[core]:x64-windows... done
Installing package libogg[core]:x64-windows...
Installing package libogg[core]:x64-windows... done
Elapsed time for package libogg:x64-windows: 16.34 s

Total elapsed time: 16.34 s

The package libogg:x64-windows provides CMake targets:

    find_package(Ogg CONFIG REQUIRED)
    target_link_libraries(main PRIVATE Ogg::Ogg)

@vicroms vicroms self-assigned this Feb 22, 2019
@vicroms vicroms merged commit 674ce6a into microsoft:master Feb 26, 2019
@Neumann-A
Copy link
Contributor

@evpobr was there a reason you renamed the library from ogg to Ogg? If not i probably will change it to OGG or ogg. Depending on what works nativly with VTK 8.2.0 (#5574)

@evpobr
Copy link
Contributor Author

evpobr commented Mar 9, 2019

VTK creates imported OGG::OGG target. Funny, Ogg is not abbreviation. I will ask CMake developers.

@evpobr evpobr deleted the ogg-package-config branch May 27, 2019 03:21
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.

3 participants