Skip to content

Conversation

@absaroj
Copy link
Contributor

@absaroj absaroj commented Jun 9, 2023

Changes in local build script and cmakelist file to build swift wrappers package for local sample main.swift file.

Below tasks are done by the cmakelist file:

  1. Locate all the required static libs.
  2. Run swift build to build swift wrappers package.
  3. Combine all object files from swift wrappers to make a static lib.
  4. Invoke swiftc compiler to compile the sample swift code while including the modules (wrappers and objc)
    and linking the static libs to generate the executable.

@absaroj absaroj changed the title Absaroj/local build Adding changes for local build script and cmakelist to create exe using swift package Jun 9, 2023
Base automatically changed from absaroj/swift_package to main June 9, 2023 20:37
@absaroj absaroj marked this pull request as ready for review June 9, 2023 21:08
@lalitb
Copy link
Contributor

lalitb commented Jun 10, 2023

Can we integrate these build steps with root CMakeLists.txt, and let it build whenever the iOS build is triggered ? This will also let this build trigger from CI.

@absaroj absaroj closed this Jun 12, 2023
@absaroj absaroj reopened this Jun 12, 2023
@absaroj
Copy link
Contributor Author

absaroj commented Jun 12, 2023

Can we integrate these build steps with root CMakeLists.txt, and let it build whenever the iOS build is triggered ? This will also let this build trigger from CI.

Yeah. I have added the changes to compile the swift wrappers as part of the global build. That is in separate PR.

@absaroj absaroj requested a review from mannie June 16, 2023 16:55
@lalitb
Copy link
Contributor

lalitb commented Jun 21, 2023

Force merge it. The CI failure is not related to this PR.

@lalitb lalitb merged commit ada085e into main Jun 21, 2023
@lalitb lalitb deleted the absaroj/local_build branch June 21, 2023 19:17
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.

5 participants