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

(wip) initial ios support #8

Draft
wants to merge 28 commits into
base: main
Choose a base branch
from
Draft

Conversation

RoootTheFox
Copy link

@RoootTheFox RoootTheFox commented Apr 22, 2024

todo:

  • get ios mods to build
  • fix combine
  • use geode prebuilt binaries (pending geode build action)
  • make the whole thing not based on workarounds
  • once Geode has regular ios releases: stop forcing nightly builds on ios

Signed-off-by: rooot <hey@rooot.gay>
Signed-off-by: rooot <hey@rooot.gay>
Signed-off-by: rooot <hey@rooot.gay>
Signed-off-by: rooot <hey@rooot.gay>
Signed-off-by: rooot <hey@rooot.gay>
Signed-off-by: rooot <hey@rooot.gay>
Signed-off-by: rooot <hey@rooot.gay>
Signed-off-by: rooot <hey@rooot.gay>
Signed-off-by: rooot <hey@rooot.gay>
Signed-off-by: rooot <hey@rooot.gay>
Signed-off-by: rooot <hey@rooot.gay>
Signed-off-by: rooot <hey@rooot.gay>
Signed-off-by: rooot <hey@rooot.gay>
Signed-off-by: rooot <hey@rooot.gay>
Signed-off-by: rooot <hey@rooot.gay>
temporary workaround for the lack of releases

Signed-off-by: rooot <hey@rooot.gay>
@RoootTheFox
Copy link
Author

did some changes here - the action now (temporarily!) uses a CLI fork to get prebuilt Geode libraries, which will be reverted once iOS support has been merged into Geode and there are regular releases for iOS to link against.

Comment on lines 54 to 56
if [$GEODE_VERSION == null]
GEODE_VERSION="3.2.0"
fi
Copy link
Author

Choose a reason for hiding this comment

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

😟

Copy link

Choose a reason for hiding this comment

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

dont worry!

i squashed some more commits as well
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

Successfully merging this pull request may close these issues.

3 participants