Skip to content

Conversation

@MehdiChinoune
Copy link
Collaborator

No description provided.

The latest version supporting msvcrt runtime.
@lazka
Copy link
Member

lazka commented Sep 30, 2023

We stopped this last time because we would lose unicode path support with 0.28.0. Has anything changed there?

@MehdiChinoune
Copy link
Collaborator Author

We stopped this last time because we would lose unicode path support with 0.28.0. Has anything changed there?

I think exiv2 did the opposite It defaults to UNICODE.

@lazka
Copy link
Member

lazka commented Sep 30, 2023

How can it do that?

@MehdiChinoune
Copy link
Collaborator Author

MehdiChinoune commented Sep 30, 2023

How can it do that?

I don't know, but according to their README they only supports UCRT which supports UTF-8 by default.
And they also add -municode flags on MINGW.
Exiv2/exiv2#2090

@lazka
Copy link
Member

lazka commented Sep 30, 2023

The utf-8 locale needs to be set by the process: https://learn.microsoft.com/en-us/cpp/c-runtime-library/reference/setlocale-wsetlocale?view=msvc-170#utf-8-support.

So every application directly or indirectly linking to exiv2 will need to call setlocale when using ucrt somewhere from what I understand.

Assuming upstream doesn't fix this, I think your approach of splitting things up between ucrt and msvcrt looks good though.

@lazka
Copy link
Member

lazka commented Sep 30, 2023

@kmilos any thoughts on this?

@MehdiChinoune MehdiChinoune mentioned this pull request Oct 1, 2023
@kmilos
Copy link
Contributor

kmilos commented Oct 1, 2023

Agree w/ the approach, but I kinda held back on pushing 0.28.0 on purpose - there are a few regressions and I'd rather have upstream release 0.28.1 (w/ or w/o Unicode restored), and I don't feel like porting back so many patches. @Exiv2/exiv2

@kmilos
Copy link
Contributor

kmilos commented Oct 1, 2023

Might as well wait for qgis 3.34 including the patch, should come in October AFAICT...

@MehdiChinoune
Copy link
Collaborator Author

Might as well wait for qgis 3.34 including the patch, should come in October AFAICT...

I have already backported the patch

What regressions are you talking about?

@MehdiChinoune
Copy link
Collaborator Author

I have backported some fixes (copied from ArchLinux).

@kmilos
Copy link
Contributor

kmilos commented Oct 1, 2023

I have backported some fixes (copied from ArchLinux).

Thanks. There's a few more I'd like to get in.

Is.there a particular rush to ship 0.28.0?

@MehdiChinoune
Copy link
Collaborator Author

I have backported some fixes (copied from ArchLinux).

Thanks. There's a few more I'd like to get in.

Is.there a particular rush to ship 0.28.0?

Yes, I have spent a lot of time to backport fixes and make sure that all packages rebuild successfully.
I want to update it as soon as possible before we get any new change from other packages that disrupt the update.

@kmilos
Copy link
Contributor

kmilos commented Oct 1, 2023

Ok, go ahead then w/ the merge, I'll patch more later as it won't require rebuilds.

@MehdiChinoune MehdiChinoune merged commit 605cc6c into msys2:master Oct 1, 2023
@MehdiChinoune MehdiChinoune deleted the exiv2-update branch October 1, 2023 16:54
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