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
add pkgconfig #18
add pkgconfig #18
Conversation
It's need by most of linux projects like ffmpeg
Name: libvvdec | ||
Description: Fraunhofer Versatile Video Decoder (VVdeC) | ||
Version: @PROJECT_VERSION@ | ||
Libs: -L${libdir} -lvvdec -lstdc++ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would like to leave a note that technically the user can choose the standard c++ library if they use clang, so this could be wrong, however, at the same time, there is no easy way to fix this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @1480c1,
Thanks for the suggstion.
Could you give more suggstion about how to fix this? add a comment in pkg-config?
thanks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As far as I know, there's no real way to solve this unless you are willing to parse the output of ${CXX} -v
and try to find the c++ library
Looks good to me. |
No worries, we can wait.
thanks, Christopher and Adam.
…On Tue, Dec 22, 2020 at 9:57 PM Adam Wieckowski ***@***.***> wrote:
Looks good to me.
I'd like the colleague who designed the build process to have a look,
though, which possibly will only happen after the holidays.
So sorry if it takes a while to review this.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#18 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AR75SZ7PYYSR5XDRXXCIIODSWCQWFANCNFSM4VDEAU4Q>
.
|
thank you both for helping |
Thank you for the merge request |
It's need by most of linux projects like ffmpeg