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

Make scripts in rlx_prv_assembler DTL templates #128

Merged
merged 13 commits into from
Mar 11, 2014

Conversation

nuex
Copy link
Contributor

@nuex nuex commented Feb 22, 2014

This moves scripts out of the rlx_prv_assembler and into their own DTL template files in priv/templates.

This will work with the latest rebar.

@jwilberding
Copy link
Member

@nuex could you clean this up so it passes the CI build?

@nuex
Copy link
Contributor Author

nuex commented Mar 4, 2014

I think it failed because Travis didn't have an updated rebar. Latest rebar should build it fine. Any way you can restart the build and see if it works?

@jwilberding
Copy link
Member

Yep, I think so, let me try!

On Tue, Mar 4, 2014 at 9:54 AM, Chase James notifications@git.luolix.topwrote:

I think it failed because Travis didn't have an updated rebar. Latest
rebar should build it fine. Any way you can restart the build and see if it
works?

Reply to this email directly or view it on GitHubhttps://github.com//pull/128#issuecomment-36603617
.

@nuex
Copy link
Contributor Author

nuex commented Mar 4, 2014

Well, this didn't work out. I'm guessing Travis isn't using the latest rebar. It builds fine on my local machine. I got on Travis IRC and asked what version of rebar they are using.

Another possibility for checking the rebar version is to have make rebuild run "rebar --version" and allowing us to see the version in Travis's build results.

@nuex
Copy link
Contributor Author

nuex commented Mar 4, 2014

@jwilberding more updates on this. I told the folks on #travis my problem and they just realized their default rebar is from the basho/rebar repository.

They suggested we include a compiled rebar bin in the relx repo and travis will use that instead by default.

I personally prefer keeping bins out of the repo and in a directory loaded from $PATH. What do you think?

@nuex
Copy link
Contributor Author

nuex commented Mar 4, 2014

I was just informed that we could pull the rebar bin before building. Maybe call include get-rebar in the make rebuild step? Or possibly have a new make travis build task?

@nuex
Copy link
Contributor Author

nuex commented Mar 4, 2014

@nuex nuex mentioned this pull request Mar 6, 2014
@jwilberding
Copy link
Member

@Neux I will try kicking off the build again

@jwilberding
Copy link
Member

@nuex Still failing :( I guess it may not be a bad idea to change the build script to automatically pull in rebar.

Do you have time to do that and submit a PR? It may be a day or two before I can do it.

@nuex
Copy link
Contributor Author

nuex commented Mar 6, 2014

@jwilberding: Sure!

@nuex
Copy link
Contributor Author

nuex commented Mar 6, 2014

@jwilberding Here is the PR: #134

@jwilberding
Copy link
Member

@Neux Still failing for R16: https://travis-ci.org/erlware/relx/builds/20039654

@nuex
Copy link
Contributor Author

nuex commented Mar 7, 2014

@jwilberding I haven't rebased this against master yet. Should be able to get to it in an hour or so.

@nuex
Copy link
Contributor Author

nuex commented Mar 7, 2014

@jwilberding I'll work on this more later this afternoon.

@nuex
Copy link
Contributor Author

nuex commented Mar 7, 2014

@jwilberding well, it almost passed: https://travis-ci.org/erlware/relx/jobs/20315459

@nuex
Copy link
Contributor Author

nuex commented Mar 7, 2014

@jwilberding this may have been a glitch in the build. Would you mind restarting it to see if it happens again?

@jwilberding
Copy link
Member

@nuex Still failing.. but I am not sure why.. do you have any ideas?

@nuex
Copy link
Contributor Author

nuex commented Mar 9, 2014

@jwilberding this is green now.

@tsloughter
Copy link
Member

Wooo! @jwilberding I'll leave merging to you in case there is anything you wanted to verify :)

jwilberding added a commit that referenced this pull request Mar 11, 2014
Make scripts in rlx_prv_assembler DTL templates
@jwilberding jwilberding merged commit 6c962c9 into erlware:master Mar 11, 2014
@jwilberding
Copy link
Member

Thanks for working through that @nuex :)

@nuex
Copy link
Contributor Author

nuex commented Mar 11, 2014

yay!

@nuex nuex deleted the dtl_scripts branch March 11, 2014 13:43
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