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 Yosemite build #18

Merged
merged 1 commit into from
Dec 9, 2014
Merged

Fix Yosemite build #18

merged 1 commit into from
Dec 9, 2014

Conversation

davidbernard
Copy link
Contributor

From CUPS 1.6, the visibility of parts of the IPP data structures controlled are by the preprocessor symbol IPP_PRIVATE_STRUCTURES.
Visibility is enabled specifically for Mac OS X 10.8 and 10.9 but not for Mac OS X 10.10 in CUPS. This update forces the required visibility.

From CUPS 1.6, the visibility of parts of the IPP data structures controlled are by the preprocessor symbol IPP_PRIVATE_STRUCTURES.
Visibility is enabled specifically for Mac OS X 10.8 and 10.9 but not for Mac OS X 10.10 in CUPS. This update forces the required visibility.
@miccheng
Copy link

miccheng commented Dec 8, 2014

+1 Do get this merged soon.

@m0wfo
Copy link
Owner

m0wfo commented Dec 8, 2014

@miccheng thanks for this, will check this out later this evening (GMT)

@toreyheinz
Copy link

I just verified this works for me on Yosemite w/Ruby 2.0 & 2.1.3

m0wfo added a commit that referenced this pull request Dec 9, 2014
@m0wfo m0wfo merged commit 220fd23 into m0wfo:master Dec 9, 2014
@m0wfo
Copy link
Owner

m0wfo commented Dec 9, 2014

@miccheng will do a release now

@m0wfo
Copy link
Owner

m0wfo commented Dec 9, 2014

@miccheng I have no time to fix / comprehensively verify this, and I'd like to move it all to the ruby FFI if I get time, but your fix is in 0.1.10- Thanks very much for the input :)

@davidbernard davidbernard deleted the yosemite branch December 12, 2014 00:36
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

4 participants