Skip to content

Conversation

@blueyed
Copy link
Member

@blueyed blueyed commented Oct 8, 2016

This is required for Neovim to suppress errors that it cannot write to
~/.local etc.

This is required for Neovim to suppress errors that it cannot write to
~/.local etc.
@blueyed blueyed merged commit e1bc2a1 into Vimjas:master Oct 8, 2016
@blueyed blueyed deleted the run_vim-separate-home branch October 8, 2016 22:11
# Vim as a user that's unable to write to your volume.
exec su -l vimtest -c "cd /testplugin && /vim-build/bin/$BIN $ARGS"
cd /testplugin || exit
exec su -l vimtest -c "env HOME=/tmp/vimtestbed-home /vim-build/bin/$BIN $ARGS"
Copy link
Member Author

Choose a reason for hiding this comment

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

This is broken - the su -l will change to the $HOME of the user, will fix it.

Copy link
Member Author

Choose a reason for hiding this comment

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

Reverted in #16.

blueyed added a commit to blueyed/vim-testbed that referenced this pull request Oct 9, 2016
This reverts commit e1bc2a1.

This could be made working with the following, but then the vimrc from
the HOME gets not picked up probably etc, so there needs to be another
fix for the Neovim case:

    exec su -l vimtest -c "env HOME=/tmp/vimtestbed-home \
      sh -c 'cd /testplugin && /vim-build/bin/$BIN $ARGS'"

Fixes https://github.com/tweekmonster/vim-testbed/pull/11/files#r82528988.
blueyed added a commit that referenced this pull request Oct 9, 2016
This reverts commit e1bc2a1.

This could be made working with the following, but then the vimrc from
the HOME gets not picked up probably etc, so there needs to be another
fix for the Neovim case:

    exec su -l vimtest -c "env HOME=/tmp/vimtestbed-home \
      sh -c 'cd /testplugin && /vim-build/bin/$BIN $ARGS'"

Fixes https://github.com/tweekmonster/vim-testbed/pull/11/files#r82528988.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant