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

Checking settings file existence before attempting to read them #23

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Gijom
Copy link

@Gijom Gijom commented Apr 21, 2016

Thanks for this great plugin. However after installation I was constantly having troubles with errors of the type ENOENT "Could not find file /path/to/local/config/litexrc and /path/to/latexfile/.litexrc". Of course creating those files would resolve the problem but why bothering when the plugin works out of the box ?

Solution:
I just added a line for testing the existence of these files before opening them in the function load-settings and used the files/file? LT function for that purpose.

@rschroll
Copy link
Owner

Out of curiousity, are you using LT 0.8? This isn't a problem in 0.7,
so I'm wondering if this was introduced in the switch to Electron.

@Gijom
Copy link
Author

Gijom commented Apr 21, 2016

Yes I do have LT 0.8.1. Note that the plugin was working despite of the error messages (6 error messages in total, 3 per local/wide settings). The proposed modification solved the problem (at least for me).

@rschroll
Copy link
Owner

Glad to hear it's (mostly) working. I still haven't gotten around to
the upgrade, and I wasn't sure if anything would be broken. I'll merge
in the change shortly.

Thanks!

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.

2 participants