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

feat(launch): init support of launching applications #37

Merged
merged 2 commits into from
Jun 6, 2021

Conversation

tai271828
Copy link
Collaborator

In the sciwork online sprint today, this is the feature we have implemented for issue: #26 .

Steps to Verify the Pull Request

  1. source scripts/init
  2. devenv use flavor-issue-26
  3. devenv launch solvcon

Expected Result

  1. SOLVCON will be built
  2. flavor/flavor-issue-26/application-solvcon is created

Additioinaly Information

This pull request only closes #26 but not fully completes #23 for the follow incomplete tasks and imperfect features:

  1. Uses still need to manually re-launch SOLVCON
  2. SOLVCON source and lib are not fully integrated with flavor/usr/, flavor/usr/bin, and flavor/src.

I will create follow-up pull requests to complete above tasks later on to prevent a single huge pull request for code review. Inputs are welcome.

@tai271828
Copy link
Collaborator Author

The core part of this feature is currently referred to the implementation of devenv build. We will trace the architecture with the discussion thread #23 .

applications/solvcon/prepare-solvcon-dev.sh Show resolved Hide resolved
applications/solvcon/prepare-solvcon-dev.sh Outdated Show resolved Hide resolved
scripts/func.d/bash_utils Outdated Show resolved Hide resolved
@tai271828 tai271828 requested a review from yungyuc June 6, 2021 07:52
@tai271828
Copy link
Collaborator Author

The updated PR is ready to review again. Thanks, team!

@yungyuc yungyuc merged commit c144dd0 into solvcon:main Jun 6, 2021
@tai271828 tai271828 deleted the mr-devenv-launch branch June 6, 2021 11:11
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.

Support devenv launch <application>
2 participants