Skip to content

Rebarified #2

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

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Rebarified #2

wants to merge 1 commit into from

Conversation

nisbus
Copy link

@nisbus nisbus commented Feb 17, 2013

Hi,

This is a minor change, basically just rebarifying erlog so it can be used as a rebar dependency.

Added rebar.config, rebar and erlog.app.src for enabling others to use erlog as a rebar dependency.
Changed the Makefile to use rebar.

Added rebar.config and erlog.app.src for enabling others to use erlog as a rebar dependency.
Changed the Makefile to use rebar
@rvirding
Copy link
Owner

I will pull this later. It is good you attacked Makefile, the old one was pretty bad. I would like to make it so it uses rebar if it can find rebar, else uses erlc. The lfe Makefile does this but there is probably a better way to do it.

@nisbus
Copy link
Author

nisbus commented Feb 17, 2013

I think the convention is to include rebar with the repo (I'm not so sure I
like that myself) so it is a part of the branch there so it will always
find it.

On Sun, Feb 17, 2013 at 4:07 PM, Robert Virding [email protected]:

I will pull this later. It is good you attacked Makefile, the old one was
pretty bad. I would like to make it so it uses rebar if it can find rebar,
else uses erlc. The lfe Makefile does this but there is probably a better
way to do it.


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

@rvirding
Copy link
Owner

Looking back over old stuff and saw this. The reason why I never pulled it was because it didn't have the option of using either rebar or normal make if there was no rebar. I don't see rebar as the answer. :-)

@zkessin
Copy link
Contributor

zkessin commented May 29, 2014

My branch has a makefile that handles both cases, so maybe we should just close this PR?

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