-
Notifications
You must be signed in to change notification settings - Fork 7
Add support for flags to build on iOS #96
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
base: fork
Are you sure you want to change the base?
Add support for flags to build on iOS #96
Conversation
Are we going to require additional flags for all of the Apple operating systems? Then also for Android, Windows, etc... ? In other words, is adding each one by one the best approach or is there a more flexible one we should consider? |
The |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Something is wrong: I'm testing this branch against our current release build, but it doesn't work.
See https://github.com/lightpanda-io/zig-v8-fork/actions/runs/17858246544/job/50782218176?pr=97
downloading https://chrome-infra-packages.appspot.com/dl/gn/gn/macos-arm64/+/latest into tools/gn.zip
% Total % Received % Xferd Average Speed Time Time Time Current
Dload Upload Total Spent Left Speed
0 0 0 0 0 0 0 0 --:--:-- --:--:-- --:--:-- 0
100 35 100 35 0 0 269 0 --:--:-- --:--:-- --:--:-- 271
End-of-central-directory signature not found. Either this file is not
a zipfile, or it constitutes one disk of a multi-part archive. In the
latter case the central directory and zipfile comment will be found on
the last disk(s) of this archive.
unzip: cannot find zipfile directory in one of tools/gn.zip or
tools/gn.zip.zip, and cannot find tools/gn.zip.ZIP, period.
Archive: tools/gn.zip
get-v8
+- run exe get-v8
+- compile exe get-v8 Debug native
+- run /bin/bash
+- run /bin/bash failure
error: the following command exited with error code 9:
cd v8 && /bin/bash get_tools.sh
Build Summary: 2/7 steps succeeded; 1 failed
get-v8 transitive failure
+- run exe get-v8 transitive failure
+- compile exe get-v8 Debug native transitive failure
+- run /bin/bash transitive failure
+- run /bin/bash failure
error: the following build command failed with exit code 1:
/Users/runner/.cache/zig/o/41b2e58e23f54e6061b8f9707a8d50e3/build /Users/runner/work/_temp/783d9ed5-a004-4bfc-9cbe-2be4b29959e1/zig-aarch64-macos-0.15.1/zig /Users/runner/work/_temp/783d9ed5-a004-4bfc-9cbe-2be4b29959e1/zig-aarch64-macos-0.15.1/lib /Users/runner/work/zig-v8-fork/zig-v8-fork /Users/runner/.cache/zig /Users/runner/.cache/zig --seed 0xf79dda71 -Z6f21bfb0cd071259 get-v8
Error: Process completed with exit code 1.
In fact I'm not sure it relates to your PR 🤔 |
Unfortunately Carson is out for the next two weeks. I can have Isaac take a look but likely it may need to wait as Carson is the Swift expert on our team |
not a problem, we can wait until Carson is back 👍 |
esac | ||
: "${OS:=unset}" | ||
case "$OS" in | ||
unset) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok I got what's wrong: we set the OS w/ macos
. I worked before b/c the OS
was ignored until this change.
unset) | |
macos) OS="mac" ;; | |
unset) |
So the
Here is the log: https://github.com/lightpanda-io/zig-v8-fork/actions/runs/17859485614/job/50786267288 zig-v8-fork/.github/workflows/build-release.yml Lines 144 to 172 in 8589ee0
OS=ios .
|
This enables v8 to be cross compiled for iOS.
OS=ios
can be set in the environment before runningmake build-v8
, and the correct extra args will be added.