We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

Bug 3670 - CMake IOS haptic error
Summary: CMake IOS haptic error
Status: RESOLVED FIXED
Alias: None
Product: SDL
Classification: Unclassified
Component: build (show other bugs)
Version: HG 2.1
Hardware: All iOS (All)
: P2 normal
Assignee: Ryan C. Gordon
QA Contact: Sam Lantinga
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-06-11 17:46 UTC by Martin Gerhardy
Modified: 2017-06-12 23:36 UTC (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Gerhardy 2017-06-11 17:46:53 UTC 
endif()
 
   if(SDL_HAPTIC)
     set(SDL_HAPTIC_IOKIT 1)
     if (IOS)
-      file(GLOB HAPTIC_SOURCES ${SDL2_SOURCE_DIR}/src/haptic/dummy/*.c)
+      file(GLOB POWER_SOURCES ${SDL2_SOURCE_DIR}/src/power/dummy/*.c)
       set(SDL_HAPTIC_DUMMY 1)
     else()
       file(GLOB HAPTIC_SOURCES ${SDL2_SOURCE_DIR}/src/haptic/darwin/*.c)


Somehow the power sources slipped into the ios cmake section for the haptic dummy driver
Comment 1 Ryan C. Gordon 2017-06-11 20:01:18 UTC 
This patch is now https://hg.libsdl.org/SDL/rev/84a49d9b3e58, thanks!

--ryan.
Comment 2 Martin Gerhardy 2017-06-12 06:39:00 UTC 
Thanks a lot - one part was missed - GLOB POWER_SOURCES should be used as variable names. (Not sure whether this is a must have - but it's the cleaner way)
Comment 3 Martin Gerhardy 2017-06-12 06:40:27 UTC 
I meant shouldn't - GLOB HAPTIC_SOURCES should be used - the patch was just recorded in the wrong direction. Sorry for the confusion.
Comment 4 Sam Lantinga 2017-06-12 23:36:12 UTC 
Fixed, thanks!
https://hg.libsdl.org/SDL/rev/5cf754a84eb7