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

Fix for current Weston. #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

vanfanel
Copy link

Current Weston doesn't have weston_watch_process() but wet_watch_process() which requires both the process and the compositor.

Since the data pointer was used to pass the command to run, I had to modify the binder data struct to accomodate the compositor there, etc.

@keithbowes
Copy link
Contributor

I wasn't sure if they were the same or different (weston.h is extremely underdocumented), but I put an ifdef in my fork regardless. I was wondering if perhaps I could just use weston_client_launch, replacing the lower-level use of system_nowait, but I just went the safer route of less breakage.

@selurvedu
Copy link

Your fork works, thank you.

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

Successfully merging this pull request may close these issues.

3 participants