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

"com.microsoft.onnxruntime_extensions" is not a valid bundle identifier #825

Open
ronyfadel opened this issue Oct 13, 2024 · 7 comments
Open

Comments

@ronyfadel
Copy link
Contributor

the exported framework has a bundle identifier of "com.microsoft.onnxruntime_extensions", which is invalid (because of the "_") and Xcode is starting to reject it.

@wenbingl
Copy link
Member

@edgchen1 , any suggestions?

@edgchen1
Copy link
Contributor

@edgchen1 , any suggestions?

maybe rename it to "com.microsoft.onnxruntime-extensions".

https://developer.apple.com/documentation/bundleresources/information_property_list/cfbundleidentifier?language=objc

ronyfadel added a commit to ronyfadel/onnxruntime-extensions that referenced this issue Oct 21, 2024
Ref: microsoft#825

"com.microsoft.onnxruntime_extensions" is not a valid identifier. Update it to "com.microsoft.onnxruntime-extensions"
@ronyfadel
Copy link
Contributor Author

#829

wenbingl pushed a commit that referenced this issue Oct 21, 2024
Ref: #825

"com.microsoft.onnxruntime_extensions" is not a valid identifier. Update it to "com.microsoft.onnxruntime-extensions"
@ronyfadel
Copy link
Contributor Author

@edgchen1 @wenbingl can we create a new release please, both here and in https://github.com/microsoft/onnxruntime-swift-package-manager ?

The issues will start piling up soon as this breaks the package in Xcode 16.0

@wenbingl
Copy link
Member

@apsonawane, can you cherry-pick PR #829 to the release branch, which is minor change but fixed a very important issue?

apsonawane pushed a commit that referenced this issue Oct 28, 2024
Ref: #825

"com.microsoft.onnxruntime_extensions" is not a valid identifier. Update it to "com.microsoft.onnxruntime-extensions"
apsonawane added a commit that referenced this issue Oct 28, 2024
Ref: #825

"com.microsoft.onnxruntime_extensions" is not a valid identifier. Update it to "com.microsoft.onnxruntime-extensions"

Co-authored-by: Rony Fadel <[email protected]>
@ronyfadel
Copy link
Contributor Author

Hey @apsonawane, @wenbingl , any updates to this?

The way I see it we'll need a new release in this project, and a new one at https://github.com/microsoft/onnxruntime-swift-package-manager that references it.

@apsonawane
Copy link
Contributor

Hello @ronyfadel, we are planning on releasing this week will keep you updated.

Thank you!

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

No branches or pull requests

4 participants