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

Ma/update pc and rebar #1495

Merged
merged 2 commits into from
May 30, 2018
Merged

Ma/update pc and rebar #1495

merged 2 commits into from
May 30, 2018

Conversation

markan
Copy link
Contributor

@markan markan commented Apr 26, 2018

Lock port compiler; it's the only thing we don't lock.

Update rebar to 3.5.0

@markan markan requested a review from a team April 26, 2018 00:57
@markan markan force-pushed the ma/update_pc_and_rebar branch 3 times, most recently from b105848 to 1ba9e48 Compare May 1, 2018 22:07
@markan markan force-pushed the ma/update_pc_and_rebar branch 3 times, most recently from a05c783 to 83fb01e Compare May 22, 2018 20:30
Copy link
Contributor

@srenatus srenatus left a comment

Choose a reason for hiding this comment

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

LGTM (with src/sky gone ;))

@@ -0,0 +1,2 @@
{erl_opts, [debug_info]}.
Copy link
Contributor

Choose a reason for hiding this comment

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

Did you intend to commit this folder? 😉

markan added 2 commits May 30, 2018 10:41
Looks like there's been some breaking changes in port compiler
(blt/port_compiler#43); I don't know what
the change plan is there, but we are unlocked on this dep; we should
lock it like we do everything else.

Signed-off-by: Mark Anderson <[email protected]>
The author of port compiler was recommending it, and it seems like
good general practice.

Signed-off-by: Mark Anderson <[email protected]>
@markan markan force-pushed the ma/update_pc_and_rebar branch from 83fb01e to 5cf7789 Compare May 30, 2018 17:42
@markan markan merged commit f7c073a into master May 30, 2018
@markan markan deleted the ma/update_pc_and_rebar branch May 30, 2018 17:58
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.

3 participants