search

elixir-ecto / ecto

A toolkit for data mapping and language integrated query.
https://hexdocs.pm/ecto
Apache License 2.0
6.2k stars 1.44k forks source link

When used with Dynamo, app_dir returns something less than ideal #111

Closed knewter closed 11 years ago

knewter commented 11 years ago 
--(jadams@jibbajabba)-(73/pts/4)-(0750/06-Nov-13)--
--($:~/elixir/dwitter)(feature/prep_episode_25)ruby-2.0.0-p247@system-- 
$ mix ecto.gen.migration Dwitter.Repo create_dweets
mix.exs:39: redefining module Mix.Tasks.Release_docs
* creating tmp/dev/dwitter/priv/repo/migrations
* creating tmp/dev/dwitter/priv/repo/migrations/20131106135013_create_dweets.exs

This happened on this repository: https://github.com/knewter/dwitter/tree/9f0abf1521fe4c170893fd6a4379d1dfa650c2e7

knewter commented 11 years ago

To be clear: I'm completely unsure if (a) this is the right place to file this issue, or (b) what a proper fix might look like. I'd have expected to end up with migrations in ./priv/ like I would have in a non-dynamo app.

josevalim commented 11 years ago

Yup, we are aware. :) This is a Dynamo bug and it will be fixed on Elixir v0.11.2, once we move the compilation directory to be _build.

knewter commented 11 years ago

Awesome, good to know :)

On Wed, Nov 6, 2013 at 8:01 AM, José Valim notifications@github.com wrote:

Yup, we are aware. :) This is a Dynamo bug and it will be fixed on Elixir v0.11.2, once we move the compilation directory to be _build.

— Reply to this email directly or view it on GitHubhttps://github.com/elixir-lang/ecto/issues/111#issuecomment-27875103 .

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

ericmj commented 11 years ago

@josevalim Updates on this? Can we fix it now? I'm not 100% clear what the issue is here.

josevalim commented 11 years ago

Oh, this is fixed on Dynamo master already!