[Webkit-unassigned] [Bug 57246] Following the instructions Windows Build instructions literally does not work.

Mon Mar 28 15:48:34 PDT 2011

https://bugs.webkit.org/show_bug.cgi?id=57246

--- Comment #3 from Adam Roben (:aroben) <aroben at apple.com>  2011-03-28 15:48:34 PST ---
(In reply to comment #2)
> (In reply to comment #1)
> > (In reply to comment #0)
> > The source for that page is checked in to Websites/webkit.org/building/tools.html. Would you be willing to make a patch that adds these instructions?
> 
> I am working on that but I want to try them out on a couple of other machines before trying them out.

Great!

> > > b. Once cygwin is downloaded and installed, be sure to select the git packages for install even if not using git, it appears that there are some dependencies on those.
> > This should be remedied by updating CygwinDownloader. Its sources are checked in to Tools/CygwinDownloader. Would you be willing to make a patch?
> 
> I am not sure whether updating cygwinDownloader is the best thing. Maybe removing those dependencies is better (I would rather try to fix the dependencies first).
> In anycase fixing the source of cygwinDownloader is not enough. The instructions have a link to a zip file containing the binaries, thus the fix needs to be built and the binary needs to be posted on the site. If it is under source control, then I could do that as well if it comes to it.

The binary is checked in, as is the script to generate it: <http://trac.webkit.org/browser/trunk/Tools/CygwinDownloader/make-zip.sh>.

> > > c. The rebaseall script for cygwin does not seem to be enough, as it does not rebase libraries used by perl and python.
> > > Thus from ash do this:
> > > unset TMP
> > > unset TEMP (For whatever reason ash thought it cannot write to the windows temp directory - likely due to Windows vs. cygwin path.
> > > find /lib/perl5 -iname "*.dll">/tmp/DLLs.txt
> > > find /lib/python2.6 -iname "*.dll">>/tmp/DLLs.txt
> > > /bin/rebaseall -T /tmp/DLLs.txt
> > Our current "best practice" instructions can be found at <http://trac.webkit.org/wiki/BuildingOnWindows#ErrorlaunchingApachefromrun-webkit-testsonVista7orothererrorsthatmentionthewordremap>. I think they would have solved your problem. It would be great to have a patch that adds these instructions to the main webkit.org page. Would you be willing to do that?
> 
> As said above. I am trying to verify that I did not miss anything and that all steps are needed (maybe refine them a bit before I make a patch). I do not believe that the instructions that you linked would have fixed the patch. They appear to fix something about appache, but I still cannot see the perl5 and python2.6 directories that contain DLLs loaded by the build scripts.

The instructions end up putting all DLLs in /lib into the rebase file, which should include the perl5 and python2.6 directories you mentioned.

> Trully, I believe that a better option is to drop cygwin altogether, install Active State Perl and Python for Windows and use those if possible. The DLL fix, causes (at least as far as I understand it) huge memory fragmentation issues, as ALL listed DLLs are mapped into the process space and each is assigned a non overlapping address range, then all cygwin processes when they load any dll, it will be loaded in this address range, thus DLLs will be loaded here and there in memory, leaving gaps for the DLLs that are not needed.

Moving off Cygwin is a long-term project tracked by bug 48166.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.