#6740 closed bug (fixed)
libroot doesn't build with DEBUG defined
Reported by: | anevilyak | Owned by: | zooey |
---|---|---|---|
Priority: | normal | Milestone: | R1 |
Component: | System/libroot.so | Version: | R1/Development |
Keywords: | Cc: | ||
Blocked By: | Blocking: | ||
Platform: | All |
Description
Currently one cannot build/link libroot in debug mode due to some missing posix locale-related symbols (which probably don't end up in the release build due to the nature of the macros used).
Error:
Link libroot.so posix_gnu_iconv.o: In function `add_alias': gconv_conf.c:(.text+0x57a): undefined reference to `__toupper_l' gconv_conf.c:(.text+0x61f): undefined reference to `__toupper_l' posix_gnu_iconv.o: In function `add_module': gconv_conf.c:(.text+0x911): undefined reference to `__toupper_l' gconv_conf.c:(.text+0x9b9): undefined reference to `__toupper_l' posix_gnu_wcsmbs.o: In function `__wcsmbs_load_conv': (.text+0x222c): undefined reference to `__toupper_l' collect2: ld returned 1 exit status
Attachments (1)
Change History (5)
by , 14 years ago
Attachment: | libroot-debug.patch added |
---|
comment:1 by , 14 years ago
patch: | 0 → 1 |
---|
comment:2 by , 14 years ago
Status: | new → in-progress |
---|
follow-up: 4 comment:3 by , 14 years ago
Resolution: | → fixed |
---|---|
Status: | in-progress → closed |
Fixed in hrev39380.
Kaliber, have you actually tested that patch?
I'm asking because it wasn't working for me, so I decided to drop the declarations for all those ..._l functions and replace the couple of uses with the standard functions.
comment:4 by , 14 years ago
Replying to zooey:
Kaliber, have you actually tested that patch?
I'm asking because it wasn't working for me, so I decided to drop the declarations for all those ..._l functions and replace the couple of uses with the standard functions.
My patch was a really workaround and it worked at least for gcc4.
proposed patch