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

Script problem #2

Closed
Madis0 opened this issue Oct 28, 2017 · 3 comments
Closed

Script problem #2

Madis0 opened this issue Oct 28, 2017 · 3 comments

Comments

@Madis0
Copy link

Madis0 commented Oct 28, 2017

The native client won't work because the add-re-style.sh doesn't.

SyntaxError: Unexpected token f in JSON at position 1 at JSON.parse (<anonymous>) at __dirname.filter.map.readFile.then.config (/snapshot/native-ext/install.js:0:0) at <anonymous>

Note that I still got a success message. Using Nightly 58.0a1 (2017-10-16) on Ubuntu 17.10.

Maybe it doesn't find the profile? Mine's located at /home/username/.mozilla/firefox-trunk/xyz.default

@NiklasGollenstede
Copy link
Owner

The error you are seeing is because I didn't consider that Linux shells treat quotes in commands different from the Windows cmd. The result is that the JSON file the script writes isn't actually JSON. The latest beta (0.2.1b4) should produce working scripts (and circumvents the chmod requirement).

Note that I still got a success message.

Yupp. It successfully wrote ... nothing. If you had had any more files in the vendors/ folder, it would have processed them. I should change the message.

located at /home/username/.mozilla/firefox-trunk/xyz.default

As long as Firefox still looks for native apps in ~/.mozilla/native-messaging-hosts that is fine.

@Madis0
Copy link
Author

Madis0 commented Oct 28, 2017

Tried the beta out, now the startup warning is gone and styles are installable. However, when installing one, I still get an error (even though it will get listed to my styles):

SyntaxError: JSON.parse: unexpected character at line 1 column 1 of the JSON data

I tried these styles, by the way.

@NiklasGollenstede
Copy link
Owner

Once running, NativeExt itself doesn't read any JSON, so this is probably a problem with reStyle. I'll create an issue there.

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

No branches or pull requests

2 participants