search

OndraFTW / seagull

Simple GUI library for Elixir language based on wxErlang
64 stars 13 forks source link

After making this accept the latest elixir, it failes to compile the demos #3

Closed knewter closed 10 years ago

knewter commented 10 years ago

I forked this and removed the restriction on the elixir version (so it would run on master). After doing that, ti fails to compile the demos. It's related to some macro expansion stuff that I couldn't quite follow. I managed to tweak define_line/3 to look like this:

  defp define_line({:from, _, [[pid: pid], [do: {:from, _, [[widget: widget, do: [{:"->", _, list}]]]}]]}, acc) do
        add_pid_and_widget(list, [pid: pid, widget: widget])++acc
  end

The change in the arguments seemed to be due to changes in macro-related things in elixir. However, after doing that I got failures from add_pid_and_widget that I can't reconcile :-\ Was hoping you could help me figure it out.

I want to do an episode on http://elixirsips.com re: using this library to build an elixir-based gui, but can't until I can get it working. If you don't have time, I was going to fork it and try to figure it out, but it'll inevitably take me a while. For what it's worth, building a GUI works fine, but receiving events is where the failures are.

Anyway, thanks for the library - it looks pretty nice, if I could get it to work :-\

OndraFTW commented 10 years ago

Elixir devs changed how -> operator is represented in AST.

Fixed in bfba80b010a76d56db57ea30d7f48b20b51c82fb.

knewter commented 10 years ago

Thanks, really looking forward to playing with this :)

On Fri, Jan 3, 2014 at 2:49 PM, Ondřej Šlampa notifications@github.comwrote:

Fixed in bfba80bhttps://github.com/oslampa/seagull/commit/bfba80b010a76d56db57ea30d7f48b20b51c82fb

— Reply to this email directly or view it on GitHubhttps://github.com/oslampa/seagull/issues/3#issuecomment-31552435 .

Josh Adams CTO | isotope|eleven http://www.isotope11.com cell 215-3957 work 476-8671 x201