Skip to content

Commit

Permalink
wrong path for env var
Browse files Browse the repository at this point in the history
  • Loading branch information
@binhex
binhex committed Jan 1, 2019
1 parent 65ca9cb commit 38cd10f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion build/root/install.sh
View file
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,7 @@ export RIDER_PROPERTIES=$(echo "${RIDER_PROPERTIES}" | sed -e 's~^[ \t]*~~;s~[ \
if [[ ! -z "${RIDER_PROPERTIES}" ]]; then
echo "[info] RIDER_PROPERTIES defined as '${RIDER_PROPERTIES}'" | ts '%Y-%m-%d %H:%M:%.S'
else
export RIDER_PROPERTIES="/home/nobody/.config/rider/idea.properties"
export RIDER_PROPERTIES="/home/nobody/.config/rider/config/idea.properties"
echo "[info] RIDER_PROPERTIES not defined, defaulting to '${RIDER_PROPERTIES}'" | ts '%Y-%m-%d %H:%M:%.S'
fi
EOF
Expand Down

3 comments on commit 38cd10f

@bbakermmc
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this default to /config/rider/config/idea.properties.
And put the "standard" config file there?

Also add this:
If you do not have write access to the JetBrains Rider configuration directory, you can add the RIDER_VM_OPTIONS

https://www.jetbrains.com/help/rider/Tuning_the_IDE.html

@bbakermmc
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe the vm options thing isnt needed. It looks to be picking up my rider64.vmoptions file in the /config/fider/config folder.

@binhex
Copy link
Owner Author

@binhex binhex commented on 38cd10f Jan 2, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this default to /config/rider/config/idea.properties.
And put the "standard" config file there?

i didn't want to expose the idea.properties file, as there shouldnt be any need to change this, it will cause support headaches if people start messing with the paths set.

Please sign in to comment.