[LU-5448] Compilation without libreadline broken by LU-5418

Duplicate — Thu, 7 Aug 2014 17:37:00 +0000

The two void declarations added by ~~LU-5418~~ (http://review.whamcloud.com/11231):
static void noop_int_fn(int unused) { }
static void noop_void_fn(void) { }

Are not used if HAVE_LIBREADLINE is not defined, which results in a compilation error:

cc1: warnings being treated as errors
util/parser.c:286: error: 'noop_int_fn' defined but not used
util/parser.c:287: error: 'noop_void_fn' defined but not used
make[5]: *** [libcfsutil_a-parser.o] Error 1
make[5]: *** Waiting for unfinished jobs....
make[4]: *** [all-recursive] Error 1
make[3]: *** [all-recursive] Error 1
make[2]: *** [all-recursive] Error 1
make[1]: *** [all] Error 2
error: Bad exit status from /var/tmp/rpm-tmp.nePJql (%build)

Fix is easy, just wrap those declarations in #ifdef HAVE_LIBREADLINE.

Patch coming momentarily.

Whamcloud Community JIRA

[LU-5448] Compilation without libreadline broken by LU-5418