cytechmobile / radicle-jetbrains-plugin

Radicle plugin for Jetbrains IDEs
https://plugins.jetbrains.com/plugin/19664-radicle
Apache License 2.0
15 stars 0 forks source link

Git push to different RAD_HOME #446

Closed Stelios123 closed 4 months ago

Stelios123 commented 10 months ago

When RAD_HOME is anything but the default, the git process started by IntelliJ ignores the RAD_HOME configured in the IDE.

All git operations of the radicle-remote-helper should take that into account, so we need to find out a way to pass this information to spawned git processes.

gsaslis commented 6 months ago

Let's check whether this will work by passing RAD_HOME as an environment variable to IntelliJ (vmoptions or something)

gsaslis commented 5 months ago

https://intellij-support.jetbrains.com/hc/en-us/articles/206544869-Configuring-JVM-options-and-platform-properties

gsaslis commented 5 months ago

Let's fix this for linux and mac at least and just investigate and document whether it is possible for windows.

JChrist commented 5 months ago

An update on this after spending some time investigating the situation. Custom VM options is not helping (in any OS), as we need to set an env var (not system properties), which is also not at all needed within the IDE itself, but is passed on to spawned git process. Possible solutions include:

gsaslis commented 5 months ago

the .profile route sounds like a good-enough compromise, considering that most people will just use the default RAD_HOME. We now have a way for the plugin to work for the ones who don't as well, so let's just go with that.