Yuri is right - just make the change in the uncommented line. The C preprocessor is pretty stupid. The comment is to explain what the seemingly magic value is, but isn't valid code... -----Burton
-----Original Message----- From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Behalf Of dsh Sent: Friday, October 17, 2008 5:34 AM To: ntop@unipi.it; [EMAIL PROTECTED] Subject: Re: [Ntop] Make Errors From Altering globals-defines.h I guess a "Duh!" is in order for that mistake. I will give that a try. Thanks. > Don't remove the '#' but just change the value in whatever you want. > Yuri On 10/17/2008 05:12 AM, dsh wrote: > Since I found that both "-c" and "-sticky-hosts" do not appear to work > for me, I thought I would change the values of some of the variables in > globals-defines.h to make ntop work the way I wanted without those > options. I started by first only uncommenting one line, line 161. That > line is: > > > > define PARM_SESSION_PURGE_MINIMUM_IDLE 10*60 > > > > I did not change the value (10*60) or make any other changes to the > file. I only removed the one single "#" symbol in front of that line. > > > > When I ran make I received the following for output. > > > > <Begin Output> > > > > make all-recursive > > make[1]: Entering directory `/home/ntop' > > Making all in . > > make[2]: Entering directory `/home/ntop' > > if /bin/sh ./libtool --tag=CC --mode=compile gcc -DHAVE_CONFIG_H -I. -I. > -I. -I. -D_REENTRANT -D_GNU_SOURCE -DTHREADS_HAVE_PIDS > -fno-strict-aliasing -pipe -Wdeclaration-after-statement -I/usr/loca > l/include -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 > -I/usr/lib/perl5/5.8.8/i486-linux-thread-multi/CORE > -I/usr/local/rrdtool/include -DLINUX -I/usr/local/include > -I/opt/local/include -g -O2 -I /usr/local/include -I/opt/local/include > -Wshadow -Wpointer-arith -Wmissing-prototypes -Wmissing-declarations > -Wnested-externs -fPIC -DPIC -MT address.lo -MD -MP -MF > ".deps/address.Tpo" -c -o addre ss.lo address.c; \ > > then mv -f ".deps/address.Tpo" ".deps/address.Plo"; else rm -f > ".deps/address.Tpo"; exit 1; fi > > gcc -DHAVE_CONFIG_H -I. -I. -I. -I. -D_REENTRANT -D_GNU_SOURCE > -DTHREADS_HAVE_PIDS -fno-strict-aliasing -pipe > -Wdeclaration-after-statement -I/usr/local/include -D_LARGEFILE_SOURCE > -D_FILE_OFFSET_ BITS=64 > -I/usr/lib/perl5/5.8.8/i486-linux-thread-multi/CORE > -I/usr/local/rrdtool/include -DLINUX -I/usr/local/include > -I/opt/local/include -g -O2 -I/usr/local/include -I/opt/local/include > -Wshadow -Wpointer-arith -Wmissing-prototypes -Wmissing-declarations > -Wnested-externs -fPIC -DPIC -MT address.lo -MD -MP -MF > .deps/address.Tpo -c address.c -fPIC -DPIC -o .libs/address.o > > In file included from ntop.h:489, > > from address.c:21: > > globals-defines.h:161: error: expected '=', ',', ';', 'asm' or > '__attribute__' before 'PARM_SESSION_PURGE_MINIMUM_IDLE' > > In file included from ntop.h:490, > > from address.c:21: > > globals-structtypes.h:113: warning: data definition has no type or > storage class > > make[2]: *** [address.lo] Error 1 > > make[2]: Leaving directory `/home/ntop' > > make[1]: *** [all-recursive] Error 1 > > make[1]: Leaving directory `/home/ntop' > > make: *** [all] Error 2 > > > > <End Output> > > > > > > I then went to ntop.h and found lines 489 and 490 commented out so I > uncommented those as well to read: > > > > include "globals-defines.h" > > include "globals-structtypes.h" > > > > As you can imagine programming is not even close to being one of my > skills, but I have a feeling that was not the thing to do as it only > produced many more errors when I ran make. > > > > Any ideas on why uncommenting a line in globals-defines.h would produce > such an error. I would have thought they were meant to be uncommented > so end users can easily make changes by changing a parameter value and > recompiling. > > Since the original unmodified program compiled fine, I would also think > I had all the correct library files etc. ?? > > > > This is the Linux 3.3.8 version of ntop running on Slackware with GNU > Make 3.81. > > > > Thanks. > > > ------------------------------------------------------------------------ > > _______________________________________________ > Ntop mailing list > Ntop@unipi.it > http://listgateway.unipi.it/mailman/listinfo/ntop -- -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-= Yuri Francalacci E-Mail: [EMAIL PROTECTED] Internet: http://www.ntop.org/ *** Esse Quam Videri -- To Be, Rather Than To Seem *** -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-= _______________________________________________ Ntop mailing list Ntop@unipi.it http://listgateway.unipi.it/mailman/listinfo/ntop _______________________________________________ Ntop mailing list Ntop@unipi.it http://listgateway.unipi.it/mailman/listinfo/ntop _______________________________________________ Ntop mailing list Ntop@unipi.it http://listgateway.unipi.it/mailman/listinfo/ntop