Bug 13204 - Submit initial version of mac project files for apollo port of WebCore
Summary: Submit initial version of mac project files for apollo port of WebCore
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Platform (show other bugs)
Version: 523.x (Safari 3)
Hardware: Other OS X 10.4
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-03-27 10:04 PDT by Chris Brichford
Modified: 2007-04-26 22:56 PDT (History)
2 users (show)

See Also:


Attachments
Patch to add mac project files for apollo port of WebCore (11.66 KB, patch)
2007-03-27 10:18 PDT, Chris Brichford
timothy: review-
Details | Formatted Diff | Diff
New patch. Puts mac project files in better location. (11.39 KB, patch)
2007-03-28 11:55 PDT, Chris Brichford
timothy: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Brichford 2007-03-27 10:04:05 PDT
Bug to track submission of mac project files for apollo port of WebCore.  Project files just build derived sources at this point, so I can move them easily if they are in the wrong spot.
Comment 1 Chris Brichford 2007-03-27 10:18:45 PDT
Created attachment 13830 [details]
Patch to add mac project files for apollo port of WebCore
Comment 2 Adam Roben (:aroben) 2007-03-27 10:47:52 PDT
Comment on attachment 13830 [details]
Patch to add mac project files for apollo port of WebCore

What is the purpose of the WebCore directory beneath the mac directory?
Comment 3 Chris Brichford 2007-03-27 11:48:45 PDT
When you tell xcode to make a new project and you give it a directory to make the project in it will always create a sub-directory named the same as the project, unless the base name of the directory you choose is the same as the project name.

The JSCore project files I submitted have the same structure.
Comment 4 Timothy Hatcher 2007-03-27 16:16:47 PDT
Comment on attachment 13830 [details]
Patch to add mac project files for apollo port of WebCore

Yes, Xcode does add this extra WebCore subdirectory, but you don't need to keep that structure. Please remove the extra subdirectory. The JavaScriptCore version should not have an extra subdirectory either.
Comment 5 Timothy Hatcher 2007-03-27 16:18:49 PDT
Does the *.apolloproj file extension map to an application? Or is it just a naming convention?
Comment 6 Timothy Hatcher 2007-03-27 16:26:08 PDT
Is there any way to do this without needing a copy of the whole Xcode project file? What is different about the Apollo project file and the main project file?

It would make life easier if there was one true project file. Otherwise, the Apollo build will break if files are added or removed on the main project.
Comment 7 Chris Brichford 2007-03-27 16:57:34 PDT
apolloproj is just a naming convention ( hyatt came up with it ).

We want our own project file for the following reasons:
  1.  Apollo project builds a static lib instead of a framework.
  2.  We do not want to build all the SVG code into our static lib.
  3.  We will have different pre-processor macros turned on.
  4.  We will have different set of platform files for the apollo port.
  5.  We want to isolate our changes from the rest of webcore as much as possible so that we are unlikely to break the mac port.

Strictly speaking the xcode project could be shared, but we have a new target and probably new configurations.  So to avoid build breakage submitters would have to build those anyway.  Once they are building something new it might as well be in a seperate project file.

I don't think the vcproj could be shared on windows.
Comment 8 Chris Brichford 2007-03-27 17:27:29 PDT
I'm also ok with the project file breaking from time to time.  Our daily build will be using a copy of the code in our internal version control system ( perforce ) which we will integrate to and from the svn repository.  When we do those integrates we will fix the build if needed.

We may setup a build bot to notify people that the apollo build broke, but we don't expect everyone to keep our port's project files up to date for us.
Comment 9 Chris Brichford 2007-03-28 11:55:40 PDT
Created attachment 13847 [details]
New patch.  Puts mac project files in better location.
Comment 10 Mark Rowe (bdash) 2007-04-26 22:56:41 PDT
Landed in r21135.  Sorry for the delay.