Skip to content

Conversation

dkimitsa
Copy link
Contributor

@dkimitsa dkimitsa commented Jan 7, 2021

This PR fixed #542 and also changes the way project compilation is triggered (for IPA/Framework creation)

Root case:

CompilerManager is not invoking build plugins such as Dagger. As result intermediate classes are not being generated and RoboVM compilation fails.

As a fix:

ProjectTaskManager.build(module) is now used with is equal to menu Build/build module. Once build is success -- RoboVM compiles IPA/Framework binaries.

other moments

RoboVmCompileTask was removed from compilation chain. This means that it will not be triggered automatically. However IPA/Framework was reworked in this fix to call it manually. Also run configuration already has manual RoboVM compilation invocation as pre-launch task

other fixes

  • better cancellation processing during IPA/Framework compilation
  • fixed issue during SDK files installation during first launch

# Root case:
`CompilerManager` is not invoking build plugins such as Dagger. As result intermediate classes are not being generated and RoboVM compilation fails.

# As a fix:
`ProjectTaskManager.build(module)` is now used with is equal to menu `Build/build module`. Once build is success -- RoboVM compiles IPA/Framework binaries.

# other moments
`RoboVmCompileTask` was removed from compilation chain. This means that it will not be triggered automatically. However IPA/Framework was reworked in this fix to call it manually. Also run configuration already has manual RoboVM compilation invocation as pre-launch task

# other fixes
- better cancellation processing during IPA/Framework compilation
- fixed issue during SDK files installation during first launch
@Tom-Ski Tom-Ski merged commit e7bdadb into MobiVM:master Jan 18, 2021
@dkimitsa dkimitsa deleted the fix/542_using_task_manager_for_builds branch January 20, 2021 01:20
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.

Dagger with Kotlin, IDE IPA Creation Problem
2 participants