[Home]

Summary:ASTERISK-11508: Configure Script does not properly test for net-snmp.
Reporter:Donny Kavanagh (donnyk)Labels:
Date Opened:2008-02-25 15:41:12.000-0600Date Closed:2008-03-26 14:31:12
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/BuildSystem
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) 12067_snmp_doc.patch
Description:The asterisk configure script does not test net-snmp properly, in my case this resulted in a compile error because lm_sensors-devel was not installed.

File had me run the following:
[root@DONNY-CENTOS ~]# net-snmp-config --agent-libs
-L/usr/lib -lnetsnmpmibs -lnetsnmpagent -lnetsnmphelpers -lnetsnmp -ldl -lrpm -lrpmio -lpopt -lz -lcrypto -lm -lsensors -L/usr/lib/lib -lwrap -Wl,-E -Wl,-rpath,/usr/lib/perl5/5.8.8/i386-linux-thread-multi/CORE -L/usr/local/lib /usr/lib/perl5/5.8.8/i386-linux-thread-multi/auto/DynaLoader/DynaLoader.a -L/usr/lib/perl5/5.8.8/i386-linux-thread-multi/CORE -lperl -lresolv -lnsl -ldl -lm -lcrypt -lutil -lpthread -lc

He thought that configure should be doing a test compile of snmp with these libs before saying it was ok, however the configure script does only:
configure:35732: checking for net-snmp-config
configure:35748: found /usr/bin/net-snmp-config
configure:35759: result: net-snmp-config

The end result of course being:
make[1]: Entering directory `/root/asterisk-2008/asterisk-NoLossCDR-Redux/res'
  [LD] res_snmp.o snmp/agent.o -> res_snmp.so
/usr/bin/ld: cannot find -lsensors
collect2: ld returned 1 exit status
make[1]: *** [res_snmp.so] Error 1
Comments:By: Russell Bryant (russell) 2008-02-25 17:48:36.000-0600

This is a broken package, really.  We do not require the sensors headers.  The library itself should be a dependency for the snmp package.
By: Donny Kavanagh (donnyk) 2008-02-25 20:36:03.000-0600

Sorry to reopen, just had additional information.

The sensors lib is a dependancy, but lm_sensors-devel is not (of net snmp)

File seemed to think you could take the output of, net-snmp-config --agent-libs and then do a test to make sure a compile & link works with those link parameters, basically to make sure the net snmp environment is sane.
By: Russell Bryant (russell) 2008-02-25 22:28:24.000-0600

what is in that package that we need?  At a quick glance, I didn't see any includes of headers from that lib ... Perhaps some of the netsnmp headers include them?  In any case, I still think this is a broken package.  But, if someone can determine what the test program should be, then I'm ok with it.

If it's the devel package that was missing, simply linking a test program against it isn't enough.  We have to include the right thing.

What was the build error you got?
By: Donny Kavanagh (donnyk) 2008-02-26 09:59:57.000-0600

As per above, the error i get is on linking res_snmp,

make[1]: Entering directory `/root/asterisk-2008/asterisk-NoLossCDR-Redux/res'
  [LD] res_snmp.o snmp/agent.o -> res_snmp.so
/usr/bin/ld: cannot find -lsensors
collect2: ld returned 1 exit status
make[1]: *** [res_snmp.so] Error 1

So, for what ever reason its linking against lm_sensors.

The package that was missing (centos) was lm_sensors-devel.  I presume this would be similar for RHEL & Fedora.


By: Digium Subversion (svnbot) 2008-03-14 12:01:14

Repository: asterisk
Revision: 108740

U   trunk/acinclude.m4
U   trunk/configure

------------------------------------------------------------------------
r108740 | russell | 2008-03-14 12:01:11 -0500 (Fri, 14 Mar 2008) | 5 lines

Do a link test in AST_EXT_TOOL_CHECK() to ensure we have all the required libs
reported by the tool.

(closes issue ASTERISK-11508, reported by Juggie, patched by me)

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=108740
By: Digium Subversion (svnbot) 2008-03-14 12:01:36

Repository: asterisk
Revision: 108741

_U  branches/1.6.0/
U   branches/1.6.0/acinclude.m4
U   branches/1.6.0/configure

------------------------------------------------------------------------
r108741 | russell | 2008-03-14 12:01:35 -0500 (Fri, 14 Mar 2008) | 13 lines

Merged revisions 108740 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
r108740 | russell | 2008-03-14 12:05:11 -0500 (Fri, 14 Mar 2008) | 5 lines

Do a link test in AST_EXT_TOOL_CHECK() to ensure we have all the required libs
reported by the tool.

(closes issue ASTERISK-11508, reported by Juggie, patched by me)

........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=108741
By: Digium Subversion (svnbot) 2008-03-26 12:20:36

Repository: asterisk
Revision: 110911

U   trunk/Makefile
U   trunk/doc/snmp.txt

------------------------------------------------------------------------
r110911 | juggie | 2008-03-26 12:20:35 -0500 (Wed, 26 Mar 2008) | 8 lines

update documentation to reflect the changes in the way configure detects net-snmp.

