Added check that icu-config exists before patching

Authored by maxrd2 on Dec 7 2019, 1:16 AM.



Fixes change on windows where icu-config doesn't exist

Diff Detail

R877 Craft Blueprints for KDE
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.
maxrd2 requested review of this revision.Dec 7 2019, 1:16 AM
maxrd2 created this revision.
maxrd2 updated this revision to Diff 71040.EditedDec 7 2019, 1:37 AM

Removed duplicate code (postInstall method)

Is cmake used only on windows msvc builds?
If that's the case I can just remove postInstall from cmake and leave it under autotools which would be neater.
I suppose msys build will create icu-config and use autotools.

bcooksley accepted this revision.Dec 7 2019, 3:01 AM

Thanks for the quick fix - i'll land this and test it out.

This revision is now accepted and ready to land.Dec 7 2019, 3:01 AM
This revision was automatically updated to reflect the committed changes.