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

JACK transport #115

Open
xlambein opened this issue Nov 26, 2019 · 6 comments
Open

JACK transport #115

xlambein opened this issue Nov 26, 2019 · 6 comments

Comments

@xlambein
Copy link

First of all, I'd like to thank you all for this crate, it's really nice :-)

For a personal project, I need to have JACK transport functionalities, which are not implemented yet here. I might be able to contribute this feature to the crate myself, but I'm not exactly sure where to begin? Anyone knows what needs to be done, in order to add transport?

@wmedrano
Copy link
Member

wmedrano commented Dec 6, 2019

Is this the same as #50, if so, then the issue was that jackctl functions weren't linked into the version of jack that I had. If not, then I'm not familiar with JACK transport.

@xlambein
Copy link
Author

xlambein commented Dec 9, 2019

Not super familiar with the inner workings of JACK in general, but from looking at the doc it seems that transport is not the same thing as jackctl. Transport is the tool that's used to synchronize the clocks of multiple JACK clients (e.g. Ardour with Hydrogen), whereas jackctl seems to be about starting and managing a JACK server.

@wmedrano
Copy link
Member

wmedrano commented Dec 9, 2019

Seems like I just missed it, so its fine to go ahead and add the feature to rust-jack.

Jack reference: https://jackaudio.org/api/group__TransportControl.html
And the functions do indeed seem to be part of the ffi in the jack-sys crate.

@x37v
Copy link
Contributor

x37v commented Nov 14, 2020

I'm working on some bindings for the query and control: https://github.com/x37v/rust-jack/tree/xnor/transport

I'm not planning to do slow sync or 'master' bindings at this time. Will submit a PR once I test out the control as at this point I've only tested the query.

@x37v
Copy link
Contributor

x37v commented Nov 16, 2020

#134

@x37v
Copy link
Contributor

x37v commented Dec 29, 2020

#134 has been merged, this provides transport query/control.

Transport timebase master is not implemented yet.

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

3 participants