Opened 9 years ago

Closed 9 years ago

Last modified 7 years ago

#2020 closed defect (fixed)

autogen.sh spews warnings

Reported by: rransom Owned by:
Priority: Low Milestone:
Component: Core Tor/Tor Version: Tor: 0.2.2.x-final
Severity: Keywords: easy
Cc: Sebastian Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

A recent version of autoconf (at least 2.68 does this; the attached output may be from a different version) spews many warnings of the form:

configure.in:300: warning: AC_LANG_CONFTEST: no AC_LANG_SOURCE call detected in body
../../lib/autoconf/lang.m4:194: AC_LANG_CONFTEST is expanded from...
../../lib/autoconf/general.m4:2662: _AC_LINK_IFELSE is expanded from...
../../lib/autoconf/general.m4:2679: AC_LINK_IFELSE is expanded from...
../../lib/m4sugar/m4sh.m4:606: AS_IF is expanded from...
../../lib/autoconf/general.m4:2032: AC_CACHE_VAL is expanded from...
../../lib/autoconf/general.m4:2053: AC_CACHE_CHECK is expanded from...
acinclude.m4:82: TOR_SEARCH_LIBRARY is expanded from...
configure.in:300: the top level

See attachment for the full message dump.

Child Tickets

Attachments (1)

94019 (72.6 KB) - added by rransom 9 years ago.

Download all attachments as: .zip

Change History (9)

Changed 9 years ago by rransom

Attachment: 94019 added

comment:1 Changed 9 years ago by Sebastian

Status: newneeds_review

Attempted fix in branch bug2020 in my repo.

comment:2 Changed 9 years ago by nickm

Looks okay. Is this tested? Does it apply to 0.2.2 or to master? I think this should go onto 0.2.2

comment:3 Changed 9 years ago by Sebastian

It goes on top of 0.2.2, which I also think it should. I tested it on OS X 10.6 and Debian Lenny on x86; Tas tested it on OpenBSD and FreeBSD; murb tested it on Debian Squeeze/armel. Windows wasn't tested (my mingw is still broken, ungh).

comment:4 Changed 9 years ago by nickm

Resolution: fixed
Status: needs_reviewclosed

Okay, that sounds tested to me. Merging.

Closing, too. rransom: please comment if that didn't in fact fix the warnings for you.

comment:5 Changed 9 years ago by Sebastian

Resolution: fixed
Status: closedreopened

Not fixed yet unfortunately; master introduced new things in configure that I overlooked. Will fix.

comment:6 Changed 9 years ago by Sebastian

Status: reopenedneeds_review

bug2020 updated with changes. This time for master.

comment:7 Changed 9 years ago by Sebastian

Resolution: fixed
Status: needs_reviewclosed

And merged. Re-closing.

comment:8 Changed 7 years ago by nickm

Component: Tor ClientTor
Note: See TracTickets for help on using tickets.