On 30.03.2009, at 13:00, Juergen Schoenwaelder wrote:

On Mon, Mar 30, 2009 at 10:34:17AM +0200, Jens Engel wrote:

...

On 30.03.2009, at 10:20, Juergen Schoenwaelder wrote:

...

On Sat, Mar 28, 2009 at 04:43:55PM +0100, Jens Engel wrote:

...

RELEASE: libsmi 0.4.8

XML mapping of default value for fixed-sized binary OctetStrings is wrong now (or at least sometimes). NOTE: Was correct in libsmi 0.4.7.

DLSW-MIB::dlswTConnConfigSapList: DLSW-MIB::dlswIfSapList: DEFVAL { 'aa000000000000000000000000000000'H } becomes 'ff000...'H in XML format: <default>0xff000000000000000000000000000000</default>

IPSEC-SPD-MIB::spdTimeFiltDayOfMonthMask: DEFVAL { 'fffffffffffffffe'H } becomes in XML format: <default>0xffffffffffffffff</default>

I can't reproduce this problem using libsmi rev=13631 on a 32-bit Linux system. The code compiles and produces the right output. Can you check again and/or provide some more details?

/js

I will try, but I have currently no easy access to Linux system. I discovered it on MACOSX 10.5.6. I can try it with Cygwin, too. It will probably take 1 day or 2.

Revision 13631 seems to work for me on MacOS X 10.4.11 and 10.5.6.

/js

Works for me, too (FIXED on the HEAD).

I also reproduced the 0.4.8 problem under Cygwin on a PC. Therefore, I assume that problem was fixed (again) sometime after the last release.

ADDITIONAL REMARKS: 1. By the way, I had some problems to compile the current HEAD on my MAC. The problem appears with "parser-yang.y" when using GNU bison 1.4.1. When I used the MAC native bison 1.3, everything was fine. Therefore, I assume there might be a problem with this grammar part when using a newer bison version during the next release.

2. In addition, 3-4 tests of the testsuite are failing on MACOSX (3 for 0.4.7/0.4.8 and 4 for the HEAD). Under Cygwin, only 1 test failed for version 0.4.8. I will look into this issue probably next week. (I already looked at some diff files. It looks like only the line numbers are slightly off?!?)

3. MINOR ISSUE: On the current HEAD, "make distclean" does not cleanup all temporary files. I did not check older versions.

shell> svn status ? depcomp ? missing ? install-sh ? test/dumps/yang/Makefile.in ! win/config.h ? lib/parser-yang.c ? lib/parser-yang.tab.h ? lib/scanner-yang.c M lib/parser-smi.c

Ciao, Jens Engel