Opened 11 years ago
Closed 11 years ago
#4921 closed enhancement (fixed)
Fix xine-ui to build with readline 6.3
Reported by: | Pierre Labastie | Owned by: | |
---|---|---|---|
Priority: | normal | Milestone: | 7.6 |
Component: | BOOK | Version: | SVN |
Severity: | normal | Keywords: | |
Cc: |
Description ¶
Error (reported by Robin, on BLFS-support):
network.c: In function ‘main’: network.c:1258:39: error: ‘CPPFunction’ undeclared (first use in this function) rl_attempted_completion_function = (CPPFunction *)completion_function; ^ network.c:1258:39: note: each undeclared identifier is reported only once for each function it appears in network.c:1258:52: error: expected expression before ‘)’ token rl_attempted_completion_function = (CPPFunction *)completion_function; ^ network.c: At top level: network.c:884:15: warning: ‘completion_function’ defined but not used [-Wunused-function] static char **completion_function(const char *text, int start, int end) {
I suppose the sed to use is:
sed -i s/CPPFunction/rl_completion_func_t/ src/xitk/network.c
Change History (7)
comment:1 by , 11 years ago
comment:2 by , 11 years ago
The change is one line. A sed is better when practical. It gives better transparency in the book.
comment:3 by , 11 years ago
I suggest the change proposed by Fernando is better (with slight change in the regexp, but just a matter of taste):
sed -i 's/(CPPFunction[^)]*)//' src/xitk/network.c
BTW, sorry for not seeing that ĸen was taking care of xine-ui update.
comment:4 by , 11 years ago
I don't remember suggesting anything releated to xine and relatives. I believe ĸen did in
http://wiki.linuxfromscratch.org/blfs/ticket/4790#comment:3
:-)
comment:5 by , 11 years ago
I think I suddenly understand why I always think Fernando did something. In the message on BLFS-book, there is:
-------------------------+----------------------- Reporter: fo | Owner: ken@…
and then
Comment (by ken@…):
And somehow, I attribute it to the "Reporter". Apologies to everybody I have mistaken...
comment:6 by , 11 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
I seem to get in a mess with parentheses in sed. Yes, pierre's version works and is clearer.
I'll attend to 0.99.8 now.
Patch from upstream (against 0.99.8, but applies to 0.99.7) is already in -patches.