Release version 0.28.3

[kevinbackhouse]

Let's release version 0.28.3 so that we can ship these two bugfixes:

• Deprecate enableBMFF API (backport) #2985
• Use readOrThrow() to detect premature EOF (backport #3005) #3006

[kevinbackhouse]

Output of git shortlog -s -n v0.28.2..0.28.x:

    11  Miloš Komarčević
    10  Rosen Penev
     3  Miguel Borges de Freitas
     2  FeRD (Frank Dana)
     2  enen92
     2  norbertwg
     1  Christoph Hasse
     1  Jim Easterbrook
     1  Kevin Backhouse
     1  Leonardo Brondani Schenkel
     1  Wolfram Manthey
     1  bwvb

That's @kmilos @neheb @enen92 @ferdnyc @norbertwg @hassec @jim-easterbrook @kevinbackhouse @lbschenkel @wmanth @bertwim

[enen92]

+1
Thanks for the announcement. Will allow us to drop pretty much all patches that we have on top of the previous version.

[kevinbackhouse]

By the way, the release notes will link to this issue, so if there are any bugfixes or new features that you'd like to highlight, please do it by adding a comment here.

[kmilos]

Perhaps these are also worth mentioning:

• remove time regex from value (backport #2933) #2941
• fix(lens): canon lens detecton should be independent of locale (backport #2938) #2942
• [backport] Add EXIV2_ENABLE_FILESYSTEM_ACCESS option #2994

[kevinbackhouse]

I think everything's ready. Any objections to me publishing the release on 2024-07-08 (Monday)?

[kevinbackhouse]

What's Changed

• Restore localisation of error messages/exceptions by @jim-easterbrook in Restore localisation of error messages/exceptions #2924
• fix(lens): canon lens detecton should be independent of locale (backport fix(lens): canon lens detecton should be independent of locale #2938) by @kmilos in fix(lens): canon lens detecton should be independent of locale (backport #2938) #2942
• Fixed compile error for epsimage.cpp by @wmanth in Fixed compile error for epsimage.cpp #2946
• remove time regex from value (backport remove time regex from value #2933) by @mergify in remove time regex from value (backport #2933) #2941
• libobsd: remove (backport libobsd: remove #2934) by @mergify in libobsd: remove (backport #2934) #2947
• Update localization files by @kmilos in Update localization files #2945
• Misc makernote updates (backport Misc makernote updates #2954) by @kmilos in Misc makernote updates (backport #2954) #2956
• In src/futils.cpp Added #include (backport In src/futils.cpp Added #include <cstdint> #2968) by @mergify in In src/futils.cpp Added #include <cstdint> (backport #2968) #2969
• meson CI: update to GCC 14 (backport meson CI: update to GCC 14 #2955) by @mergify in meson CI: update to GCC 14 (backport #2955) #2972
• change print of Exif.Canon.SerialNumber (backport change print of Exif.Canon.SerialNumber #2806) by @mergify in change print of Exif.Canon.SerialNumber (backport #2806) #2971
• Set conditional HTTP depending on EXIV2_ENABLE_WEBREADY (backport Set conditional HTTP depending on EXIV2_ENABLE_WEBREADY #2844) by @mergify in Set conditional HTTP depending on EXIV2_ENABLE_WEBREADY (backport #2844) #2975
• CMake: Fix building samples without webready (backport CMake: Fix building samples without webready #2963) by @mergify in CMake: Fix building samples without webready (backport #2963) #2973
• Meson: Fix building conntest sample without curl (backport Meson: Fix building conntest sample without curl #2964) by @mergify in Meson: Fix building conntest sample without curl (backport #2964) #2974
• coverity fixes (backport coverity fixes #2696) by @mergify in coverity fixes (backport #2696) #2977
• Fix contact name encoding (backport Fix contact name encoding #2978) by @mergify in Fix contact name encoding (backport #2978) #2979
• Fix libcurl linkage on Windows (backport Fix libcurl linkage on Windows #2983) by @mergify in Fix libcurl linkage on Windows (backport #2983) #2984
• Sync translations with Crowdin by @lbschenkel in Sync translations with Crowdin #2986
• [CI] Fix unsupported MSVC version (backport [CI] Fix unsupported MSVC version #2991) by @mergify in [CI] Fix unsupported MSVC version (backport #2991) #2992
• Disable psapi in UWP (unsupported) (backport Disable psapi in UWP (unsupported) #2990) by @mergify in Disable psapi in UWP (unsupported) (backport #2990) #2993
• Always copy the JPEG segment size (backport Always copy the JPEG segment size #2989) by @mergify in Always copy the JPEG segment size (backport #2989) #2996
• [backport] Add EXIV2_ENABLE_FILESYSTEM_ACCESS option by @enen92 in [backport] Add EXIV2_ENABLE_FILESYSTEM_ACCESS option #2994
• [backport] ci: add noFilesystemAccess special build by @enen92 in [backport] ci: add noFilesystemAccess special build #2995
• meson: fix tests as subproject (backport meson: fix tests as subproject #2998) by @mergify in meson: fix tests as subproject (backport #2998) #3000
• fix clang-cl compilation (backport fix clang-cl compilation #2999) by @mergify in fix clang-cl compilation (backport #2999) #3001
• Deprecate enableBMFF API (backport) by @kmilos in Deprecate enableBMFF API (backport) #2985
• Use readOrThrow() to detect premature EOF (backport Use readOrThrow() to detect premature EOF #3005) by @mergify in Use readOrThrow() to detect premature EOF (backport #3005) #3006
• Release Exiv2 version 0.28.3 by @kevinbackhouse in Release Exiv2 version 0.28.3 #3009

Full Changelog: v0.28.2...v0.28.3

[kevinbackhouse]

https://github.com/Exiv2/exiv2/releases/tag/v0.28.3

[ferdnyc]

What's Changed

[...]

• remove time regex from value (backport remove time regex from value #2933) by @mergify in remove time regex from value (backport #2933) #2941
• libobsd: remove (backport libobsd: remove #2934) by @mergify in libobsd: remove (backport #2934) #2947
  [...]
• In src/futils.cpp Added #include (backport In src/futils.cpp Added #include  #2968) by @mergify in In src/futils.cpp Added #include (backport #2968) #2969
• meson CI: update to GCC 14 (backport meson CI: update to GCC 14 #2955) by @mergify in meson CI: update to GCC 14 (backport #2955) #2972
• change print of Exif.Canon.SerialNumber (backport change print of Exif.Canon.SerialNumber #2806) by @mergify in change print of Exif.Canon.SerialNumber (backport #2806) #2971
• Set conditional HTTP depending on EXIV2_ENABLE_WEBREADY (backport Set conditional HTTP depending on EXIV2_ENABLE_WEBREADY #2844) by @mergify in Set conditional HTTP depending on EXIV2_ENABLE_WEBREADY (backport #2844) #2975

I don't know what query/command is being used to generate this, or if there's an easy solution, but it seems like every commit from a contributor PR is being attributed to @mergify. I probably won't shock anyone when I say that none of those commits were actually contributed by the Mergify bot. 😀

Not a huge deal, but it makes the changelog attributions sort of pointless. Almost better to leave them out entirely, I'd argue, than listing @mergify for everything.

(Edit: By which I mean, leave out just the author tags, not the commits themselves.)

[kevinbackhouse]

I don't know what query/command is being used to generate this, or if there's an easy solution, but it seems like every commit from a contributor PR is being attributed to @mergify. I probably won't shock anyone when I say that none of those commits were actually contributed by the Mergify bot. 😀

Not a huge deal, but it makes the changelog attributions sort of pointless. Almost better to leave them out entirely, I'd argue, than listing @mergify for everything.

When I created the release, this was the auto-generated comment. I didn't like it much either, so I decided to paste it into a comment here instead.

A lot of the PRs on the 0.28.x branch are created by the @mergify app because we use it to backport PRs from the main branch.

[ferdnyc]

@kevinbackhouse

A lot of the PRs on the 0.28.x branch are created by the @mergify app because we use it to backport PRs from the main branch.

*nod* And I eventually realized that the changelog was listing PRs (and their owners), rather than actual commits and their authors.

Which... I can see arguments for and against, TBH.

When I created the release, this was the auto-generated comment.

Oh, auto-generated by GitHub? Well, I suppose it makes sense that they'd choose PRs as the ChangeLog granularity, at least. Shame that it clashes so severely with the Mergify process (or any process) for building release branches via backport PRs.