These are chat archives for ensime/ensime-atom

27th
Nov 2015
Viktor Hedefalk
@hedefalk
Nov 27 2015 06:40
@mdedetrich Suggestions in issue.
Matthew de Detrich
@mdedetrich
Nov 27 2015 06:44
@hedefalk Updated issue
Matthew de Detrich
@mdedetrich
Nov 27 2015 10:53
@hedefalk I am still getting the whole .ensime_cache/server.log problem
Arnout Engelen
@raboof
Nov 27 2015 11:17
@mdedetrich so currently your problem is that ensime-atom only works when your project path is the directory containing the .ensime folder, right? Or something else too?
Matthew de Detrich
@mdedetrich
Nov 27 2015 11:17
@raboof Its not starting in a standard SBT project
Arnout Engelen
@raboof
Nov 27 2015 11:18
ah the original problem in ensime/ensime-atom#101 (first message there)?
Viktor Hedefalk
@hedefalk
Nov 27 2015 12:00
@mdedetrich Please try to reproduce in a standalone github project.
Matthew de Detrich
@mdedetrich
Nov 27 2015 12:01
@hedefalk Will do, its a company project so I will try and replicate it
Boris Chazalet
@bchazalet
Nov 27 2015 14:51
ensime seems to insist on formatting my files (via scalariform) with different rules than my own.. how could I prevent that?
Ghost
@ghost~540393fe163965c9bc2018ce
Nov 27 2015 20:57
@bchazalet it uses the rules in your scalariform-sbt config
although you might have different versions of scalariform in sbt than ensime
this is a long standing issue, stemming from the fact that the scalariform author just walked away from scala one day
I've wasted several man days trying to track him down
hopefully scala-ide will take ownership of scalariform in january+
Boris Chazalet
@bchazalet
Nov 27 2015 21:24
@fommil thanks for the info. I think it comes down to my projects using a different version of scalariform indeed.
for now, I am working around it by forking ensime-sbt and using my own jar (with scalariform downgraded)
Boris Chazalet
@bchazalet
Nov 27 2015 21:29
so I look forward to the scalariform re-unification
Ghost
@ghost~540393fe163965c9bc2018ce
Nov 27 2015 21:31
uuh, that doesn't sound right. if anything, ensime will be using a much more recent version
which version of scalariform sbt are you using?
btw, I would recommend just copying and pasting EnsimePlugin rather than bothering with publishing local
the sbt plugin artefact stuff is basically insane and an exercise in ivy wanking
Boris Chazalet
@bchazalet
Nov 27 2015 21:32
we're using "com.typesafe.sbt" % "sbt-scalariform" % "1.3.0" in our projects
Ghost
@ghost~540393fe163965c9bc2018ce
Nov 27 2015 21:32
I actually wanted to refactor into a single file to make this easier (I need to do this in work)
which version of scalariform does that use?
ensime-sbt is using the org.scalariform sbt-scalariform (i know, don't ask, it's a nightmare.. at least there are only two now instead of three)
so I think you're actually on an old version there.
but there is a regression in the recent versions (yay!) which won't be fixed until january
this is really the sort of thing that typesafe should be fixing
Boris Chazalet
@bchazalet
Nov 27 2015 21:34
yes, that's what I meant
i forked ensime-sbt to downgrade it to an older version of sbt-scalariform
Ghost
@ghost~540393fe163965c9bc2018ce
Nov 27 2015 21:35
oh, I see, I thought you meant ENSIME was doing the reformatting
over the RPC call
Boris Chazalet
@bchazalet
Nov 27 2015 21:35
no, it's sbt via sbt-ensime
Ghost
@ghost~540393fe163965c9bc2018ce
Nov 27 2015 21:35
this isn't ensime doing the reformatting, it's just an upgraded scalariform-sbt
no, it's not ensime-sbt, it's scalariform-sbt doing it
lemme find the regression...
Boris Chazalet
@bchazalet
Nov 27 2015 21:36
but not using my rules somehow
i guess the real fixing would be to upgrade my scalariform
Ghost
@ghost~540393fe163965c9bc2018ce
Nov 27 2015 21:36
it is, join the queue of people trying to get the maintainer to merge this fix daniel-trinh/scalariform#80
Boris Chazalet
@bchazalet
Nov 27 2015 21:37
but we don't want yet because of that DandlingParenthesis change
yes, that one
Ghost
@ghost~540393fe163965c9bc2018ce
Nov 27 2015 21:37
yes, this is what happens when somebody who doesn't care about introducing regressions takes maintainership of a project that every single project in the scalaverse depends on
so, like I said, htis is something that typesafe should really have fixed as a priority
it's actually a blocker for people upgrading scala-ide
Boris Chazalet
@bchazalet
Nov 27 2015 21:38
yeah, I understand it's not ensime's fault
i just wanted a quick fix to be able to use atom-ensime
Ghost
@ghost~540393fe163965c9bc2018ce
Nov 27 2015 21:38
I've been pestering them for months (years?)
try voting here sbt/sbt-scalariform#20
don't expect a response ;-)
Boris Chazalet
@bchazalet
Nov 27 2015 21:39
ahah
but the regression you mentionned, it's not the only formatting that i see happening
Ghost
@ghost~540393fe163965c9bc2018ce
Nov 27 2015 21:40
but the non-typesafe scala-ide chaps are good, so hopefully we'll get this fixed
Boris Chazalet
@bchazalet
Nov 27 2015 21:40
it alignes the arguments of a class for example
which isn't following our rules
Ghost
@ghost~540393fe163965c9bc2018ce
Nov 27 2015 21:40
uuuugh, why am I not surprised...
sounds like he broke something else too then
unfortunately only the regressed version of scalariform is available for 2.10 and 2.11, so we have to use it in ensime-server
Boris Chazalet
@bchazalet
Nov 27 2015 21:41
well, from sbt point's of view there are two sbt-scalariform doing the same job no?
sbt-ensime brings one
and i have mine
(different organizations)
that can't be any good
yeah, try telling typesafe
they were completely unresponsive when I was flagging this up as a concern
daniel used to publish his own, and he still does, but its identical to the org.scalariform one
I tried to get him to publish a pom-only release that tried to rationalise the dependencies, but he wasn't interested
sorry for ranting :-) I really have literally spent days of my time on this
Boris Chazalet
@bchazalet
Nov 27 2015 21:43
i know, i have seen your comments all over the place
ahah
Ghost
@ghost~540393fe163965c9bc2018ce
Nov 27 2015 21:43
pretty much every single critical person who needs to do something is unresponsive, and it is very frustrating
Boris Chazalet
@bchazalet
Nov 27 2015 21:43
well, i'll continue with my ugly hack and try again when the DanglingParenthensis PR is merged
Ghost
@ghost~540393fe163965c9bc2018ce
Nov 27 2015 21:44
matt russell, daniel trinh and typesafe... all on a perpetual holiday as far as I can tell
I'd still strongly recommend that you just copy/paste ensime-sbt into your project dir and be done with it
Boris Chazalet
@bchazalet
Nov 27 2015 21:45
well, isn't the problem that those a maintain by a single person?
Ghost
@ghost~540393fe163965c9bc2018ce
Nov 27 2015 21:45
you mean mdr/scalariform#137 ? :-)
Boris Chazalet
@bchazalet
Nov 27 2015 21:46
i'll try that.. but my jar is already published internally so I know I am going to be lazy on that one
ahah, i see you've been there before
Ghost
@ghost~540393fe163965c9bc2018ce
Nov 27 2015 21:46
what you don't see are the 10s of emails I've sent to these people
but I finally got daniel to say that he'd be happy transferring to scala-ide and the good chaps there will take it
Boris Chazalet
@bchazalet
Nov 27 2015 21:47
:thumbsup: that's good news
Ghost
@ghost~540393fe163965c9bc2018ce
Nov 27 2015 21:47
(been a long time coming)
FWIW this is also a problem for me at work, so I will be pushing and reminding scala-ide about it!
I actually copy / paste EnsimePlugin at work with an old scalariform :-)
but the server formatting is all wrong
Boris Chazalet
@bchazalet
Nov 27 2015 21:50
yeah, I must say I didn't think of copy pasting
somehow publishing my own jar came first to my mind
Ghost
@ghost~540393fe163965c9bc2018ce
Nov 27 2015 21:51
this'll make it easier ensime/ensime-sbt#115
this way you don't need to hack around with the scalariform
actually I was going to work on this ticket tonight, but then my wife opened a bottle of wine :-D
so concentration is waning
Boris Chazalet
@bchazalet
Nov 27 2015 21:53
so is it taking those 4 files and merging them into one?
Ghost
@ghost~540393fe163965c9bc2018ce
Nov 27 2015 21:53
pretty much, yup!
Boris Chazalet
@bchazalet
Nov 27 2015 21:53
let me see if I can make you a PR then
Ghost
@ghost~540393fe163965c9bc2018ce
Nov 27 2015 21:53
:-D
Ghost
@ghost~540393fe163965c9bc2018ce
Nov 27 2015 22:00
btw, @bchazalet don't forget to sbt gen-ensime-meta so you can uses ENSIME when hacking your sbt files
(the project/*.scala ones anyway)
Viktor Hedefalk
@hedefalk
Nov 27 2015 22:01
@fommil Are the .scala sbt files gonna be removed or were the protests loud enough?
Ghost
@ghost~540393fe163965c9bc2018ce
Nov 27 2015 22:02
@hedefalk hopefully the protests were strong enough and now they have a loud customer telling them to stop being so stupid :-)
(which will also hopefully convince them to fix the ridiculous performance problems)
(instead of creating crazy :sparkles: Reactive :sparkles: frameworks that nobody really cares about)
Viktor Hedefalk
@hedefalk
Nov 27 2015 22:05
You grumpy man :)
Ghost
@ghost~540393fe163965c9bc2018ce
Nov 27 2015 22:08
:-P
Boris Chazalet
@bchazalet
Nov 27 2015 22:11
gotta go, but thanks for the info @fommil