elm-lang / elm-package

Command line tool to share Elm libraries
BSD 3-Clause "New" or "Revised" License
214 stars 66 forks source link

Call elm-make with --yes to avoid failure #140

Closed laszlopandy closed 9 years ago

laszlopandy commented 9 years ago

I have not tested this change, but it seems like it should solve the problem that I experienced today:

$ elm-package diff

Error: failure when running: elm-make --docs=elm-stuff/documentation.json
elm-make: <stdin>: hGetLine: end of file
Some new packages are needed. Here is the upgrade plan.

  Install:
    elm-lang/core 2.1.0

Do you approve of this plan? (y/n) 

$

Because I tried to do a diff immediately after cloning the repo, there was nothing in elm-stuff. Notice that although it prints out Do you approve of this plan? (y/n) you cannot type; the program already crashed by that point and returned you to your shell.

jvoigtlaender commented 9 years ago

Does this by chance also fix https://github.com/elm-lang/elm-package/issues/137?

laszlopandy commented 9 years ago

@jvoigtlaender seems like it will.