Opened 4 years ago

Closed 4 years ago

#397 closed Bug / Defect (fixed)

configure error for --enable-small vs. debug

Reported by: Bgs Owned by: cron2
Priority: minor Milestone: release 2.3.4
Component: Building / Compiling Version: OpenVPN 2.3.3 (Community Ed)
Severity: Not set (select this one, unless your'e a OpenVPN developer) Keywords: configure enable-small debug
Cc:

Description

I found a configure error while trying to compile 2.3.3 on an arm system.

Apparently --enable-small goes with default --enable-debug while they are incompatible. Yet it does not complain for this setting.

Manually running with both --enable-small and --disable-debug configures and compiles without errors.

This check should be added to configure.

Attachments (1)

0001-Conditionalize-calls-to-print_default_gateway-on-ENA.patch (1.5 KB) - added by cron2 4 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 4 years ago by Bgs

My original configure line:
./configure --disable-server --enable-plugins --disable-socks --enable-small --enable-iproute2 --enable-static

Tail of compilation:

/bin/bash ../../libtool --tag=CC --mode=link gcc -g -O2 -o openvpn base64.o buffer.o clinat.o crypto.o crypto_openssl.o crypto_polarssl.o dhcp.o error.o event.o fdmisc.o forward.o fragment.o gremlin.o helper.o httpdigest.o lladdr.o init.o interval.o list.o lzo.o manage.o mbuf.o misc.o platform.o console.o mroute.o mss.o mstats.o mtcp.o mtu.o mudp.o multi.o ntlm.o occ.o pkcs11.o pkcs11_openssl.o pkcs11_polarssl.o openvpn.o options.o otime.o packet_id.o perf.o pf.o ping.o plugin.o pool.o proto.o proxy.o ps.o push.o reliable.o route.o schedule.o session_id.o shaper.o sig.o socket.o socks.o ssl.o ssl_openssl.o ssl_polarssl.o ssl_verify.o ssl_verify_openssl.o ssl_verify_polarssl.o status.o tun.o win32.o cryptoapi.o ../../src/compat/libcompat.la -lnsl -lresolv -llzo2 -lssl -lcrypto -ldl
libtool: link: gcc -g -O2 -o openvpn base64.o buffer.o clinat.o crypto.o crypto_openssl.o crypto_polarssl.o dhcp.o error.o event.o fdmisc.o forward.o fragment.o gremlin.o helper.o httpdigest.o lladdr.o init.o interval.o list.o lzo.o manage.o mbuf.o misc.o platform.o console.o mroute.o mss.o mstats.o mtcp.o mtu.o mudp.o multi.o ntlm.o occ.o pkcs11.o pkcs11_openssl.o pkcs11_polarssl.o openvpn.o options.o otime.o packet_id.o perf.o pf.o ping.o plugin.o pool.o proto.o proxy.o ps.o push.o reliable.o route.o schedule.o session_id.o shaper.o sig.o socket.o socks.o ssl.o ssl_openssl.o ssl_polarssl.o ssl_verify.o ssl_verify_openssl.o ssl_verify_polarssl.o status.o tun.o win32.o cryptoapi.o ../../src/compat/.libs/libcompat.a -lnsl -lresolv -llzo2 -lssl -lcrypto -ldl
options.o: In function `add_option':
/home/src/openvpn/openvpn-2.3.3/src/openvpn/options.c:4138: undefined reference to `print_default_gateway'
route.o: In function `init_route_list':
/home/src/openvpn/openvpn-2.3.3/src/openvpn/route.c:599: undefined reference to `print_default_gateway'
collect2: ld returned 1 exit status
make[3]: * [openvpn] Error 1
make[3]: Leaving directory `/home/src/openvpn/openvpn-2.3.3/src/openvpn'
make[2]:
* [all-recursive] Error 1
make[2]: Leaving directory `/home/src/openvpn/openvpn-2.3.3/src'
make[1]: * [all-recursive] Error 1
make[1]: Leaving directory `/home/src/openvpn/openvpn-2.3.3'
make:
* [all] Error 2

comment:2 Changed 4 years ago by cron2

  • Milestone set to release 2.3.4
  • Owner set to cron2
  • Status changed from new to accepted

Argh, we have way too many options.

Actually this doesn't need to be handled in configure, but in the code - if the function is not compiled in, it should not be called either. A patch to that extent has been made and sent to openvpn-devel (and also attached here), waiting for an ACK.

comment:3 Changed 4 years ago by cron2

patch commited to master and release/2.3

commit c29e08a2f33234fb705a8323c0d9e1e07b0773fd (master)
commit d08a6a94e14a73b62603500b9a1a89cb9ec5cb2f (release/2.3)

2.3.4 with the patch will be released soon, quite likely "this week".

I'm closing this ticket now - if it does *not* work for you, please reopen.

comment:4 Changed 4 years ago by cron2

  • Resolution set to fixed
  • Status changed from accepted to closed
Note: See TracTickets for help on using tickets.