search

NiklasGollenstede / native-ext

A node.js endpoint for Chrome and Firefox to connect to via Native Messaging
MIT License
56 stars 3 forks source link

Script problem #2

Closed Madis0 closed 6 years ago

Madis0 commented 6 years ago

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 commented 6 years ago

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 commented 6 years ago

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 commented 6 years ago

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