(closes issue ASTERISK-11508)
Reported by: juggie
Patches:
     12067_snmp_doc.patch uploaded by juggie (license 24)
Tested by: juggie

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=110911
By: Digium Subversion (svnbot) 2008-03-26 12:31:10

Repository: asterisk
Revision: 110959

_U  branches/1.6.0/
U   branches/1.6.0/doc/snmp.txt

------------------------------------------------------------------------
r110959 | juggie | 2008-03-26 12:31:09 -0500 (Wed, 26 Mar 2008) | 16 lines

Merged revisions 110911 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
r110911 | juggie | 2008-03-26 13:24:54 -0400 (Wed, 26 Mar 2008) | 8 lines

update documentation to reflect the changes in the way configure detects net-snmp.

(closes issue ASTERISK-11508)
Reported by: juggie
Patches:
     12067_snmp_doc.patch uploaded by juggie (license 24)
Tested by: juggie

........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=110959
By: Digium Subversion (svnbot) 2008-03-26 13:37:34

Repository: asterisk
Revision: 111015

_U  team/group/cdr_backend_ast_str/
U   team/group/cdr_backend_ast_str/CHANGES
U   team/group/cdr_backend_ast_str/UPGRADE.txt
U   team/group/cdr_backend_ast_str/cdr/cdr_custom.c
U   team/group/cdr_backend_ast_str/codecs/Makefile
U   team/group/cdr_backend_ast_str/codecs/codec_ilbc.c
D   team/group/cdr_backend_ast_str/codecs/ilbc/FrameClassify.c
D   team/group/cdr_backend_ast_str/codecs/ilbc/FrameClassify.h
D   team/group/cdr_backend_ast_str/codecs/ilbc/LPCdecode.c
D   team/group/cdr_backend_ast_str/codecs/ilbc/LPCdecode.h
D   team/group/cdr_backend_ast_str/codecs/ilbc/LPCencode.c
D   team/group/cdr_backend_ast_str/codecs/ilbc/LPCencode.h
D   team/group/cdr_backend_ast_str/codecs/ilbc/StateConstructW.c
D   team/group/cdr_backend_ast_str/codecs/ilbc/StateConstructW.h
D   team/group/cdr_backend_ast_str/codecs/ilbc/StateSearchW.c
D   team/group/cdr_backend_ast_str/codecs/ilbc/StateSearchW.h
D   team/group/cdr_backend_ast_str/codecs/ilbc/anaFilter.c
D   team/group/cdr_backend_ast_str/codecs/ilbc/anaFilter.h
D   team/group/cdr_backend_ast_str/codecs/ilbc/constants.c
D   team/group/cdr_backend_ast_str/codecs/ilbc/constants.h
D   team/group/cdr_backend_ast_str/codecs/ilbc/createCB.c
D   team/group/cdr_backend_ast_str/codecs/ilbc/createCB.h
D   team/group/cdr_backend_ast_str/codecs/ilbc/doCPLC.c
D   team/group/cdr_backend_ast_str/codecs/ilbc/doCPLC.h
D   team/group/cdr_backend_ast_str/codecs/ilbc/enhancer.c
D   team/group/cdr_backend_ast_str/codecs/ilbc/enhancer.h
D   team/group/cdr_backend_ast_str/codecs/ilbc/filter.c
D   team/group/cdr_backend_ast_str/codecs/ilbc/filter.h
D   team/group/cdr_backend_ast_str/codecs/ilbc/gainquant.c
D   team/group/cdr_backend_ast_str/codecs/ilbc/gainquant.h
D   team/group/cdr_backend_ast_str/codecs/ilbc/getCBvec.c
D   team/group/cdr_backend_ast_str/codecs/ilbc/getCBvec.h
D   team/group/cdr_backend_ast_str/codecs/ilbc/helpfun.c
D   team/group/cdr_backend_ast_str/codecs/ilbc/helpfun.h
D   team/group/cdr_backend_ast_str/codecs/ilbc/hpInput.c
D   team/group/cdr_backend_ast_str/codecs/ilbc/hpInput.h
D   team/group/cdr_backend_ast_str/codecs/ilbc/hpOutput.c
D   team/group/cdr_backend_ast_str/codecs/ilbc/hpOutput.h
D   team/group/cdr_backend_ast_str/codecs/ilbc/iCBConstruct.c
D   team/group/cdr_backend_ast_str/codecs/ilbc/iCBConstruct.h
D   team/group/cdr_backend_ast_str/codecs/ilbc/iCBSearch.c
D   team/group/cdr_backend_ast_str/codecs/ilbc/iCBSearch.h
D   team/group/cdr_backend_ast_str/codecs/ilbc/iLBC_decode.c
D   team/group/cdr_backend_ast_str/codecs/ilbc/iLBC_decode.h
D   team/group/cdr_backend_ast_str/codecs/ilbc/iLBC_define.h
D   team/group/cdr_backend_ast_str/codecs/ilbc/iLBC_encode.c
D   team/group/cdr_backend_ast_str/codecs/ilbc/iLBC_encode.h
D   team/group/cdr_backend_ast_str/codecs/ilbc/libilbc.vcproj
D   team/group/cdr_backend_ast_str/codecs/ilbc/lsf.c
D   team/group/cdr_backend_ast_str/codecs/ilbc/lsf.h
D   team/group/cdr_backend_ast_str/codecs/ilbc/packing.c
D   team/group/cdr_backend_ast_str/codecs/ilbc/packing.h
D   team/group/cdr_backend_ast_str/codecs/ilbc/syntFilter.c
D   team/group/cdr_backend_ast_str/codecs/ilbc/syntFilter.h
U   team/group/cdr_backend_ast_str/doc/snmp.txt
U   team/group/cdr_backend_ast_str/main/manager.c

