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

Daml stdlib with versions #2479

Merged
4 commits merged into from
Aug 12, 2019
Merged

Daml stdlib with versions #2479

4 commits merged into from
Aug 12, 2019

Conversation

ghost
Copy link

@ghost ghost commented Aug 9, 2019

This adds versions to the daml-stdlib name so we avoid name clashes when different packages import different versions of the stdlib.

Pull Request Checklist

  • Read and understand the contribution guidelines
  • Include appropriate tests
  • Set a descriptive title and thorough description
  • Add a reference to the issue this PR will solve, if appropriate
  • Add a line to the release notes, if appropriate
  • Normal production system change, include purpose of change in description

NOTE: CI is not automatically run on non-members pull-requests for security
reasons. The reviewer will have to comment with /AzurePipelines run to
trigger the build.

@ghost ghost requested a review from hurryabit as a code owner August 9, 2019 14:23
@ghost ghost force-pushed the daml_stdlib_with_versions branch from 4da3e5a to a8e65e6 Compare August 9, 2019 14:27
Copy link
Contributor

@cocreature cocreature left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@ghost ghost force-pushed the daml_stdlib_with_versions branch from c48f150 to 055f2b1 Compare August 9, 2019 16:34
Copy link
Contributor

@hurryabit hurryabit left a comment

Choose a reason for hiding this comment

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

Good job. Thanks a lot.

if [ "daml-prim" = {pkg_name} ]; then
echo {pkg_name} > {pkg_name_version_file}
else
echo {pkg_name}-`cat {sdk_version_file}` > {pkg_name_version_file}
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
echo {pkg_name}-`cat {sdk_version_file}` > {pkg_name_version_file}
echo -n {pkg_name} > {pkg_name_version_file}
cat {sdk_version_file} >> {pkg_name_version_file}

@ghost ghost force-pushed the daml_stdlib_with_versions branch 3 times, most recently from fa515f1 to 00b04a2 Compare August 12, 2019 13:17
@ghost ghost force-pushed the daml_stdlib_with_versions branch from 00b04a2 to 92f5797 Compare August 12, 2019 13:32
@ghost ghost merged commit 829cb4c into master Aug 12, 2019
@ghost ghost deleted the daml_stdlib_with_versions branch August 12, 2019 14:05
This pull request was closed.
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.

2 participants