Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

erlang: consider `brew --prefix` as root dir #16146

Closed
wants to merge 1 commit into from

3 participants

Dirk Hörner Adam Vandenberg Yousef Ourabi
Dirk Hörner

By default, erlang check's it's #{prefix} for installed applications,
however 3rd-party applications typically install to brew --prefix
instead.

This commit fixes this by crudely overwriting ROOTDIR in
lib/erlang/bin/{erl,start}.

Dirk Hörner dirker erlang: consider `brew --prefix` as root dir
By default, erlang check's it's #{prefix} for installed applications,
however 3rd-party applications typically install to `brew --prefix`
instead.

This commit fixes this by crudely overwriting ROOTDIR in
lib/erlang/bin/{erl,start}.
6e02968
Dirk Hörner

bump

Adam Vandenberg
Owner

I don't know enough to know if this is considered a good idea or not.

Dirk Hörner

I have written an email to the erlang-questions list to get more information: https://groups.google.com/forum/?fromgroups=#!topic/erlang-programming/rFExtQpVuiQ

Yousef Ourabi

It seems like the consensus from that thread would be against this change... do you still think it is the best course of action?

Adam Vandenberg
Owner

Passing on this.

Adam Vandenberg adamv closed this
Dirk Hörner

I still think the issue persists in homebrew and will likely hit other erlang users/developers sooner or later, but since my solution certainly is not brilliant and what they recommend in the thread is working for me, passing on this is the right thing to do.

Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Nov 19, 2012
  1. Dirk Hörner

    erlang: consider `brew --prefix` as root dir

    dirker authored
    By default, erlang check's it's #{prefix} for installed applications,
    however 3rd-party applications typically install to `brew --prefix`
    instead.
    
    This commit fixes this by crudely overwriting ROOTDIR in
    lib/erlang/bin/{erl,start}.
This page is out of date. Refresh to see the latest.
Showing with 4 additions and 0 deletions.
  1. +4 −0 Library/Formula/erlang.rb
4 Library/Formula/erlang.rb
View
@@ -86,6 +86,10 @@ def install
system "make"
system "make install"
+ # fixup pathes in erlang wrappers
+ inreplace ["#{lib}/erlang/bin/erl", "#{lib}/erlang/bin/start"],
+ "#{prefix}", HOMEBREW_PREFIX
+
unless build.include? 'no-docs'
manuals = build.head? ? ErlangHeadManuals : ErlangManuals
manuals.new.brew { man.install Dir['man/*'] }
Something went wrong with that request. Please try again.