-
Notifications
You must be signed in to change notification settings - Fork 6.4k
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
[easyexif] create a new port #27118
[easyexif] create a new port #27118
Conversation
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.
Why not review properly?
I think it could be complemented by unofficial cmake config.
PROPERTIES | ||
CXX_STANDARD 11 | ||
PUBLIC_HEADER exif.h |
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.
Missing indentation.
PUBLIC_HEADER DESTINATION ${CMAKE_INSTALL_INCLUDEDIR} | ||
ARCHIVE DESTINATION ${CMAKE_INSTALL_LIBDIR} | ||
LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR} |
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.
Missing double qutoes.
ARCHIVE DESTINATION ${CMAKE_INSTALL_LIBDIR} | ||
LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR} | ||
) | ||
install(FILES LICENSE DESTINATION ${CMAKE_INSTALL_DATAROOTDIR}/${PROJECT_NAME}) # share/easyexif |
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.
Missing Double quotes. But actually obsolete, because it is handled in the portfile, including renaming to copyright
.
file(REMOVE_RECURSE | ||
"${CURRENT_PACKAGES_DIR}/debug/include" | ||
"${CURRENT_PACKAGES_DIR}/debug/share" | ||
"${CURRENT_PACKAGES_DIR}/share" |
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.
We never remove share
.
"version-date": "2022-10-07", | ||
"description": "Tiny ISO-compliant C++ EXIF parsing library, third-party dependency free", | ||
"homepage": "https://github.com/mayanklahiri/easyexif", | ||
"license": null, |
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.
https://tools.spdx.org/app/check_license/ says: "Close match found! The following license ID(s) match: BSD-2-Clause"
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.
Oh there was a tool! I will make another PR with this.
What does your PR fix?
Create a new port with https://github.com/mayanklahiri/easyexif.
My team started using the project recently. Opening a PR for integration.
Which triplets are supported/not supported? Have you updated the CI baseline?
Create a very small static library with CMakeLists.txt.
Every triplet must be available.
Does your PR follow the maintainer guide?
Yes.