Skip Menu |
 

This queue is for tickets about the XML-LibXML CPAN distribution.

Report information
The Basics
Id: 73825
Status: resolved
Priority: 0/
Queue: XML-LibXML

People
Owner: Nobody in particular
Requestors: VKON [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 1.89
Fixed in: (no value)



Download (untitled) / with headers
text/plain 4.6k
building with latest perl on Win32 I get this error: cl -c -ID:\apps\XML\libxml2\include -nologo -GF -W3 -Od -MD -Zi -DDEBUGGING -DWIN32 -D_CONSOLE -DNO_STRICT -DPERL_TEXT MODE_SCRIPTS -DUSE_PERLIO -D_USE_32BIT_TIME_T -Od -MD -Zi -DDEBUGGING -DVERSION=\"1.89\" -DXS_VERSION=\"1.89\" "-ID:\perl-t ests\perl-5.15.6-383d-nothr\lib\CORE" -DHAVE_UTF8 -DHAVE_BLANK Av_CharPtrPtr.c Av_CharPtrPtr.c cl -c -ID:\apps\XML\libxml2\include -nologo -GF -W3 -Od -MD -Zi -DDEBUGGING -DWIN32 -D_CONSOLE -DNO_STRICT -DPERL_TEXT MODE_SCRIPTS -DUSE_PERLIO -D_USE_32BIT_TIME_T -Od -MD -Zi -DDEBUGGING -DVERSION=\"1.89\" -DXS_VERSION=\"1.89\" "-ID:\perl-t ests\perl-5.15.6-383d-nothr\lib\CORE" -DHAVE_UTF8 -DHAVE_BLANK Devel.c Devel.c Devel.xs(64) : error C2275: 'xmlNode' : illegal use of this type as an expression D:\apps\XML\libxml2\include\libxml/tree.h(448) : see declaration of 'xmlNode' Devel.xs(64) : error C2065: 'node' : undeclared identifier Devel.xs(65) : error C2275: 'xmlNode' : illegal use of this type as an expression D:\apps\XML\libxml2\include\libxml/tree.h(448) : see declaration of 'xmlNode' Devel.xs(65) : error C2065: 'owner' : undeclared identifier Devel.xs(66) : warning C4047: 'function' : 'struct _xmlNode *' differs in levels of indirection from 'int ' Devel.xs(66) : warning C4024: 'PmmNodeToSv' : different types for formal and actual parameter 1 Devel.xs(66) : error C2223: left of '->_private' must point to struct/union Devel.xs(66) : error C2198: 'PmmNodeToSv' : too few actual parameters NMAKE : fatal error U1077: 'cl' : return code '0x2' Stop. 1.88 version builds fine. Summary of my perl5 (revision 5 version 15 subversion 6 patch blead 2012-01-04.19:12:48 990c89d7f48761c1d0383d6378653b6b1fe2c99c v5.15.6-383-g990c89d) configuration: Snapshot of: 990c89d7f48761c1d0383d6378653b6b1fe2c99c Platform: osname=MSWin32, osvers=5.1, archname=MSWin32-x86-perlio uname='' config_args='undef' hint=recommended, useposix=true, d_sigaction=undef useithreads=undef, usemultiplicity=undef useperlio=define, d_sfio=undef, uselargefiles=define, usesocks=undef use64bitint=undef, use64bitall=undef, uselongdouble=undef usemymalloc=y, bincompat5005=undef Compiler: cc='cl', ccflags ='-nologo -GF -W3 -Od -MD -Zi -DDEBUGGING -DWIN32 -D_CONSOLE -DNO_STRICT -DPERL_TEXTMODE_SCRIPTS -DUSE_PERLIO -D_USE_32BIT_TIME_T', optimize='-Od -MD -Zi -DDEBUGGING', cppflags='-DWIN32' ccversion='12.00.8804', gccversion='', gccosandvers='' intsize=4, longsize=4, ptrsize=4, doublesize=8, byteorder=1234 d_longlong=undef, longlongsize=8, d_longdbl=define, longdblsize=8 ivtype='long', ivsize=4, nvtype='double', nvsize=8, Off_t='__int64', lseeksize=8 alignbytes=8, prototype=define Linker and Libraries: ld='link', ldflags ='-nologo -nodefaultlib -debug -libpath:"D:\perl-tests\perl-5.15.6-383d-nothr\lib\CORE" -machine:x86' libpth=D:\MSVStudio\VC98\lib libs=oldnames.lib kernel32.lib user32.lib gdi32.lib winspool.lib comdlg32.lib advapi32.lib shell32.lib ole32.lib oleaut32.lib netapi32.lib uuid.lib ws2_32.lib mpr.lib winmm.lib version.lib odbc32.lib odbccp32.lib comctl32.lib msvcrt.lib perllibs=oldnames.lib kernel32.lib user32.lib gdi32.lib winspool.lib comdlg32.lib advapi32.lib shell32.lib ole32.lib oleaut32.lib netapi32.lib uuid.lib ws2_32.lib mpr.lib winmm.lib version.lib odbc32.lib odbccp32.lib comctl32.lib msvcrt.lib libc=msvcrt.lib, so=dll, useshrplib=true, libperl=perl515.lib gnulibc_version='' Dynamic Linking: dlsrc=dl_win32.xs, dlext=dll, d_dlsymun=undef, ccdlflags=' ' cccdlflags=' ', lddlflags='-dll -nologo -nodefaultlib -debug -libpath:"D:\perl-tests\perl-5.15.6-383d-nothr\lib\CORE" -machine:x86' Characteristics of this binary (from libperl): Compile-time options: DEBUGGING HAS_TIMES HAVE_INTERP_INTERN MYMALLOC PERLIO_LAYERS PERL_DONT_CREATE_GVSV PERL_MALLOC_WRAP PERL_PRESERVE_IVUV USE_LARGE_FILES USE_LOCALE USE_LOCALE_COLLATE USE_LOCALE_CTYPE USE_LOCALE_NUMERIC USE_PERLIO USE_PERL_ATOF Built under MSWin32 Compiled at Jan 4 2012 23:59:13 %ENV: PERL5LIB="D:\Work\PerlScripts\translations;D:\Work\PerlScripts\utl;D:\Work\PerlScripts\xml;D:\Work\PerlScripts\sgml;D:\Work\PerlScripts\pleps;D:\Work\PerlScripts\frutl;D:\Personal\perlutl" @INC: D:\Work\PerlScripts\translations D:\Work\PerlScripts\utl D:\Work\PerlScripts\xml D:\Work\PerlScripts\sgml D:\Work\PerlScripts\pleps D:\Work\PerlScripts\frutl D:\Personal\perlutl D:/perl-tests/perl-5.15.6-383d-nothr/site/lib D:/perl-tests/perl-5.15.6-383d-nothr/lib .
(sorry - I accidentally pressed "open" - I did not meant this, the status should be "new")
Download (untitled) / with headers
text/plain 137b
On Sun Jan 08 06:07:18 2012, VKON wrote: Show quoted text
> (sorry - I accidentally pressed "open" - I did not meant this, the > status should be "new")
Download (untitled) / with headers
text/plain 366b
I think this commit on this repo should fix the problem: https://bitbucket.org/aaroncrane/perl-xml-libxml/changeset/c7644f7829f2 Shlomi, assuming that change works, I'm afraid I don't know Mercurial well enough to know the most convenient way to send it to you. If there's something you'd like me to run in my repo (or on the Bitbucket site), just let me know.
I confirm that fixed Devel.xs file works. Thanks!
Download (untitled) / with headers
text/plain 980b
Hi Aaron, On Sun Jan 08 06:55:00 2012, ARC wrote: Show quoted text
> I think this commit on this repo should fix the problem: > > https://bitbucket.org/aaroncrane/perl-xml- > libxml/changeset/c7644f7829f2 >
Thanks! I applied it (after reviewing it, and running "make disttest"). Show quoted text
> Shlomi, assuming that change works, I'm afraid I don't know Mercurial > well enough to know the > most convenient way to send it to you. If there's something you'd > like me to run in my repo (or > on the Bitbucket site), just let me know.
There wasn't any need for that. I just "hg clone"d my local repository, pulled from your Bitbucket repository, reviewed and tested the changes, and ran "hg push" to push it to the main local repository where I did more changes to prepare it for release. Bitbucket also gives you "pull requests", but I'm not sure how well they work for changes requiring review and testing. Thanks! Your changes were incorporated into XML-LibXML-1.90 which was just uploaded to CPAN.
Should be resolved in XML-LibXML-1.90. Thanks for the report and the fix.
Subject: RE: [rt.cpan.org #73825] Resolved:
Date: Sun, 8 Jan 2012 22:50:16 +0100
To: "bug-XML-LibXML [...] rt.cpan.org" <bug-XML-LibXML [...] rt.cpan.org>, "VKON [...] cpan.org" <VKON [...] cpan.org>
From: "Konovalov, Vadim (Vadim)** CTR **" <vadim.konovalov [...] alcatel-lucent.com>
Download (untitled) / with headers
text/plain 572b
Show quoted text
> From: Shlomi Fish via RT > <URL: https://rt.cpan.org/Ticket/Display.html?id=73825 > > > According to our records, your request has been resolved. If > you have any > further questions or concerns, please respond to this message.
Indeed, the bug was fixed, I am happy :) For the record, the problem was in a fact that you're using more forgiving compiler - GCC - it allows for declarations anywhere, whereas this C++-isms not allowed in C. (my previous report with XML::LibXML had similar problem) Thanks for your quick help and support! Vadim.
Resolving again because a comment reopens the bug report. Please don't reply again.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.