Skip to content

Bugfix: Extract dependency installation from CLP core's library installation scripts; Replace usage of wget with curl. - #58

Merged
kirkrodrigues merged 1 commit into
y-scope:mainfrom
kirkrodrigues:refactor-libinstall
Apr 10, 2022
Merged

Conversation

@kirkrodrigues

Copy link
Copy Markdown
Member

References

Fixes bug in #57

Description

  • Rely on users / containers installing the necessary dependencies for the library installation scripts rather than having them in the scripts themselves. This is because each distro may be have different package managers, etc. which make the scripts unwieldy.
  • Replace wget with curl since curl is preinstalled in some Linux distributions.

Validation performed

  • Built all docker images successfully.
  • Verified CLP's core builds in each dependency container.
  • Verified CLP runs in the execution container.

@kirkrodrigues
kirkrodrigues merged commit bb81eec into y-scope:main Apr 10, 2022
@kirkrodrigues
kirkrodrigues deleted the refactor-libinstall branch April 10, 2022 02:26
junhaoliao pushed a commit to junhaoliao/clp that referenced this pull request May 17, 2026
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.

1 participant