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

Don't skip checking for Raspberry Pi library (Fixes #166). #168

Merged
merged 1 commit into from
Jan 19, 2016

Conversation

kingosticks
Copy link
Contributor

This fixes the build for me on Raspberry Pi.

Before:

-- Looking for XRRGetScreenResources in Xrandr
-- Looking for XRRGetScreenResources in Xrandr - found
-- Found SWIG: /usr/bin/swig2.0 (found version "2.0.12") 
-- Configured features:
-- Pulse-Eight CEC Adapter:                yes
-- Pulse-Eight CEC Adapter detection:      yes
-- xrandr support:                         yes
-- Raspberry Pi support:                   no

After:

-- Looking for XRRGetScreenResources in Xrandr
-- Looking for XRRGetScreenResources in Xrandr - found
-- Looking for bcm_host_init in bcm_host
-- Looking for bcm_host_init in bcm_host - found
-- Found SWIG: /usr/bin/swig2.0 (found version "2.0.12") 
-- Configured features:
-- Pulse-Eight CEC Adapter:                yes
-- Pulse-Eight CEC Adapter detection:      yes
-- xrandr support:                         yes
-- Raspberry Pi support:                   yes

I believe I have already signed the developer agreement.

@valdisrigdon
Copy link

👍

1 similar comment
@apowers313
Copy link

👍

@kikeonline
Copy link

Works great in the Latest OSMC.

@bardal
Copy link

bardal commented Jan 13, 2016

Please merge this in. I was hit by the same issue. Without this fix the pi build is broken (builds cleanly but no pi support).

@kingosticks kingosticks changed the title Don't skip checking for Rasperry Pi library (Fixes #166). Don't skip checking for Raspberry Pi library (Fixes #166). Jan 13, 2016
@ruudboon
Copy link

Same here. Patch works fine

@apowers313
Copy link

Hi @opdenkamp, can this get a little merging love? :)

@bardal
Copy link

bardal commented Jan 14, 2016

Also fixes #172

@opdenkamp
Copy link
Contributor

i want to get a new version out this week, but I'm a bit busy with other stuff right now (as usual), so no promises

opdenkamp pushed a commit that referenced this pull request Jan 19, 2016
Don't skip checking for Raspberry Pi library (Fixes #166).
@opdenkamp opdenkamp merged commit c9423dd into Pulse-Eight:master Jan 19, 2016
@kingosticks
Copy link
Contributor Author

Thanks!

@apowers313
Copy link

@opdenkamp 🍺 :)

@ruudboon
Copy link

@apowers313 @opdenkamp Make it 3 🍺 🍺 🍺 :D

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.

None yet

7 participants