Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Modified interface to personal-files to allow access to hidden files/folders #5

Closed
wants to merge 1 commit into from

Conversation

Staberinde
Copy link

@Staberinde Staberinde commented Jun 30, 2019

The home interface on snap provides access to non-privileged user files/folders in the user's home directory. But sometimes it is necessary for hidden files (i.e. files prepended by .).

Personally my reason for needing this change is that I use the universal-ctags snap in conjunction with the neotags plugin for neovim, which requires read/write access to $HOME/.vim-tags. This currently fails with permission errors

This provides a fix for the following issues:

@@ -12,7 +12,7 @@ apps:
universal-ctags:
command: ctags
plugs:
- home

Choose a reason for hiding this comment

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

You probably still want to specify the home interface so that ctags may access source files in your home folder. Removing this interface will prevent ctags from reading any files other than those exposed by the personal-files interface (see next comment)

@@ -12,7 +12,7 @@ apps:
universal-ctags:
command: ctags
plugs:
- home
- personal-files

Choose a reason for hiding this comment

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

You need to add a top-level plugs block that details which personal files you want to be exposed by your usage of the personal-files interface. You also need to call it something other than personal-files. See the documentation for more information.

@tartley
Copy link
Collaborator

tartley commented Nov 14, 2019

I'm closing this - the equivalent changes have since been merged. Many thanks for the participation.

@tartley tartley closed this Nov 14, 2019
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.

3 participants