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

[Discussion] vcpkg port #183

Closed
Nemirtingas opened this issue Sep 24, 2020 · 10 comments
Closed

[Discussion] vcpkg port #183

Nemirtingas opened this issue Sep 24, 2020 · 10 comments

Comments

@Nemirtingas
Copy link
Contributor

Nemirtingas commented Sep 24, 2020

Hi,

Can we talk about vcpkg in this issue ?

I am trying to port your libs libjuice and libdatachannel to vcpkg. I have a working set of portfiles that I now use with vcpkg and I'm setting em up for a merge into the main vcpkg branch.
microsoft/vcpkg#13703
microsoft/vcpkg#13704

1st question: Can you add an option to build the tests in the libjuice CMake file ? BUILD_TESTS to enable/disable juice-tests ?

@paullouisageneau
Copy link
Owner

Sure, this PR adds the NO_TESTS (to be coherent with libdatachannel) parameter to libjuice: paullouisageneau/libjuice#37

@Nemirtingas
Copy link
Contributor Author

libjuice looks nice with the:

Dependencies

None!

@paullouisageneau
Copy link
Owner

Yes indeed, that's pretty cool! Thanks for the suggestion.

I don't know if the pun was intended, since you can use libjuice instead of libnice 😆

@Nemirtingas
Copy link
Contributor Author

Nemirtingas commented Sep 30, 2020

And thats easier to port to vcpkg ;). Just waiting a new tag with the internal HMAC

@paullouisageneau
Copy link
Owner

@Nemirtingas
Copy link
Contributor Author

Nemirtingas commented Sep 30, 2020

Oh, ok thanks. No hurry, the vcpkg team doesn't seem to look at my changes for now.
But I'll update my port :).

@Nemirtingas
Copy link
Contributor Author

Hooray, libjuice is now supported in vcpkg.

@paullouisageneau
Copy link
Owner

Yay, great news! Thank you for handling this.

@Nemirtingas
Copy link
Contributor Author

Guess this discussion is over now. libdatachannel and libjuice has been merged to vcpkg.
We just need to keep it up to date.

@paullouisageneau
Copy link
Owner

paullouisageneau commented Nov 23, 2020

Congrats, thank you very much for this, I think it'll be useful to many people!

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

No branches or pull requests

2 participants