Xymon Mailing List Archive search

Compile Issue with latest SVN version of 4.3.0

6 messages in this thread

list Iain M Conochie · Fri, 27 Nov 2009 12:23:26 +0000 ·
Hey People,

I have downloaded the latest SVN tarball and am having issues compiling xymon. The error I get is:

In file included from do_rrd.c:546:
rrd/do_net.c: In function ‘do_net_rrd’:
rrd/do_net.c:123: error: ‘plusminuts’ undeclared (first use in this function)
rrd/do_net.c:123: error: (Each undeclared identifier is reported only once
rrd/do_net.c:123: error: for each function it appears in.)
rrd/do_net.c:131: error: ‘offset’ undeclared (first use in this function)

I have rrdtoool-1.2.23-1 and I notice that this part of the function is not in the version that I am currently running. The code snippet that has the issue is:

if ( offsetdirection && ((strcmp(offsetdirection, "+") == 0) || (strcmp(offsetdirection, "-") == 0)) &&
plusminuts && (strcmp(plusminus, "+/-") == 0) &&
secs && (strcmp(secs, "secs") == 0) ) {
/* Looks sane */
sprintf(offsetval, "%s%s", offsetdirection, offset);
}
}

if (offsetval) {
sprintf(dataforntpstat, "offset=%s", offset);
do_ntpstat_rrd(hostname, testname, classname, pagepaths, dataforntpstat, tstamp);
}


Any Ideas?

If any more info needed please do not hesitate to ask

Cheers

Iain
list Malcolm Hunter · Fri, 27 Nov 2009 19:04:47 +0100 ·
quoted from Iain M Conochie
I have downloaded the latest SVN tarball and am having issues compiling xymon. The error I get is:

In file included from do_rrd.c:546:
rrd/do_net.c: In function ‘do_net_rrd’:
rrd/do_net.c:123: error: ‘plusminuts’ undeclared (first use in this function)
rrd/do_net.c:123: error: (Each undeclared identifier is reported only once
rrd/do_net.c:123: error: for each function it appears in.)
rrd/do_net.c:131: error: ‘offset’ undeclared (first use in this
function)

I have rrdtoool-1.2.23-1 and I notice that this part of the function is not in the version that I am currently running. The code snippet that has the issue is:

if ( offsetdirection && ((strcmp(offsetdirection, "+") == 0) || (strcmp(offsetdirection, "-") == 0)) &&
plusminuts && (strcmp(plusminus, "+/-") == 0) &&
secs && (strcmp(secs, "secs") == 0) ) {
/* Looks sane */
sprintf(offsetval, "%s%s", offsetdirection, offset);
}
}

if (offsetval) {
sprintf(dataforntpstat, "offset=%s", offset);
do_ntpstat_rrd(hostname, testname, classname, pagepaths, dataforntpstat, tstamp);
}


Any Ideas?
It's a typo - plusminuts should be plusminus.

Regards,

Malcolm


-- 
Technical copy-editor & proofreader

KDE Proofreading Team
KDE British English Translation Team

http://l10n.kde.org/team-infos.php?teamcode=en_GB

Facebook: http://www.facebook.com/FriendlyAtheist

Sarah Kreuz, die DSDS-Siegerin der Herzen, mit ihrem eindrucksvollen           Debütalbum "One Moment in Time". http://portal.gmx.net/de/go/musik
list Malcolm Hunter · Fri, 27 Nov 2009 19:16:53 +0100 ·
quoted from Malcolm Hunter
I have downloaded the latest SVN tarball and am having issues compiling > xymon. The error I get is:
In file included from do_rrd.c:546:
rrd/do_net.c: In function ‘do_net_rrd’:
rrd/do_net.c:123: error: ‘plusminuts’ undeclared (first use in this > function)
rrd/do_net.c:123: error: (Each undeclared identifier is reported only
once
rrd/do_net.c:123: error: for each function it appears in.)
rrd/do_net.c:131: error: ‘offset’ undeclared (first use in this
function)
I have rrdtoool-1.2.23-1 and I notice that this part of the function is > not in the version that I am currently running. The code snippet that > has the issue is:
if ( offsetdirection && ((strcmp(offsetdirection, "+") == 0) || > (strcmp(offsetdirection, "-") == 0)) &&
plusminuts && (strcmp(plusminus, "+/-") == 0) &&
secs && (strcmp(secs, "secs") == 0) ) {
/* Looks sane */
sprintf(offsetval, "%s%s", offsetdirection, offset);
}
}
if (offsetval) {
sprintf(dataforntpstat, "offset=%s", offset);
do_ntpstat_rrd(hostname, testname, classname, pagepaths, dataforntpstat,
tstamp);
}
Any Ideas?
It's a typo - plusminuts should be plusminus.
Actually the code looks a bit buggy because if the result of the first "if":