------------------------------------------------------------------------
r111015 | seanbright | 2008-03-26 13:37:31 -0500 (Wed, 26 Mar 2008) | 80 lines

Merged revisions 110780,110831,110881,110911,110930,110963 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
r110780 | qwell | 2008-03-25 18:51:55 -0400 (Tue, 25 Mar 2008) | 14 lines

Merged revisions 110779 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r110779 | qwell | 2008-03-25 17:51:17 -0500 (Tue, 25 Mar 2008) | 6 lines

Make file access in cdr_custom similar to cdr_csv.

Fixes issue ASTERISK-11691.

Patch borrowed from r82344

........

................
r110831 | mmichelson | 2008-03-25 20:02:31 -0400 (Tue, 25 Mar 2008) | 6 lines

This ensures that the manager interface is not enabled by default. Prior to this
change, it was possible to start Asterisk with the manager interface enabled, then
either comment out the enabled option or make manager.conf unopenable and the manager
interface would still be enabled.


................
r110881 | kpfleming | 2008-03-26 13:10:28 -0400 (Wed, 26 Mar 2008) | 18 lines

Merged revisions 110880 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

................
r110880 | kpfleming | 2008-03-26 09:42:35 -0700 (Wed, 26 Mar 2008) | 10 lines

Merged revisions 110869 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.2

........
r110869 | kpfleming | 2008-03-26 08:53:46 -0700 (Wed, 26 Mar 2008) | 2 lines

due to licensing restrictions, we cannot distribute the source code for iLBC encoding and decoding... so remove it, and add instructions on how the user can obtain it themselves

........

................

................
r110911 | juggie | 2008-03-26 13:24:54 -0400 (Wed, 26 Mar 2008) | 8 lines

update documentation to reflect the changes in the way configure detects net-snmp.

(closes issue ASTERISK-11508)
Reported by: juggie
Patches:
     12067_snmp_doc.patch uploaded by juggie (license 24)
Tested by: juggie

................
r110930 | juggie | 2008-03-26 13:28:49 -0400 (Wed, 26 Mar 2008) | 1 line

revert something dumb, because i was running svn diff in a subfolder not the root of trunk, before doing my commit and did not see it
................
r110963 | kpfleming | 2008-03-26 13:44:09 -0400 (Wed, 26 Mar 2008) | 10 lines

Merged revisions 110962 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r110962 | kpfleming | 2008-03-26 12:43:02 -0500 (Wed, 26 Mar 2008) | 2 lines

add note that the user will need to enable codec_ilbc to get it to build

........

................

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=111015
By: Digium Subversion (svnbot) 2008-03-26 14:25:20

Repository: asterisk
Revision: 111082

U   team/group/cdr_backend_ast_str/channels/chan_usbradio.c
U   team/group/cdr_backend_ast_str/channels/chan_vpb.cc
U   team/group/cdr_backend_ast_str/channels/chan_zap.c

------------------------------------------------------------------------
r111082 | juggie | 2008-03-26 14:25:15 -0500 (Wed, 26 Mar 2008) | 16 lines

Merged revisions 110911 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
r110911 | juggie | 2008-03-26 13:24:54 -0400 (Wed, 26 Mar 2008) | 8 lines

update documentation to reflect the changes in the way configure detects net-snmp.

(closes issue ASTERISK-11508)
Reported by: juggie
Patches:
     12067_snmp_doc.patch uploaded by juggie (license 24)
Tested by: juggie

........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=111082
By: Digium Subversion (svnbot) 2008-03-26 14:31:12

Repository: asterisk
Revision: 111109

U   team/group/NoLossCDR-Redux2/channels/chan_usbradio.c
U   team/group/NoLossCDR-Redux2/channels/chan_vpb.cc
U   team/group/NoLossCDR-Redux2/channels/chan_zap.c

------------------------------------------------------------------------
r111109 | juggie | 2008-03-26 14:31:11 -0500 (Wed, 26 Mar 2008) | 16 lines

Merged revisions 110911 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
r110911 | juggie | 2008-03-26 13:24:54 -0400 (Wed, 26 Mar 2008) | 8 lines

update documentation to reflect the changes in the way configure detects net-snmp.

(closes issue ASTERISK-11508)
Reported by: juggie
Patches:
     12067_snmp_doc.patch uploaded by juggie (license 24)
Tested by: juggie

........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=111109