You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Even if we don't yet make ICU 56 default as in #2917 I'd like to make it so that people who manually choose 56 won't get a failure.
A file was renamed from derb.c to derb.cpp and so currently a build issue results in this file not being properly excluded. Since it's not explicitly built, it's fine to just list both the .c and .cpp versions in the exclude list.
Activity
srl295 commentedon Sep 25, 2015
(by the way, this was caught by the continuous integration of ICU-trunk with Node-master, as discussed at the collaborator's summit.)
build: Intl: avoid 'duplicate main()' on ICU 56
srl295 commentedon Sep 25, 2015
@mhdawson fyi
srl295 commentedon Oct 7, 2015
Unfortunately this fix was incomplete. It only works for thefull-icucase. Forsmall-icuit missed some other.cto.cppchanges.Given the time, just fix the rest of the issues in #2917Looks like a false alarm / unclean local build dir. Clean build worked fine.
build: intl: avoid 'duplicate main()' on ICU 56
build: intl: avoid 'duplicate main()' on ICU 56