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

Mac - command line invokation ignores file names with spaces in #31

Open
mrlimbic opened this issue Nov 25, 2018 · 9 comments
Open

Mac - command line invokation ignores file names with spaces in #31

mrlimbic opened this issue Nov 25, 2018 · 9 comments

Comments

@mrlimbic
Copy link

I need to start Jadeo from the command line as will use multiple instances with different video files playing at different offsets.

However, if a video file is specified, it only loads if the filename contains no spaces.

Any idea? Sounds like would be a simple fix.

@x42
Copy link
Owner

x42 commented Nov 25, 2018

Which version of xjadeo, which OS? With a quick-test it works just fine here on Linux using xjadeo v0.8.8.

Also remote-control via Ardour is known to work on all major platforms with spaces in the file-name.

@mrlimbic
Copy link
Author

mrlimbic commented Nov 25, 2018

Mac OS 10.13.6.

This doesn't load..

/Applications/Jadeo.app/Contents/MacOS/Jadeo /Users/john/Movies/Zombie\ Walk\ Beach\ of\ the\ Dead\ 2011\ -\ Puscifer\ Sour\ Grapes.MP4

When I copied the file and renamed it, it then does load..

/Applications/Jadeo.app/Contents/MacOS/Jadeo /Users/john/Movies/Zombie.MP4

I also tried using quotes instead of backslash method but no different.

How does remote-control specify the file? Is it via CLI interface or native?

@x42
Copy link
Owner

x42 commented Nov 25, 2018

I'm away from a mac right now. I'll check next time I have access to one. It may be the wrapper script. Can you try

/Applications/Jadeo.app/Contents/MacOS/Jadeo-bin "/path/to/your/video with spaces.mp4" ?


As for the remote ctrl, see http://xjadeo.sourceforge.net/remote.html
e.g. /Applications/Jadeo.app/Contents/MacOS/Jadeo --remote and interact via standard I/O
load /Users/john/Movies/Zombie Walk Beach of the Dead 2011 - Puscifer Sour Grapes.MP4
you could probably try to echo or cat <<EOF the file.

IIRC there is also /Applications/Jadeo.app/Contents/MacOS/xjremote that decouples from a running instance.

@mrlimbic
Copy link
Author

Still doesn't load using quotes if spaces. Yeah a wrapper script sounds like a very plausible cause.

This doesn't load with quotes..

/Applications/Jadeo.app/Contents/MacOS/Jadeo "/Users/john/Movies/Zombie Walk Beach of the Dead 2011 - Puscifer Sour Grapes.MP4"

This does load with quotes

/Applications/Jadeo.app/Contents/MacOS/Jadeo "/Users/john/Movies/Zombie.MP4"

Thanks.

x42 added a commit that referenced this issue Nov 25, 2018
This hands over all arguments as received and should fix an issue passing
filesnames with whitespace at application start.

This needs testing to not interfere with Mac session-parameters e.g.
"-psn ..."  may have been ignored in the past due to this and now produce
an issue.
@mrlimbic
Copy link
Author

Also thanks for the remote control interface info. That doesn't seem to care about spaces. So I could just use that for the time being.

@x42
Copy link
Owner

x42 commented Nov 25, 2018

I meant ..../Jadeo-bin "..." (don't use the wrapper script to test, directly start the binary)

@mrlimbic
Copy link
Author

mrlimbic commented Nov 25, 2018

Yes using Jadeo-bin directly solves the space issue.

Although when starting via CLI how do you shut it down cleanly? If I hit ctrl-C it just says CAUGHT SIGINT - shutting down. But doesn't actually shut down until I hit ctrl-C a second time. Then it says bye! and actually exits.

In remote mode quit shuts down cleanly first time.

@x42
Copy link
Owner

x42 commented Nov 25, 2018

If I hit ctrl-C it just says CAUGHT SIGINT - shutting down.

That sounds like a bug. So it seems like the clean-up phase hangs somewhere. Is the window closed? Maybe it's waiting for some Mac specific GUI event?!

perhaps send a SigQuit directly: ctrl+\ instead of a SigInt ctrl+c

@mrlimbic
Copy link
Author

ctrl-\ quits immediately but leaves Mac OS popup "Jadeo quite unexpectedly" so it crashed

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