search

LibCMaker / LibCMaker_ICU

LibCMaker build scripts for ICU library.
GNU General Public License v3.0
0 stars 0 forks source link

Land in ICU repository #1

Open donny-dont opened 5 years ago

donny-dont commented 5 years ago

Is there any plan to land this CMake port in ICU? The files would need an update to support some changes in the ICU repo since 61.1. I also have some local fixes for some Windows build issues that I can land in here if that would help expedite things.

NikitaFeodonit commented 5 years ago

Is there any plan to land this CMake port in ICU?

As noted in the README.md, ICU data building from scratch ('files' mode) is not released. Extras, samples and test building is not released too. Layoutex building is partially implemented, need to study the work with the library icu-le-hb without pkg-config, only with CMake.

Also see my comment in the ICU bug tracker.

Without the implementation of these things, these files will not be accepted in the ICU repository, as I think.

At the moment, I do not need these things and do not plan their implementation. For this reason, I do not plan to land them in the ICU. But I will be glad if someone completes them and places them in the ICU.

The files would need an update to support some changes in the ICU repo since 61.1.

In the near future I plan to add support for the new version of the ICU.

I also have some local fixes for some Windows build issues that I can land in here if that would help expedite things.

I will be glad to see your fixes. But, as noted above, I do not plan to place these files in the ICU.

NikitaFeodonit commented 5 years ago

I talked about this README.md in the ICU_CMake_Files repository.

This repository is for the LibCMaker_ICU which is used the ICU_CMake_Files as a copy.

NikitaFeodonit commented 5 years ago

I also have some local fixes for some Windows build issues that I can land in here if that would help expedite things.

Please send your fixes to the ICU_CMake_Files repository if you have the opportunity to do so.

NikitaFeodonit commented 5 years ago

The files would need an update to support some changes in the ICU repo since 61.1.

Added the support for ICU 62.1 and ICU 63.1 to ICU build files for CMake build tools.

Made some fixes in the CMake files.