ff2mpv-rust doesn't launch on Windows #5

Closed
opened 2023-10-18 22:28:21 +02:00 by TheLongLife · 15 comments
TheLongLife commented 2023-10-18 22:28:21 +02:00 (Migrated from github.com)

Excuse my noob-iness.
I'm trying to get this working on Windows.

I have it working with the default client (python?) but it seems sluggish, how would I go about switching it with the rust one?

I tried following the installation steps but it's not very clear to me what to do after I use the ff2mpv-rust manifest command.
Should I swap the contents of ff2mpv-windows.json with the output text I get after running the manifest command? (I've tried that but it doesn't work).
I also confirmed that the config is valid.

Excuse my noob-iness. I'm trying to get this working on Windows. I have it working with the default client (python?) but it seems sluggish, how would I go about switching it with the rust one? I tried following the installation steps but it's not very clear to me what to do after I use the `ff2mpv-rust manifest` command. Should I swap the contents of **ff2mpv-windows.json** with the output text I get after running the manifest command? (I've tried that but it doesn't work). I also confirmed that the config is valid.
ryze312 commented 2023-10-19 01:48:37 +02:00 (Migrated from github.com)

Did you follow the steps outlined here? In your case, swapping out the contents of the JSON file should be enough.

What are the original contents of it and the output of ff2mpv-rust manifest?

