Re: [libsmi] libsmi release 0.2.7

Yigal> I built the new libsmi described below on NT4 using cygwin. The built Yigal> went ok. I use the following config file where I changed the path Yigal> directive to include : in front of the path.
Yigal> ================<< start quote ============================ Yigal> # Last edit: Thu Nov 09 11:39:37 2000 Yigal> # By: Yigal Hochberg Yigal> #
Yigal> # File: smi.cfg Yigal> # libsmi initialization file Yigal> #
Yigal> # path of mib modules Yigal> # you can use the posix notation: //x/<path> to specify drive x: Yigal> # e.g //c/dmhsnmp-agent/mibdir/ Yigal> # Yigal> path ://f/yigal/dmh-libsmi/libsmi-0.2.7/mibs/ietf/ Yigal> path :../mibs/ietf/ Yigal> path :../mibs/iana/ Yigal> ================== end quote >>==============================
Yigal> Unfortunately with or without the : smidump cannot find files in the Yigal> mib directories.
Yigal> Any idea what is wrong? Note that up to and including libsmi 0.2.5 Yigal> smidump works very well in this environment.
The path separator character on `#ifdef _WIN32' systems is `;' instead of `:'.
I cannot confirm the comments of your configuration file on the `posix notation'. I would expect that something like
path ;f:\yigal\dmh-libsmi\libsmi-0.2.7\mibs\ietf path ;..\mibs\ietf\ path ;..\mibs\iana\
is ok. -- !! This message is brought to you via the `libsmi' mailing list. !! Please do not reply to this message to unsubscribe. To subscribe or !! unsubscribe, send a mail message to libsmi-request@ibr.cs.tu-bs.de. !! See http://www.ibr.cs.tu-bs.de/projects/libsmi/ for more information.

Hi Frank,
Frank Strauss strauss@ibr.cs.tu-bs.de wrote:
The path separator character on `#ifdef _WIN32' systems is `;' instead of `:'.
I changed the config file to use ; instead of : This works fine.
The problem is that the different path-separator char (; vs. :) now requires different configuration file for wintel environment and unix environment.
I cannot confirm the comments of your configuration file on the `posix notation'.
For the cygwin build the posix path syntax like:
//c/dmhsnmp-agent/mibdir/
Works ok. I use it all the time in my smi config file.
I would expect that something like
path ;f:\yigal\dmh-libsmi\libsmi-0.2.7\mibs\ietf path ;..\mibs\ietf\ path ;..\mibs\iana\
is ok.
Thanks,

Hi Frank,
On a slightly different topic:
I added "-Wall -Wformat" to CFLAGS in tool/Makefile. It looks like this:
CFLAGS = -g -O2 -Wall -Wformat
The -Wformat checks the validity of printfs syntax.
I discovered several places with printf syntax problems. If you add those flags and recompile you will see those warnings/errors.
I suggest to add both flags as a standard to CFLAGS in Makefile. This will help in creating a better quality source code.
Thanks,
participants (2)
-
Frank Strauss
-
Yigal Hochberg