Problem with imports (wrappers?)

[alexarchambault]

Using the current master branch, I get

Loading Ammonite Repl...

@ load.ivy("com.chuusai" %% "shapeless" % "2.1.0")
:: resolving dependencies :: com.chuusai#shapeless_2.11-caller;working
    ...
res0: Unit = ()

@ import shapeless._
import shapeless._

@ () // Nothing compiles starting from here, whatever the line we enter
Compilation Failed
Main.scala:1136: '}' expected but identifier found.
  ~?>$,
     ^

@ 

(re-formatted/stripped for readability).

The wrapper fails to compile once one imported shapeless._

[lihaoyi]

Definitely a problem with the wrapper generation, shouldn't be too hard to fix once i have time to take a look

[alexarchambault]

Probably related to #35

[lihaoyi]

Don't try to fix this yet, I have a bunch of WIP improving the import handling (and tests) that I haven't pushed

[lihaoyi]

I pushed a576aa4 which is still kinda/sorta WIP but might (maybe?) fix some of these issues. Will take a look later

[alexarchambault]

Hope they don't conflict too much with my last PR... :)

[alexarchambault]

Ok, conflict =|

[alexarchambault]

@lihaoyi Not fixed :-| Last error is now

@ ()
Compilation Failed
Main.scala:1136: '}' expected but identifier found.
  ~?>$ => ~?>$,
     ^

[lihaoyi]

This particular one looks like an issue with the backtickWrap function not properly wrapping this, and perhaps a problem in importing things we shouldn't import (Private? Synthetic? Who knows?).

In theory we could fall-back to the old behavior and just backtick-wrap everything, but I'd prefer to instead fix-forward and fix the backtick-wrapper. The parboiled parser must be correct and if it's not we should fix it until it is (like @paulp has done for his) and import fixing can't be that hard

[lihaoyi]

This should work now https://github.com/lihaoyi/Ammonite/blob/master/repl/src/test/scala/ammonite/repl/AdvancedTests.scala#L184-L193