Bug 143086 – REGRESSION(181977): It broke the build on the WinCairo bot

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Csaba Osztrogonác]

Can't locate Archive/Zip.pm in @INC (@INC contains: /usr/lib/perl5/site_perl/5.14/i686-cygwin-threads-64int /usr/lib/perl5/site_perl/5.14 /usr/lib/perl5/vendor_perl/5.14/i686-cygwin-threads-64int /usr/lib/perl5/vendor_perl/5.14 /usr/lib/perl5/5.14/i686-cygwin-threads-64int /usr/lib/perl5/5.14 /usr/lib/perl5/site_perl/5.10 /usr/lib/perl5/vendor_perl/5.10 /usr/lib/perl5/site_perl/5.8 .) at /home/webkitbot/win-cairo-release/build/Tools/Scripts/update-webkit-dependency line 35.
BEGIN failed--compilation aborted at /home/webkitbot/win-cairo-release/build/Tools/Scripts/update-webkit-dependency line 35.
Died at Tools/Scripts/update-webkit-wincairo-libs line 41.
Died at ./Tools/Scripts/build-webkit line 210.

[Csaba Osztrogonác]

It seems https://trac.webkit.org/changeset/181977 is the culprit.

[Brent Fulgham]

Alex, can the WinCairo bot be updated to include the Zip Perl package?

[Alex Christensen]

I installed the zip perl package with this command:
setup-x86.exe -q -P perl-Archive-Zip

Now it fails with a different error:
Can't load '/usr/lib/perl5/5.14/i686-cygwin-threads-64int/auto/IO/IO.dll' for module IO: No such process at /usr/lib/perl5/5.14/XSLoader.pm line 71.
 at /usr/lib/perl5/5.14/i686-cygwin-threads-64int/IO.pm line 11


/usr/lib/perl5/5.14/i686-cygwin-threads-64int/auto/IO/IO.dll exists in cygwin.  What is going on?

[Alex Christensen]

Installing perl-Archive-Zip on Cygwin killed my perl installation somehow.  I installed ActivePerl, and I think with this an Myles' work we could get that working.

[Alex Christensen]

Created attachment 249549 [details]
Patch

[Alex Christensen]

This gets it to build WTF.  I see these errors after this patch.

 Creating library C:\cygwin\home\webkitbot\win-cairo-release\build\WebKitBuild\             Release_WinCairo\lib32\WTF.lib and object C:\cygwin\home\webkitbot\win-cairo-releas             e\build\WebKitBuild\Release_WinCairo\lib32\WTF.exp
  WTF.vcxproj -> C:\cygwin\home\webkitbot\win-cairo-release\build\WebKitBuild\Relea             se_WinCairo\bin32\WTF.dll
  No C:\cygwin\home\webkitbot\win-cairo-release\build\WebKitLibraries\win\tools\Ver             sionStamper\VersionStamper.exe executable. Exiting.
  /usr/bin/bash
  The system cannot find the path specified.
  Failed to build C:\cygwin\home\webkitbot\win-cairo-release\build\Source\JavaScrip             tCore\DerivedSources.make: No such file or directory at build-generated-files.pl li             ne 78.
NMAKE : fatal error U1077: 'perl' : return code '0x2' [C:\cygwin\home\webkitbot\win             -cairo-release\build\Source\JavaScriptCore\JavaScriptCore.vcxproj\JavaScriptCoreGen             erated.vcxproj]
  Stop.

[Myles C. Maxfield]

(In reply to comment #6)
> This gets it to build WTF.  I see these errors after this patch.
> 
>  Creating library
> C:\cygwin\home\webkitbot\win-cairo-release\build\WebKitBuild\            
> Release_WinCairo\lib32\WTF.lib and object
> C:\cygwin\home\webkitbot\win-cairo-releas            
> e\build\WebKitBuild\Release_WinCairo\lib32\WTF.exp
>   WTF.vcxproj ->
> C:\cygwin\home\webkitbot\win-cairo-release\build\WebKitBuild\Relea          
> se_WinCairo\bin32\WTF.dll
>   No
> C:\cygwin\home\webkitbot\win-cairo-
> release\build\WebKitLibraries\win\tools\Ver            
> sionStamper\VersionStamper.exe executable. Exiting.
>   /usr/bin/bash
>   The system cannot find the path specified.
>   Failed to build
> C:\cygwin\home\webkitbot\win-cairo-release\build\Source\JavaScrip           
> tCore\DerivedSources.make: No such file or directory at
> build-generated-files.pl li             ne 78.
> NMAKE : fatal error U1077: 'perl' : return code '0x2'
> [C:\cygwin\home\webkitbot\win            
> -cairo-release\build\Source\JavaScriptCore\JavaScriptCore.
> vcxproj\JavaScriptCoreGen             erated.vcxproj]
>   Stop.

I get the VersionStamper error too, but it seems to be benign. Note that the next error you see is from JavaScriptCore, not WTF.

[Myles C. Maxfield]

Looks fine to me but I'll wait for Brent to give the final go-ahead.

[Brent Fulgham]

Comment on attachment 249549 [details]
Patch

Looks good. I was just waiting for EWS to finish running on it.

[WebKit Commit Bot]

Comment on attachment 249549 [details]
Patch

Clearing flags on attachment: 249549

Committed r182065: <http://trac.webkit.org/changeset/182065>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Csaba Osztrogonác]

(In reply to comment #10)
> Comment on attachment 249549 [details]
> Patch
> 
> Clearing flags on attachment: 249549
> 
> Committed r182065: <http://trac.webkit.org/changeset/182065>

The build is still broken:

mkdir File::.: Invalid argument; The filename, directory name, or volume label syntax is incorrect at C:/cygwin/home/webkitbot/win-cairo-release/build/Tools/Scripts/webkitdirs.pm line 1678.

[Alex Christensen]

There are also some problems with cygwin's make using /usr/bin/perl because of headers in files like create_hash_table.  I'm taking the bot offline and I plan to fix it next week after Myles finishes patches like https://bugs.webkit.org/show_bug.cgi?id=143084

[Myles C. Maxfield]

(In reply to comment #12)
> (In reply to comment #10)
> > Comment on attachment 249549 [details]
> > Patch
> > 
> > Clearing flags on attachment: 249549
> > 
> > Committed r182065: <http://trac.webkit.org/changeset/182065>
> 
> The build is still broken:
> 
> mkdir File::.: Invalid argument; The filename, directory name, or volume
> label syntax is incorrect at
> C:/cygwin/home/webkitbot/win-cairo-release/build/Tools/Scripts/webkitdirs.pm
> line 1678.

I believe https://bugs.webkit.org/show_bug.cgi?id=143084 fixes this.