Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

iOS release build error #308

Closed
gsanseverino opened this issue Apr 5, 2023 · 6 comments · Fixed by #309
Closed

iOS release build error #308

gsanseverino opened this issue Apr 5, 2023 · 6 comments · Fixed by #309

Comments

@gsanseverino
Copy link

gsanseverino commented Apr 5, 2023

Hello,

running release build for ios I'm getting this error:

SentryCordova.m:105:13: error: use of undeclared identifier 'options'; did you mean 'optind'?
         if (options[@'store']) {
             ^~~~~~~
             optind
             
SentryCordova.m:105:19: error: subscript requires size of interface 'NSNumber', which is not constant for this architecture and platform
        if (options[@'store']) {
                   ^~~~~~~~~

it's not happening for debug builds.

I'm using cordova 11, sentry-cordova 1.1.4

@gsanseverino
Copy link
Author

shouldn't the options variable be commandDictoinary instead?

@marian-turchyn-oril
Copy link

@gsanseverino You made my day. Thx. Changing variable is working.

@awaken180-dev
Copy link

How were you able to adjust this for the build?

@gsanseverino
Copy link
Author

If you're using Ionic, the file is in the platforms/ios/<ProjectName>/Plugins/sentry-cordova folder

@awaken180-dev
Copy link

So you modify that file? Do you run into any issues when building in AppFlow? I was able to downgrade to 1.0.4 rather than 1.1.4 and my issue was resolved.

@gsanseverino
Copy link
Author

If you change the file platforms/ios/<ProjectName>/Plugins/sentry-cordova/SentryCordova.m as explained in the first message, you can fix the issue.

But I'm not using AppFlow, so if platform files are generated in that pipeline, you have to find a different solution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants