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

[libyuv] Add libyuv #7486

Merged
merged 2 commits into from
Sep 17, 2019
Merged

[libyuv] Add libyuv #7486

merged 2 commits into from
Sep 17, 2019

Conversation

UnaNancyOwen
Copy link
Contributor

This pull request will add libyuv.

@UnaNancyOwen UnaNancyOwen force-pushed the add_libyuv branch 3 times, most recently from de7bcac to 22a2cae Compare August 7, 2019 12:30
@ras0219-msft
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@@ -0,0 +1,4 @@
Source: libyuv
Version: fec9121
Build-Depends: libjpeg-turbo
Copy link
Contributor

Choose a reason for hiding this comment

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

You likely don't depend on the default features, please specify exactly what features you depend on.

Copy link
Contributor Author

@UnaNancyOwen UnaNancyOwen Sep 11, 2019

Choose a reason for hiding this comment

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

yuvconvert requires libjpeg (or libjpeg-turbo). And, yuv_shared needed it.

Copy link
Contributor

Choose a reason for hiding this comment

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

What I was suggesting was that you write something like this: libjpeg-turbo[core, feature1, feature2, ...], it's fine for now but we're probably going to want to change it in the future.

ports/libyuv/Findlibyuv.cmake.in Outdated Show resolved Hide resolved
ports/libyuv/vcpkg-cmake-wrapper.cmake Outdated Show resolved Hide resolved
@UnaNancyOwen
Copy link
Contributor Author

@cbezault Please re-review! Thanks,

@@ -0,0 +1,4 @@
Source: libyuv
Version: fec9121
Build-Depends: libjpeg-turbo
Copy link
Contributor

Choose a reason for hiding this comment

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

What I was suggesting was that you write something like this: libjpeg-turbo[core, feature1, feature2, ...], it's fine for now but we're probably going to want to change it in the future.

@cbezault cbezault merged commit b41a4fa into microsoft:master Sep 17, 2019
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