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

AnalyzeView: Remove easyexif lib #11954

Merged
merged 1 commit into from
Oct 8, 2024

Conversation

HTRamsey
Copy link
Collaborator

@HTRamsey HTRamsey commented Sep 26, 2024

Follow up to #11906
This had been replaced by Exiv2 entirely now since easyexif only does reading.
Future work is to enable XMP compatibility which is easy for linux/mac but requires an external Expat build for Windows & Android. Use package manager like conan or consider simpler library like https://github.com/libexif/libexif.
Alternative to exiv2: https://github.com/AcademySoftwareFoundation/OpenImageIO

@HTRamsey HTRamsey force-pushed the dev-analyzeview-easyexif branch 5 times, most recently from 511f29e to 3935aec Compare September 27, 2024 17:45
@HTRamsey HTRamsey marked this pull request as draft September 27, 2024 20:30
@HTRamsey HTRamsey force-pushed the dev-analyzeview-easyexif branch from 3935aec to d24c3cf Compare October 6, 2024 01:57
@HTRamsey HTRamsey marked this pull request as ready for review October 6, 2024 02:26
@HTRamsey HTRamsey force-pushed the dev-analyzeview-easyexif branch 4 times, most recently from 0957e0a to 14933b6 Compare October 6, 2024 04:57
@HTRamsey HTRamsey requested a review from DonLakeFlyer October 6, 2024 05:00
@HTRamsey HTRamsey force-pushed the dev-analyzeview-easyexif branch from 14933b6 to ee61ea0 Compare October 6, 2024 08:06
@HTRamsey HTRamsey force-pushed the dev-analyzeview-easyexif branch from ee61ea0 to ca2cd94 Compare October 8, 2024 03:23
@HTRamsey HTRamsey merged commit 9ad5fe3 into mavlink:master Oct 8, 2024
7 checks passed
@HTRamsey HTRamsey deleted the dev-analyzeview-easyexif branch October 8, 2024 04:05
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.

1 participant