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)

History Show all quoted text
  Sun Jan 08 06:06:02 2012 VKON [...] cpan.org - Ticket created
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 .
  Sun Jan 08 06:06:18 2012 VKON [...] cpan.org - Status changed from 'new' to 'open'
  Sun Jan 08 06:07:18 2012 VKON [...] cpan.org - Correspondence added
(sorry - I accidentally pressed "open" - I did not meant this, the status should be "new")
  Sun Jan 08 06:07:35 2012 VKON [...] cpan.org - Correspondence added
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")
  Sun Jan 08 06:07:36 2012 VKON [...] cpan.org - Status changed from 'open' to 'new'
  Sun Jan 08 06:55:00 2012 cpan [...] aaroncrane.co.uk - Correspondence added
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.
  Sun Jan 08 06:55:01 2012 The RT System itself - Status changed from 'new' to 'open'
  Sun Jan 08 08:58:09 2012 VKON [...] cpan.org - Correspondence added
I confirm that fixed Devel.xs file works. Thanks!
  Sun Jan 08 14:09:11 2012 SHLOMIF [...] cpan.org - Correspondence added
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.
  Sun Jan 08 14:12:16 2012 SHLOMIF [...] cpan.org - Correspondence added
Should be resolved in XML-LibXML-1.90. Thanks for the report and the fix.
  Sun Jan 08 14:12:18 2012 SHLOMIF [...] cpan.org - Status changed from 'open' to 'resolved'
  Sun Jan 08 16:50:29 2012 vadim.konovalov [...] alcatel-lucent.com - Correspondence added
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>
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.
  Sun Jan 08 16:50:31 2012 The RT System itself - Status changed from 'resolved' to 'open'
  Mon Jan 09 01:39:08 2012 SHLOMIF [...] cpan.org - Correspondence added
Resolving again because a comment reopens the bug report. Please don't reply again.
  Mon Jan 09 01:39:09 2012 SHLOMIF [...] cpan.org - Status changed from 'open' to 'resolved'