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

Mon Mar 28 15:16:41 PDT 2011

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

--- Comment #2 from Carol Szabo <carol.szabo at nokia.com>  2011-03-28 15:16:41 PST ---
(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.

> > 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.

> > 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.
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.

> > 2. The platform SDK:
> > Apparently it is not enough to to add its bin diretory to the VC bin directory, but it also needs to be in the PATH environment variable in cygwin.
> That sounds kind of like what's described here: <http://trac.webkit.org/wiki/BuildingOnWindows#VisualCExpressEdition>. Again, a patch to add relevant instructions to webkit.org would be much appreciated!

Actually I believe that it does not have anythign to do with the VisualCExpressEdition problem, it is rather due to an error in my instalation of Platform SDK. This is why I want to doublecheck the instructions before I propose a patch. I posted them here so that people have access to them and can issue comments like yours, or can tell me whether they worked for them or not.

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