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