-
Notifications
You must be signed in to change notification settings - Fork 205
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
Conversation
4da3e5a
to
a8e65e6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
c48f150
to
055f2b1
Compare
There was a problem hiding this 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} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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} |
fa515f1
to
00b04a2
Compare
00b04a2
to
92f5797
Compare
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
NOTE: CI is not automatically run on non-members pull-requests for security
reasons. The reviewer will have to comment with
/AzurePipelines run
totrigger the build.