Summary: | Enable WML Support for Windows | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Brent Fulgham <bfulgham> | ||||||||
Component: | WebKit Misc. | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Enhancement | CC: | aroben, ddkilzer | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | PC | ||||||||||
OS: | Windows XP | ||||||||||
Attachments: |
|
Description
Brent Fulgham
2008-11-29 21:32:10 PST
Note that the Windows build already autogenerates the necessary WML support files from the *.in files, since the generic DerivedSources.make and related Perl scripts already do the heavy lifting. Consequently, this update consists of two changes: 1. Add the necessary WML files to the build. 2. Extend the preprocessor macros to include ENABLE_WML. Once this is done, some testing will be needed to confirm the new format is completely supported. Created attachment 25615 [details]
Add WML files to Windows Build
Note: For these files to actually execute the ENABLE_WML tag must be added to two places:
1. The Visual Studio preprocessor definitions in the MSVC project.
2. The build-generated-files.sh script.
Comment on attachment 25615 [details] Add WML files to Windows Build > Index: WebCore/ChangeLog > =================================================================== > --- WebCore/ChangeLog (revision 38852) > +++ WebCore/ChangeLog (working copy) > @@ -1,3 +1,11 @@ > +2008-11-30 Brent Fulgham <bfulgham@gmail.com> > + > + Reviewed by NOBODY (OOPS!). > + > + Add WML related files to Visual Studio projects. > + > + * WebCore.vcproj/WebCore.vcproj: You should mention the other changes you made to the vcproj file in the ChangeLog, since the diffs are so hard to read. > + PreprocessorDefinitions="__WIN32__;ENABLE_DOM_STORAGE;ENABLE_OFFLINE_WEB_APPLICATIONS;ENABLE_SVG;ENABLE_SVG_ANIMATION;ENABLE_SVG_AS_IMAGE;ENABLE_SVG_USE;ENABLE_SVG_FOREIGN_OBJECT;ENABLE_SVG_FONTS;WEBCORE_CONTEXT_MENUS;ENABLE_VIDEO;ENABLE_WML;ENABLE_WORKERS;ENABLE_XSLT;ENABLE_XPATH" WML support is disabled by default for all ports right now, so I don't think we should turn it on by default for Windows. The added files/include paths look fine. Created attachment 25656 [details]
Add WML files to project
Revised patch with better comments about changes. Please note that the Preprocessor Definition changes are purely from alphabetizing the list. There are no additions or omissions.
Comment on attachment 25656 [details]
Add WML files to project
r=me
Created attachment 25666 [details]
Regenerated VCPROJ against ToT to aid in landing
Reviewed previously. Diff regenerated to avoid conflicts during landing.
Committed revision 38897. |