icudt64.dll built from ICU 64.2 sources is a PE32, not PE32+

Description

We observed that ICU data binary (icudt64.dll) built for Win64 platform from ICU 64.2 source code is not a PE 32+ dll, rather it’s a PE 32 dll. On the contrary, icudt58.dll built for win64 from ICU 58.3 source code is a PE 32+ dll.

Additionally, icudt65.dll released for Win64 platform is also PE 32 dll.

Activity

Show:
Shashank Oberoi
March 11, 2020, 1:04 PM

Thanks a lot for providing me constant support and clarifying various aspects in the run-up to this contribution. It is heartening to see the PR getting merged into the master branch of ICU.

Jeff Genovy
March 9, 2020, 5:46 PM

Resolving this ticket as Fixed, now that changes have been merged to master.
(See PR https://github.com/unicode-org/icu/pull/1003).

Jeff Genovy
March 9, 2020, 5:33 PM

Thank you very kindly for all your work on this!

That is great news that you now have an organization-wide CLA in place! (I’m guessing it is for all of Adobe?).

I’ve merged the PR (#1003) into the master branch (see the commit on master), so this should be included as part of the upcoming ICU 67.1 release.

Thank you again!

Shashank Oberoi
March 6, 2020, 8:08 PM

, CLA has finally been signed for the pull request. Actually, instead of individual CLA, we wanted to sign the org-specific CLA which took some time. So, now, you may go ahead and merge this PR with the master branch.

Jeff Genovy
February 28, 2020, 2:52 AM

Hello . It looks like the automated CLA-Bot got stuck for some reason. I forced it to re-check the PR and now it should show up on the pull-request in the list of checks.

Alternatively, you can go directly to this URL as well to sign the CLA:

https://cla-assistant.io/unicode-org/icu?pullRequest=1003

Thank you again for the PR and for all your hard work on this issue!

Fixed
Your pinned fields
Click on the next to a field label to start pinning.

Assignee

Jeff Genovy

Reporter

Shashank Oberoi

Components

Reviewer

Axel Andrejs

Priority

minor

Fix versions