if (strstr(msgcopy, "ntpdate") != NULL) {

is true then char *offset won't be declared and could lead to a runtime error in the second "if" statement:

if (offsetval) {
quoted from Malcolm Hunter


Malcolm


-- 
Technical copy-editor & proofreader

KDE Proofreading Team
KDE British English Translation Team

http://l10n.kde.org/team-infos.php?teamcode=en_GB

Facebook: http://www.facebook.com/FriendlyAtheist

Jetzt kostenlos herunterladen: Internet Explorer 8 und Mozilla Firefox 3.5 -
sicherer, schneller und einfacher! http://portal.gmx.net/de/go/chbrowser
list Malcolm Hunter · Fri, 27 Nov 2009 19:45:10 +0100 ·
quoted from Malcolm Hunter
Any Ideas?
It's a typo - plusminuts should be plusminus.
Actually the code looks a bit buggy because if the result of the first
"if":

if (strstr(msgcopy, "ntpdate") != NULL) {

is true then char *offset won't be declared and could lead to a runtime
error in the second "if" statement:

if (offsetval) {
Correction: it still doesn't compile. I suspect "offset" on line 131 should be "offsetval".
quoted from Malcolm Hunter

Malcolm

-- 
Technical copy-editor & proofreader

KDE Proofreading Team
KDE British English Translation Team

http://l10n.kde.org/team-infos.php?teamcode=en_GB

Facebook: http://www.facebook.com/FriendlyAtheist

Jetzt kostenlos herunterladen: Internet Explorer 8 und Mozilla Firefox 3.5 -

sicherer, schneller und einfacher! http://portal.gmx.net/de/go/atbrowser
list Iain M Conochie · Tue, 01 Dec 2009 11:47:42 +0000 ·
quoted from Malcolm Hunter
On Fri, 2009-11-27 at 19:45 +0100, Malcolm Hunter wrote:
Any Ideas?
It's a typo - plusminuts should be plusminus.
Actually the code looks a bit buggy because if the result of the first
"if":
if (strstr(msgcopy, "ntpdate") != NULL) {
is true then char *offset won't be declared and could lead to a runtime
error in the second "if" statement:
if (offsetval) {
Correction: it still doesn't compile. I suspect "offset" on line 131 should be "offsetval".
Thanks Malcolm. That compiles now :)

Cheers

Iain
Malcolm
list Malcolm Hunter · Tue, 01 Dec 2009 20:15:26 +0100 ·
quoted from Iain M Conochie
Correction: it still doesn't compile. I suspect "offset" on line 131
should be "offsetval".
Thanks Malcolm. That compiles now :)
I see Henrik has fixed it in Subversion now. Looks like I suspected correctly :).
quoted from Malcolm Hunter

Malcolm


-- 
Technical copy-editor & proofreader

KDE Proofreading Team
KDE British English Translation Team

http://l10n.kde.org/team-infos.php?teamcode=en_GB

Facebook: http://www.facebook.com/FriendlyAtheist

Sarah Kreuz, die DSDS-Siegerin der Herzen, mit ihrem eindrucksvollen           
Debütalbum "One Moment in Time". http://portal.gmx.net/de/go/musik