On Tue, May 11, 2010 at 5:51 PM, Dave Shield <[email protected]>wrote:
> On 8 May 2010 12:40, Bart Van Assche <[email protected]> wrote: > > The Perl Counter64 code is buggy anyway, with or without patch #2890931. > > According to the strtoul() and strtoull() man pages, it is not guaranteed > > that errno is set when the conversion of a non-zero argument succeeded. > See > > also > http://www.opengroup.org/onlinepubs/000095399/functions/strtoul.html. > > As far as I can tell - that's an issue with both strtoull and strtoul. > Reverting > to the 32-bit version wouldn't change anything as far as this is concerned. > > I've applied a simple patch to revert to strtoul whem compiling under > Windows > This allows the build.bat build to succeed when perl is enabled, while > still > allowing 64-bit support on other architectures. > > I'm not sure whether this also affect Cygwin and/or MinGW builds. > (Or even whether the perl modules build at all in these environments). > > If you can let me know the status of the perl support with these builds, > we can extend the patch if necessary. Otherwise I'm looking to cut > the final final release candidates later this week. > Although I consider r18689 as an ugly change, it should be sufficient to address the strtoull() linker errors that were reported by the MSVC batch build. Please note that this change must not be applied to the trunk or to the V5.5 branches since the strtoull.c source file is being compiled and linked by the Win32 batch build for these branches. After having applied patch r18692 the Perl modules do now build with MSVC 6 and MSVC 2008. The MSVC 6 build went fine. Several crashes were reported however during the perl tests run as part of the MSVC 2008 build. These crashes have been fixed on the trunk and V5.5 branches via revisions r18375, r18369 and probably a few more commits. Bart.
------------------------------------------------------------------------------
_______________________________________________ Net-snmp-users mailing list [email protected] Please see the following page to unsubscribe or change other options: https://lists.sourceforge.net/lists/listinfo/net-snmp-users
