getsentry / sentry-cordova

The official Sentry SDK for Cordova
https://sentry.io/
MIT License
68 stars 55 forks source link

IOS - Command PhaseScriptExecution failed with a nonzero exit code #311

Closed 9t96 closed 5 months ago

9t96 commented 11 months ago

I get "Command PhaseScriptExecution failed with a nonzero exit code" on Strip unused archs phase when building on Xcode 14.3.

sentry-cordova: v1.1.5

I check this and its ok: https://docs.sentry.io/platforms/javascript/guides/cordova/troubleshooting/?original_referrer=https%3A%2F%2Fwww.npmjs.com%2F

I tried the following workaround but doesnt work for me: Originally posted by @RedEars in https://github.com/getsentry/sentry-cordova/issues/160#issuecomment-590592202

Also tried replacing the script to the suggestes on this issue, but only work for installing on devices, keeps failing when tryuing to Archive for example: https://github.com/getsentry/sentry-cordova/issues/29#issuecomment-524251380

image

9t96 commented 11 months ago

There's something i could do about it? Thanks!

pivalig commented 11 months ago

Any updates?

github-actions[bot] commented 10 months ago

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you label it Status: Backlog or Status: In Progress, I will leave it alone ... forever!


"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

mayainle commented 10 months ago

I have the same issue. Did you find a solution?

igorsantos07 commented 10 months ago

This is happening to us as well...

Somewhere around the error output, it says this phase can be skipped by using SENTRY_SKIP_FRAMEWORK_PATCH=true before the build command you're running (which seems to have been originated at https://github.com/getsentry/sentry-wizard/issues/39). This did the trick for me, but I'm unsure if this will cause trouble when we submit the app to iTunes Connect.

Thus, also waiting for a final solution from maintainers.

northkode commented 10 months ago

same issue Sentry.framework/Sentry: No such file or directory Command PhaseScriptExecution failed with a nonzero exit code

igorsantos07 commented 10 months ago

Funny, that file indeed doesn't seem to exist after the build fails, but during it, it exists: adding if [ -f "$FRAMEWORK_EXECUTABLE_PATH" ] inside the for has no effect.

northkode commented 10 months ago

Have you been able to successfully archive a build? My build works to run it, but i can't archive it to submit to apple. @igorsantos07

igorsantos07 commented 10 months ago

I don't have the privileges in the company to do it, so I didn't reach this step hahah However, given the links I posted in the previous comment, I guess it indeed wouldn't work. I think we'll need to sit and wait for some Sentry staff to find this issue. Maybe it's worth opening a support ticket mentioning this issue?

lucas-zimerman commented 9 months ago

sorry for the late response, I will take a look at it.

@9t96 are you facing this issue on a Mac M1 or X86/X64?

9t96 commented 9 months ago

sorry for the late response, I will take a look at it.

@9t96 are you facing this issue on a Mac M1 or X86/X64?

No, im using Macbook Pro 2020 with Intel chip

northkode commented 9 months ago

I'm on m1 and this happens

fbfdsouza commented 9 months ago

any updates on here?

I don't know if this can shed some light on the issue, but for me in relation to these two paths.

lipo -extract arm64 myproject_folder/platforms/ios/ProjectName/Plugins/sentry-cordova/Sentry.xcframework/ios-arm64/Sentry.framework/Sentry -output ~/Desktop/arm64_file

this I get the error: must be a fat file when the -extract option is specified

lipo -extract arm64 myproject_folder/platforms/ios/ProjectName/Plugins/sentry-cordova/Sentry.xcframework/ios-arm64_x86_64-simulator/Sentry.framework/Sentry -output ~/Desktop/arm64_file

image

and this works.

but the second one seems to be related to the binaries that are meant to the simulator since the folder is named ios-arm64_x86_64-simulator and the first one for the device build with folder name guess ios-arm64?

OBS: I couldn't find the ones related to the xcode path, these are from my local cordova react project after runing the build command.

northkode commented 9 months ago

Any updates? This blocks any Apple submission..

lucas-zimerman commented 9 months ago

Still checking it.

mafikes commented 9 months ago

The same problem. M1. cordova 12 ios: 7.0.0

igorsantos07 commented 9 months ago

I was also facing this issue from an Intel Mac Mini 2020 (2019?).

HappyTepid commented 8 months ago

@lucas-zimerman do you have any update on this? I'm facing the possibility of having to remove Sentry from an iOS app just to be able to make releases

northkode commented 8 months ago

We have already removed sentry, the lack of support and change on this thread as made us lose confidence in there solutions. It completely stops any IOS building to the app store and they seem to not really care.

lucas-zimerman commented 8 months ago

@lucas-zimerman do you have any update on this? I'm facing the possibility of having to remove Sentry from an iOS app just to be able to make releases

Does previous versions works for you? There's a fix being worked on it but lots of changes internally so it was being validated.

EDIT: are you able to build it by altering the last lines on Sentry strip unused archs from framework

for: rm "$FRAMEWORK_EXECUTABLE_PATH" || true mv "$FRAMEWORK_EXECUTABLE_PATH-merged" "$FRAMEWORK_EXECUTABLE_PATH" || true ?

gabn88 commented 6 months ago

@lucas-zimerman

I changed the lines and the build did continue.

But now I have:

Failed to verify code signature of /var/installd/Library/Caches/com.apple.mobile.installd.staging/temp.ujoouV/extracted/My.app/Frameworks/Sentry.framework : 0xe800801c (No code signature found.
lucas-zimerman commented 6 months ago

@gabn88 have you tried the latest release of Sentry Cordova?

lucas-zimerman commented 5 months ago

closing as completed, if the same persist, feel free to open a new issue.