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

Add sim target to cordova ios build script #158

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

danielSanchezQ
Copy link
Contributor

No description provided.

@@ -7,7 +7,7 @@ license = "MIT OR Apache-2.0"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
[lib]
crate-type = [ "cdylib" ]
crate-type = [ "cdylib", "staticlib" ]
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apparently cdylib are deprecated for aarch64 builds, I added staticlib but I'm not sure If we need to remove the cdylib one.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmmm, things won't work if we remove the cdylib though, as we are actually using the .so file.

I'm not sure if we can load static libraries in jni directly, so maybe we need to tweak something. I'll try to find out.

@danielSanchezQ danielSanchezQ changed the title Add sim target to build cordova ios script Add sim target to cordova ios build script Jun 11, 2021
@@ -52,4 +59,6 @@ def run():


if __name__ == "__main__":
run()
# Set flag for emulator compatible lib
sim = "--sim" in sys.argv
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so we need to have plugin builds for simulators and builds for apps? I wonder if we can compile the two and choose one at (app) build time.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe I'll have to rename the target in order to have all into one lib. Will research that.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All right, it may not be possible, though... In that case maybe we can put a hook on cordova or something

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

Successfully merging this pull request may close these issues.

2 participants