Fork me on GitHub
#cursive
<
2018-08-01
>
cfleming00:08:31

@kenny They should be if you use that alias to sync the project. Is that not working for you?

kenny21:08:59

It was not for the project I was testing. I'll double check tomorrow when I get back on that computer.

cfleming22:08:18

Ok thanks, I’ll see if I can repro today too.

kenny22:08:56

I know one interesting aspect of that particular project was that the dependencies were circular. A -> B -> A. B is included under an alias defined in A and A is in the :exclusions of B.

cfleming22:08:44

That is… “interesting” 🙂

😝 4
cfleming00:08:23

@kenny So the Cursive code currently won’t work for something that funky - it doesn’t handle circular deps at all.

cfleming00:08:19

That’s because it does an initial pass to gather all the modules before resolving them all using tools.deps. So it doesn’t have the level of detail of something like :exclusions

cfleming00:08:01

If you think it’s important to fix that, I might be able to do something - let me know.

kenny00:08:14

I'll describe the reason for the circular dependency to let you decide the level of importance. We have a DB utility project and a test-components project. test-components is only ever included in a project via an alias (typically for testing). test-components has a clojure.test fixture that initializes the database with a set of test data. We then test the DB utility project against this test data. If it's possible to detect, I'd say you should at least show some sort of warning that behavior will not be what you expect (i.e. changing a file in a local dependent project will not be recognized in the REPL).

cfleming03:08:29

Cursive 1.8.0-eap4 is out now! If you’re on Windows you can now use deps in Cursive, let me know how it works for you.

cfleming03:08:46

@tbaldridge @potetm This contains your fix too, have at it!

oc15:08:54

Has anyone seen this issue? On the latest intellij + cursive versions. 10:56 AM facade.password must not be null: facade.password must not be null 10:59 AM Error running 'REPL for <project name>': facade.password must not be null

cfleming22:08:38

@oc No, I’ve never seen that. That’s not something I recognise from Cursive, could that be from a lein plugin?

oc13:08:58

I do not have any lein plugins added to the project. here's the stacktrace if it's helpful. java.lang.IllegalStateException: facade.password must not be null at cursive.leiningen.LeiningenUtil.standardArgs(LeiningenUtil.kt:116) at cursive.leiningen.project.LeiningenProjectDetails$getProjectDetails$1.invoke(ProjectDetails.kt:123) at cursive.leiningen.project.LeiningenProjectDetails$getProjectDetails$1.invoke(ProjectDetails.kt:103) at cursive.leiningen.project.LeinShimProvider.withShim(ProjectDetails.kt:51) at cursive.leiningen.project.LeiningenProjectDetails.getProjectDetails(ProjectDetails.kt:117) at cursive.leiningen.project.LeiningenProjectsManager$reimportAllProjects$task$1.run(LeiningenProjectsManager.kt:131) at cursive.leiningen.LeiningenUtil$runTask$1$1.run(LeiningenUtil.kt:76) at com.intellij.openapi.progress.impl.CoreProgressManager$TaskRunnable.run(CoreProgressManager.java:736) at com.intellij.openapi.progress.impl.CoreProgressManager.lambda$runProcess$1(CoreProgressManager.java:157) at ... here's the project.clj if thats helpful

oc13:08:42

It's been happening for a few days when I upgraded to the latest intellij. is there a way to 'reinstall' cursive? I removed the plugin/restarted/reinstalled/restarted but it didnt help, not sure if there's anything left on the disk after an uninstall. my friend can run the project with cursive without issues.

kenny23:08:28

I forked Depot and opened it as a Deps project and all the defs are marked as "cannot be resolved." Any idea why?

kenny23:08:00

Tried opening and closing the project and refreshing the deps. No dice.

cfleming23:08:01

No, that’s very strange.

cfleming23:08:09

Is this an open source project?

cfleming23:08:22

Does invalidating indexes help?

kenny23:08:40

I'll try.

kenny23:08:03

Always annoying to do that because I need to close out all my REPLs 😞

cfleming23:08:29

Yeah, sorry about that 😞

kenny23:08:15

Yeah that fixed it.

cfleming23:08:30

Sorry about the REPLs though.

kenny23:08:05

Gave me a chance to get up and stretch, so it's all good 🙂