Did you follow the steps outlined [here](https://github.com/woodruffw/ff2mpv/wiki/Installation-on-Windows#manual-installation)? In your case, swapping out the contents of the JSON file should be enough. What are the original contents of it and the output of `ff2mpv-rust manifest`?
TheLongLife commented 2023-10-19 02:53:38 +02:00 (Migrated from github.com)

In the meantime, I also tried it on Vivaldi browser and it works just fine there, but not in the Firefox.

Did you follow the steps outlined here?

Yes, that is exactly the instruction that I followed.

What are the original contents of it and the output of ff2mpv-rust manifest?

These were the original contents of ff2mpv-windows.json

{
    "name": "ff2mpv",
    "description": "ff2mpv's native manifest",
    "path": "ff2mpv.bat",
    "type": "stdio",
    "allowed_extensions": ["ff2mpv@yossarian.net"]
}

The output I get when I use the manifest command is:

{
  "name": "ff2mpv",
  "description": "ff2mpv's external manifest",
  "path": "C:\\Miscellaneous tools\\ff2mpv-master\\ff2mpv-rust.exe",
  "type": "stdio",
  "allowed_extensions": [
    "ff2mpv@yossarian.net"
  ]
}


This is what my JSON files look like:

This is the one for the Vivaldi (this one works):
ff2mpv-windows-chrome.json

{
    "name": "ff2mpv",
    "description": "ff2mpv's native manifest",
    "path": "C:\\Miscellaneous tools\\ff2mpv-master\\ff2mpv-rust.exe",
    "type": "stdio",
    "allowed_origins": ["chrome-extension://ephjcajbkgplkjmelpglennepbpmdpjg/"]
}

This one is for the Firefox (this one doesn't work):
ff2mpv-windows.json

{
    "name": "ff2mpv",
    "description": "ff2mpv's native manifest",
    "path": "C:\\Miscellaneous tools\\ff2mpv-master\\ff2mpv-rust.exe",
    "type": "stdio",
    "allowed_extensions": ["ff2mpv@yossarian.net"]
}

It works with the original client, "path": "ff2mpv.bat",

In the meantime, I also tried it on Vivaldi browser and it works just fine there, but not in the Firefox. >Did you follow the steps outlined [here](https://github.com/woodruffw/ff2mpv/wiki/Installation-on-Windows#manual-installation)? Yes, that is exactly the instruction that I followed. >What are the original contents of it and the output of ff2mpv-rust manifest? These were the original contents of **ff2mpv-windows.json** ``` { "name": "ff2mpv", "description": "ff2mpv's native manifest", "path": "ff2mpv.bat", "type": "stdio", "allowed_extensions": ["ff2mpv@yossarian.net"] } ``` The output I get when I use the _manifest_ command is: ``` { "name": "ff2mpv", "description": "ff2mpv's external manifest", "path": "C:\\Miscellaneous tools\\ff2mpv-master\\ff2mpv-rust.exe", "type": "stdio", "allowed_extensions": [ "ff2mpv@yossarian.net" ] } ``` ----- This is what my JSON files look like: This is the one for the Vivaldi (this one works): **ff2mpv-windows-chrome.json** ``` { "name": "ff2mpv", "description": "ff2mpv's native manifest", "path": "C:\\Miscellaneous tools\\ff2mpv-master\\ff2mpv-rust.exe", "type": "stdio", "allowed_origins": ["chrome-extension://ephjcajbkgplkjmelpglennepbpmdpjg/"] } ``` This one is for the Firefox (this one doesn't work): **ff2mpv-windows.json** ``` { "name": "ff2mpv", "description": "ff2mpv's native manifest", "path": "C:\\Miscellaneous tools\\ff2mpv-master\\ff2mpv-rust.exe", "type": "stdio", "allowed_extensions": ["ff2mpv@yossarian.net"] } ``` _It works with the original client, `"path": "ff2mpv.bat",`_
ryze312 commented 2023-10-19 02:57:08 +02:00 (Migrated from github.com)

Interesting, where is ff2mpv.bat located at?

Interesting, where is `ff2mpv.bat` located at?
TheLongLife commented 2023-10-19 03:00:07 +02:00 (Migrated from github.com)

Interesting, where is ff2mpv.bat located at?

In the same folder where the JSON file is, so C:\\Miscellaneous tools\\ff2mpv-master\\
I also placed the ff2mpv-rust.exe in the same folder, and tried to use just the "path": "ff2mpv-rust.exe", , but that didn't work either.

> Interesting, where is `ff2mpv.bat` located at? In the same folder where the JSON file is, so `C:\\Miscellaneous tools\\ff2mpv-master\\` I also placed the *ff2mpv-rust.exe* in the same folder, and tried to use just the `"path": "ff2mpv-rust.exe",` , but that didn't work either.
ryze312 commented 2023-10-19 03:03:34 +02:00 (Migrated from github.com)

It might be that Firefox doesn't allow executing EXE directly.
What if you wrap it up in a batch script and call it there?

@echo off
call ff2mpv-rust.exe
It might be that Firefox doesn't allow executing EXE directly. What if you wrap it up in a batch script and call it there? ```bat @echo off call ff2mpv-rust.exe ```
TheLongLife commented 2023-10-19 03:06:12 +02:00 (Migrated from github.com)

Actually I tried that as well, I forgot to mention. It didn't work either.
I re-tried it right now and still nothing.

Actually I tried that as well, I forgot to mention. It didn't work either. I re-tried it right now and still nothing.
ryze312 commented 2023-10-19 03:11:45 +02:00 (Migrated from github.com)

Could you show how you got the registry key set up?

Could you show how you got the registry key set up?
TheLongLife commented 2023-10-19 03:19:49 +02:00 (Migrated from github.com)

This one is for Vivaldi:
image

This one is for Firefox:
image

This one is for Vivaldi: ![image](https://github.com/ryze312/ff2mpv-rust/assets/10586628/4d4f05e8-d94c-4379-bd04-064f6744b202) This one is for Firefox: ![image](https://github.com/ryze312/ff2mpv-rust/assets/10586628/c0f2560d-5510-403c-8f43-cc82e572a698)
ryze312 commented 2023-10-19 03:23:19 +02:00 (Migrated from github.com)

That looks right. This might just be a Windows-specific bug, as the script was only properly tested on Linux.
I'll get access to Windows machine in a couple days and try to reproduce it.

That looks right. This might just be a Windows-specific bug, as the script was only properly tested on Linux. I'll get access to Windows machine in a couple days and try to reproduce it.
TheLongLife commented 2023-10-19 03:26:24 +02:00 (Migrated from github.com)

Alright, thank you for your time and effort.

Alright, thank you for your time and effort.
Samson7 commented 2023-10-20 11:24:51 +02:00 (Migrated from github.com)

I confirm that i got the same bug on Windows 10 22h2 build 19045.3570 using custom-firefox browser Mercury. With original python script ff2mpv.py everything works as expected. Also i noticed that ff2mpv-rust doesn't recognize config file located in %APPDATA% as mentioned in the readme(if i execute ff2mpv-rust validate it says "Config doesn't exist"), so maybe that the reason why the native-host doesn't work?

I confirm that i got the same bug on Windows 10 22h2 build 19045.3570 using custom-firefox browser Mercury. With original python script ff2mpv.py everything works as expected. Also i noticed that ff2mpv-rust doesn't recognize config file located in %APPDATA% as mentioned in the readme(if i execute ff2mpv-rust validate it says "Config doesn't exist"), so maybe that the reason why the native-host doesn't work?
ryze312 commented 2023-10-21 20:14:03 +02:00 (Migrated from github.com)

Can anyone verify if it works on the linked branch? I've published a pre-release with fix applied, if you don't want to compile it yourself.

Can anyone verify if it works on the linked branch? I've published a pre-release with fix applied, if you don't want to compile it yourself.
ryze312 commented 2023-10-21 20:21:03 +02:00 (Migrated from github.com)

I confirm that i got the same bug on Windows 10 22h2 build 19045.3570 using custom-firefox browser Mercury. With original python script ff2mpv.py everything works as expected. Also i noticed that ff2mpv-rust doesn't recognize config file located in %APPDATA% as mentioned in the readme(if i execute ff2mpv-rust validate it says "Config doesn't exist"), so maybe that the reason why the native-host doesn't work?

ff2mpv-rust will fallback to the default configuration if there is an error parsing config file. This is unrelated. Please check the path and name of the config file. If the problem persists, open up a new issue.

> I confirm that i got the same bug on Windows 10 22h2 build 19045.3570 using custom-firefox browser Mercury. With original python script ff2mpv.py everything works as expected. Also i noticed that ff2mpv-rust doesn't recognize config file located in %APPDATA% as mentioned in the readme(if i execute ff2mpv-rust validate it says "Config doesn't exist"), so maybe that the reason why the native-host doesn't work? ff2mpv-rust will fallback to the default configuration if there is an error parsing config file. This is unrelated. Please check the path and name of the config file. If the problem persists, open up a new issue.
TheLongLife commented 2023-10-21 20:28:56 +02:00 (Migrated from github.com)

Can anyone verify if it works on the linked branch? I've published a pre-release with fix applied, if you don't want to compile it yourself.

I just tested it. It works now 👍

> Can anyone verify if it works on the linked branch? I've published a pre-release with fix applied, if you don't want to compile it yourself. I just tested it. It works now 👍
ryze312 commented 2023-10-21 20:31:02 +02:00 (Migrated from github.com)

Thanks for helping with the issue! I'll make a couple more changes and do a proper release soon.

Thanks for helping with the issue! I'll make a couple more changes and do a proper release soon.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: ryze/ff2mpv-rust#5
No description provided.