Skip to content
This repository has been archived by the owner on Feb 29, 2020. It is now read-only.

Sentry breaking the build #3

Closed
orta opened this issue May 22, 2017 · 11 comments
Closed

Sentry breaking the build #3

orta opened this issue May 22, 2017 · 11 comments

Comments

@orta
Copy link
Contributor

orta commented May 22, 2017

Seeing getsentry/sentry-react-native#32

Since we got Sentry + RN set up, builds are failing

▸ Compiling RNSentry.m
⚠️  /Users/travis/build/artsy/emission-nebula/emission/node_modules/react-native-sentry/ios/RNSentry.m:27:5: expression result unused [-Wunused-value]
[[bridge moduleForName:@"ExceptionsManager"] initWithDelegate:sentry];
        ~~~ ^
⚠️  /Users/travis/build/artsy/emission-nebula/emission/node_modules/react-native-sentry/ios/RNSentry.m:108:86: undeclared selector 'callNativeModule:method:params:' [-Wundeclared-selector]
uintptr_t callNativeModuleAddress = [RCTBatchedBridge instanceMethodForSelector:@selector(callNativeModule:method:params:)];
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
⚠️  /Users/travis/build/artsy/emission-nebula/emission/node_modules/react-native-sentry/ios/RNSentry.m:108:15: incompatible pointer to integer conversion initializing 'uintptr_t' (aka 'unsigned long') with an expression of type 'IMP' (aka 'void (*)(void)') [-Wint-conversion]
uintptr_t callNativeModuleAddress = [RCTBatchedBridge instanceMethodForSelector:@selector(callNativeModule:method:params:)];
                                                                                     ^
⚠️  /Users/travis/build/artsy/emission-nebula/emission/node_modules/react-native-sentry/ios/RNSentry.m:111:30: undeclared selector 'callNativeModule:method:params:' [-Wundeclared-selector]
@selector(callNativeModule:method:params:),
              ^                         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
❌  /Users/travis/build/artsy/emission-nebula/emission/node_modules/react-native-sentry/ios/RNSentry.m:249:28: no visible @interface for 'SentryClient' declares the selector 'reportReactNativeCrashWithError:stacktrace:terminateProgram:'
[[SentryClient shared] reportReactNativeCrashWithError:error stacktrace:stack terminateProgram:terminateProgram];
                             ^
▸ Compiling RNSentryEventEmitter.m
▸ Compiling SentryReactNative-dummy.m
▸ Compiling SentryReactNative_vers.c
▸ Compiling RNSentry.m
** ARCHIVE FAILED **
The following build commands failed:
CompileC /Users/travis/Library/Developer/Xcode/DerivedData/Emission-fillzptesxgqaydtzjjvxbvqdesz/Build/Intermediates/ArchiveIntermediates/Emission/IntermediateBuildFilesPath/Pods.build/Deploy-iphoneos/SentryReactNative.build/Objects-normal/armv7/RNSentry.o /Users/travis/build/artsy/emission-nebula/emission/node_modules/react-native-sentry/ios/RNSentry.m normal armv7 objective-c com.apple.compilers.llvm.clang.1_0.compiler
(1 failure)
[09:08:43]: Exit status: 65
[09:08:43]: 📋  For a more detailed error log, check the full log at:
[09:08:43]: 📋  /Users/travis/Library/Logs/gym/Emission-Emission.log
@orta
Copy link
Contributor Author

orta commented May 22, 2017

I added a pod repo update but that didn't help - 2058e78

@orta
Copy link
Contributor Author

orta commented May 22, 2017

It installed Installing Sentry (2.1.9) which isn't latest, will try updating the lockfile

orta added a commit that referenced this issue May 25, 2017
@orta
Copy link
Contributor Author

orta commented May 25, 2017

It's because our build toolchain now contains Swift, we have to use a specific build of Xcode now.

Making sure travis is running on Xcode 8.3 conceptually fixed this, but the build failed due to the pod resource script which feels really weird = https://travis-ci.org/artsy/emission-nebula/builds/236080641

@orta
Copy link
Contributor Author

orta commented May 26, 2017

@orta
Copy link
Contributor Author

orta commented May 27, 2017

screen shot 2017-05-27 at 15 42 09

Sigh, getting there. Slowly

@orta
Copy link
Contributor Author

orta commented May 27, 2017

Shame, I'll have to do all this then 3.0 will come out and it'll all be for nothing - getsentry/sentry-cocoa#154

@orta
Copy link
Contributor Author

orta commented May 27, 2017

Also tried: artsy/emission#572

@orta
Copy link
Contributor Author

orta commented May 27, 2017

Ran the setup locally, found this:
screen shot 2017-05-27 at 17 17 02

@orta
Copy link
Contributor Author

orta commented May 29, 2017

Came to the conclusion on an airplane that I can override the Pod at our app level to just provide a fake API which we use, and we can use that to avoid all these swift issues

@orta
Copy link
Contributor Author

orta commented Jun 16, 2017

Now using libraries again, shipped a few releases manually. I can archive correctly in Xcode, but struggling to do it right inside Fastlane

@orta
Copy link
Contributor Author

orta commented Jul 22, 2017

So, the later issues around empty archives are fixed with 705a32d

@orta orta closed this as completed Jul 22, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant