diff options
author | commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2014-04-05 01:13:43 +0000 |
---|---|---|
committer | commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2014-04-05 01:13:43 +0000 |
commit | ae0054e48ac65caf0e808e254dfdff6ac945fa2e (patch) | |
tree | c386ced1ad74a0b03f674ad721523a4a822c1617 | |
parent | 78842e4ad2c9af860690907cb3a8c07f9f5be24e (diff) |
get iOS building again
1) Disable SampleApp. Seems like it's totally horked? SkOSFile_iOS.mm is missing about half the functions needed, and SkOSFile_stdio.cpp is double-providing the others.
2) Drop armv6.
3) Switch from putting headers in sources to putting the corresponding directories in includes.
4) Force cast the type of glShaderSource. Something to do with GR_GL_USE_NEW_SHADER_SOURCE_SIGNATURE?
After all this,
env CC=clang CXX=clang++ GYP_DEFINES=skia_os=ios make
builds for me.
BUG=skia:2363
R=bsalomon@google.com, epoger@google.com, mtklein@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/226413005
git-svn-id: http://skia.googlecode.com/svn/trunk@14069 2bbb7eff-a529-9590-31e7-b0007b416f81
-rw-r--r-- | gyp/SampleApp.gyp | 14 | ||||
-rw-r--r-- | gyp/SimpleiOSApp.gyp | 6 | ||||
-rw-r--r-- | gyp/apptype_console.gypi | 6 | ||||
-rw-r--r-- | gyp/common_conditions.gypi | 2 | ||||
-rw-r--r-- | gyp/most.gyp | 3 | ||||
-rw-r--r-- | src/gpu/gl/iOS/GrGLCreateNativeInterface_iOS.cpp | 3 |
6 files changed, 9 insertions, 25 deletions
diff --git a/gyp/SampleApp.gyp b/gyp/SampleApp.gyp index a06082e8a2..edcba1078e 100644 --- a/gyp/SampleApp.gyp +++ b/gyp/SampleApp.gyp @@ -208,37 +208,26 @@ '../samplecode/SampleDecode.cpp', ], 'sources': [ - '../src/views/mac/SkEventNotifier.h', '../src/views/mac/SkEventNotifier.mm', - '../experimental/iOSSampleApp/SkSampleUIView.h', '../experimental/iOSSampleApp/SkSampleUIView.mm', '../experimental/iOSSampleApp/SkiOSSampleApp-Base.xcconfig', '../experimental/iOSSampleApp/SkiOSSampleApp-Debug.xcconfig', '../experimental/iOSSampleApp/SkiOSSampleApp-Release.xcconfig', '../experimental/iOSSampleApp/iOSSampleApp-Info.plist', - '../experimental/iOSSampleApp/Shared/SkOptionListController.h', '../experimental/iOSSampleApp/Shared/SkOptionListController.mm', - '../experimental/iOSSampleApp/Shared/SkUIRootViewController.h', '../experimental/iOSSampleApp/Shared/SkUIRootViewController.mm', - '../experimental/iOSSampleApp/Shared/SkOptionsTableViewController.h', '../experimental/iOSSampleApp/Shared/SkOptionsTableViewController.mm', - '../experimental/iOSSampleApp/Shared/SkUIView.h', '../experimental/iOSSampleApp/Shared/SkUIView.mm', - '../experimental/iOSSampleApp/Shared/SkUIDetailViewController.h', '../experimental/iOSSampleApp/Shared/SkUIDetailViewController.mm', '../experimental/iOSSampleApp/Shared/skia_ios.mm', # iPad - '../experimental/iOSSampleApp/iPad/AppDelegate_iPad.h', '../experimental/iOSSampleApp/iPad/AppDelegate_iPad.mm', - '../experimental/iOSSampleApp/iPad/SkUISplitViewController.h', '../experimental/iOSSampleApp/iPad/SkUISplitViewController.mm', '../experimental/iOSSampleApp/iPad/MainWindow_iPad.xib', # iPhone - '../experimental/iOSSampleApp/iPhone/AppDelegate_iPhone.h', '../experimental/iOSSampleApp/iPhone/AppDelegate_iPhone.mm', - '../experimental/iOSSampleApp/iPhone/SkUINavigationController.h', '../experimental/iOSSampleApp/iPhone/SkUINavigationController.mm', '../experimental/iOSSampleApp/iPhone/MainWindow_iPhone.xib', @@ -247,7 +236,6 @@ '../src/utils/ios/SkStream_NSData.mm', '../src/utils/ios/SkOSFile_iOS.mm', - '../include/utils/mac/SkCGUtils.h', '../src/utils/mac/SkCreateCGImageRef.cpp', '../experimental/iOSSampleApp/SkiOSSampleApp-Debug.xcconfig', '../experimental/iOSSampleApp/SkiOSSampleApp-Release.xcconfig', @@ -269,7 +257,9 @@ '../experimental/iOSSampleApp', '../experimental/iOSSampleApp/iPad', '../experimental/iOSSampleApp/iPhone', + '../experimental/iOSSampleApp/Shared', '../include/utils/ios', + '../src/views/mac', ], 'xcode_settings' : { 'INFOPLIST_FILE' : '../experimental/iOSSampleApp/iOSSampleApp-Info.plist', diff --git a/gyp/SimpleiOSApp.gyp b/gyp/SimpleiOSApp.gyp index a44c8d1f35..bb5fc8f34e 100644 --- a/gyp/SimpleiOSApp.gyp +++ b/gyp/SimpleiOSApp.gyp @@ -11,19 +11,15 @@ 'mac_bundle' : 1, 'include_dirs' : [ '../experimental/iOSSampleApp/Shared', + '../src/views/mac', ], 'sources': [ '../src/views/ios/SkOSWindow_iOS.mm', - '../src/views/mac/SkEventNotifier.h', '../src/views/mac/SkEventNotifier.mm', - '../experimental/iOSSampleApp/iPad/AppDelegate_iPad.h', '../experimental/iOSSampleApp/iPad/AppDelegate_iPad.mm', - '../experimental/iOSSampleApp/iPhone/AppDelegate_iPhone.h', '../experimental/iOSSampleApp/iPhone/AppDelegate_iPhone.mm', - '../experimental/iOSSampleApp/Shared/SkUIView.h', '../experimental/iOSSampleApp/Shared/SkUIView.mm', '../experimental/iOSSampleApp/Shared/skia_ios.mm', - '../experimental/SimpleiOSApp/SimpleApp.h', '../experimental/SimpleiOSApp/SimpleApp.mm', '../experimental/SimpleiOSApp/SimpleiOSApp-Info.plist', ], diff --git a/gyp/apptype_console.gypi b/gyp/apptype_console.gypi index b9c1ff651c..85cc502da3 100644 --- a/gyp/apptype_console.gypi +++ b/gyp/apptype_console.gypi @@ -34,19 +34,15 @@ '../include/views', '../include/xml', '../include/utils/mac', + '../src/views/mac', ], 'sources': [ '../src/views/ios/SkOSWindow_iOS.mm', - '../src/views/mac/SkEventNotifier.h', '../src/views/mac/SkEventNotifier.mm', - '../experimental/iOSSampleApp/iPad/AppDelegate_iPad.h', '../experimental/iOSSampleApp/iPad/AppDelegate_iPad.mm', - '../experimental/iOSSampleApp/iPhone/AppDelegate_iPhone.h', '../experimental/iOSSampleApp/iPhone/AppDelegate_iPhone.mm', - '../experimental/iOSSampleApp/Shared/SkUIView.h', '../experimental/iOSSampleApp/Shared/SkUIView.mm', '../experimental/iOSSampleApp/Shared/skia_ios.mm', - '../experimental/SimpleiOSApp/SimpleApp.h', '../experimental/SimpleiOSApp/SimpleApp.mm', ], 'dependencies': [ diff --git a/gyp/common_conditions.gypi b/gyp/common_conditions.gypi index 93788a1153..3743b60d3c 100644 --- a/gyp/common_conditions.gypi +++ b/gyp/common_conditions.gypi @@ -533,7 +533,7 @@ }, }, 'xcode_settings': { - 'ARCHS': ['armv6', 'armv7'], + 'ARCHS': ['armv7'], 'CODE_SIGNING_REQUIRED': 'NO', 'CODE_SIGN_IDENTITY[sdk=iphoneos*]': '', 'IPHONEOS_DEPLOYMENT_TARGET': '<(ios_sdk_version)', diff --git a/gyp/most.gyp b/gyp/most.gyp index 3d9bad45ca..b805eec1e7 100644 --- a/gyp/most.gyp +++ b/gyp/most.gyp @@ -29,6 +29,9 @@ ['skia_os == "android"', { 'dependencies': [ 'android_system.gyp:SampleApp_APK' ], }], + ['skia_os == "ios"', { + 'dependencies!': [ 'SampleApp.gyp:SampleApp' ], + }], [ 'skia_skip_gui', { 'dependencies!': [ diff --git a/src/gpu/gl/iOS/GrGLCreateNativeInterface_iOS.cpp b/src/gpu/gl/iOS/GrGLCreateNativeInterface_iOS.cpp index 553831d6ef..6af047159a 100644 --- a/src/gpu/gl/iOS/GrGLCreateNativeInterface_iOS.cpp +++ b/src/gpu/gl/iOS/GrGLCreateNativeInterface_iOS.cpp @@ -6,7 +6,6 @@ * found in the LICENSE file. */ - #include "gl/GrGLInterface.h" #import <OpenGLES/ES2/gl.h> @@ -70,7 +69,7 @@ const GrGLInterface* GrGLCreateNativeInterface() { functions->fReadBuffer = NULL; functions->fReadPixels = glReadPixels; functions->fScissor = glScissor; - functions->fShaderSource = glShaderSource; + functions->fShaderSource = (GrGLShaderSourceProc) glShaderSource; functions->fStencilFunc = glStencilFunc; functions->fStencilFuncSeparate = glStencilFuncSeparate; functions->fStencilMask = glStencilMask; |