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

Add plan for rebar3 #921

Merged
merged 1 commit into from
Jun 30, 2016
Merged

Add plan for rebar3 #921

merged 1 commit into from
Jun 30, 2016

Conversation

elliott-davis
Copy link
Contributor

Signed-off-by: Elliott Davis [email protected]

@thesentinels
Copy link
Contributor

By analyzing the blame information on this pull request, we identified @adamhjk to be a potential reviewer

@elliott-davis elliott-davis mentioned this pull request Jun 19, 2016
# The `/usr/bin/env` path is hardcoded, so we'll add a symlink if needed.
if [[ ! -r /usr/bin/env ]]; then
ln -sv $(pkg_path_for coreutils)/bin/env /usr/bin/env
_clean_env=true
Copy link
Contributor

Choose a reason for hiding this comment

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

this appears to be unused

Copy link
Contributor Author

Choose a reason for hiding this comment

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

updated so that it's used now

@bookshelfdave
Copy link
Contributor

bookshelfdave commented Jun 20, 2016

this needs a runtime dep for core/sed:

root@5416bc1c1e72:/src/plans# hab pkg exec core/rebar3 rebar3
/hab/pkgs/core/erlang/18.3/20160612151228/bin/erl: 1: /hab/pkgs/core/erlang/18.3/20160612151228/bin/erl: sed: not found

@bookshelfdave bookshelfdave self-assigned this Jun 24, 2016
_clean_env=true
fi
}

Copy link
Collaborator

Choose a reason for hiding this comment

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

Possible to remove the one extra newline?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

removed

@fnichol
Copy link
Collaborator

fnichol commented Jun 24, 2016

When I ran the rebar3 plan and examined bin/rebar3 I found that it was largely binary, but thinly wrapped with a shebang:

[11][default:/hab/pkgs/core/rebar3/3.1.1/20160624160040/bin:0]# head -n 2 rebar3
#!/usr/bin/env escript
%%

Given that strictly inside Habitat, we can't guarantee that /usr/bin/env exists, it's probably worth changing this shebang. There are 2 ways to go, in my mind:

  1. Taking a dependency on core/busybox-static, core/coreutils-static, or core/coreutils in order to to use the fix_interpreter() helper function
  2. Update the shebang to directly run escript as there isn't really a downside to doing this directly with an absolute path. The resulting value would be something like: #!${pkg_path_for erlang}/bin/escript. This strategy is used in the node Plan.

@bookshelfdave bookshelfdave removed their assignment Jun 24, 2016
* Updated Erlang plan to include missing sed dep

Signed-off-by: Elliott Davis <[email protected]>
@elliott-davis
Copy link
Contributor Author

good catch on that binary @fnichol - I got it patched up using fix_interpreter. I also got the newline removed.

@bookshelfdave
Copy link
Contributor

@thesentinels delegate=elliott-davis

I'm delegating merge access to you, please read these instructions for more info!

@thesentinels
Copy link
Contributor

✌️ @elliott-davis can now approve this pull request

@elliott-davis
Copy link
Contributor Author

@thesentinels r+

@thesentinels
Copy link
Contributor

📌 Commit 191022c has been approved by elliott-davis

@thesentinels
Copy link
Contributor

⌛ Testing commit 191022c with merge 59dcaa6...

thesentinels added a commit that referenced this pull request Jun 30, 2016
Add plan for rebar3

Signed-off-by: Elliott Davis <[email protected]>
@thesentinels
Copy link
Contributor

☀️ Test successful - travis

@thesentinels thesentinels merged commit 191022c into habitat-sh:master Jun 30, 2016
@adamhjk adamhjk removed the In Review label Jun 30, 2016
thesentinels added a commit that referenced this pull request Jun 30, 2016
Add plan for relx

* Note - this is blocked by the PR #921

Signed-off-by: Elliott Davis <[email protected]>
@smith
Copy link
Contributor

smith commented Jun 30, 2016

Built and uploaded to depot https://app.habitat.sh/#/pkgs/core/rebar3/3.1.1/20160630160805

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.

6 participants