Compilation error: "attribute 'dialyzer' after function definitions"

[henrik]

I get this trying to mix deps.compile floki on my OS X machine:

/usr/local/Cellar/erlang/18.0.2/lib/erlang/lib/parsetools-2.1/include/leexinc.hrl:47: attribute 'dialyzer' after function definitions
/usr/local/Cellar/erlang/18.0.2/lib/erlang/lib/parsetools-2.1/include/leexinc.hrl:118: attribute 'dialyzer' after function definitions
/usr/local/Cellar/erlang/18.0.2/lib/erlang/lib/parsetools-2.1/include/leexinc.hrl:194: attribute 'dialyzer' after function definitions
/usr/local/Cellar/erlang/18.0.2/lib/erlang/lib/parsetools-2.1/include/leexinc.hrl:247: attribute 'dialyzer' after function definitions
==> floki
could not compile dependency floki, mix compile failed. You can recompile this dependency with `mix deps.compile floki` or update it with `mix deps.update floki`

[philss]

@henrik and @licyeus can you provide your Elixir and Erlang versions?

To check Erlang version with brew, use: brew info erlang | grep "*"
To check Elixir version, run: elixir -v.

[sllvn]

➜  brew info erlang | grep "*"
/usr/local/Cellar/erlang/17.5 (7398 files, 282M) *
➜  elixir -v
Elixir 1.0.5

Interesting that my stacktrace references 18.0.2, but brew states I'm running 17.5. I'm upgrading erlang with homebrew and will let you know after lunch if it works.

[henrik]

$ brew info erlang | grep "*"
/usr/local/Cellar/erlang/17.5 (7398 files, 283M) *
$ elixir -v
Elixir 1.0.4

I'll try to hold off on upgrading things so I can evaluate any backwards-compatibility fix (if that's the issue).

[philss]

@licyeus @henrik thank you! I will take a look later today. This seems to be something related to the Erlang version , but I'm not sure. I tested Floki against Erlang 18.0.2 and 18.0.3, and it works fine.

@henrik Ok.

[sllvn]

Upgrading to erlang 18.0.3 (via brew upgrade erlang) fixed the issue. Floki now compiles successfully. @philss, thanks for the heads up on how to check system erlang version.

[henrik]

👍 If you bump the version I'm happy to try the latest.

Sent from my iPhone

On 18 Sep 2015, at 06:35, Philip Sampaio notifications@github.com wrote:

Closed #24 via #26.

—
Reply to this email directly or view it on GitHub.

[philss]

@henrik great! You can take a look now in the new version 0.4.1 (it's already published).

My bad, guys. The problem was that I added a file that shouldn't be added to the Hex package. You can't see the file because is not under versioning, but is a file that is generated by the "leex" compiler.
This explain why you were seeing "/usr/local/Cellar/erlang/18.0.2/".

Let me know if you have any issues.

Thanks again for reporting!

cc @licyeus

[philss]

@henrik can you try this new version (0.4.1)?

[henrik]

Works! 👍 Thanks so much, Philip!

Sent from my iPhone

On 18 Sep 2015, at 20:28, Philip Sampaio notifications@github.com wrote:

@henrik can you try this new version (0.4.1)?

—
Reply to this email directly or view it on GitHub.

[philss]

@henrik Ok, thanks for testing it!