-
-
Notifications
You must be signed in to change notification settings - Fork 357
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
Add compatibility with 10.11 #144
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That is super cool.
Thank you so much for this!
Since retro compatibility can affect couple of different things, I would be more comfortable to have it all in one file retro.h
that we would include wherever it is required.
That way we don't pollute the other files and where we detect new stuff we just append it in one dedicated place.
cmd/goapp/mac.go
Outdated
@@ -213,7 +213,7 @@ func fillBundle(b driver.Bundle, root string) driver.Bundle { | |||
} | |||
|
|||
if len(b.DeploymentTarget) == 0 { | |||
b.DeploymentTarget = "10.13" | |||
b.DeploymentTarget = "10.11" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would like to keep the latest version as a default value.
Something like this? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Almost there :)
drivers/mac/image.h
Outdated
@@ -3,6 +3,10 @@ | |||
|
|||
#import <Cocoa/Cocoa.h> | |||
|
|||
#if MAC_OS_X_VERSION_MAX_ALLOWED < 101200 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Get rid of these #if and put it just once in retro.h
done :) |
Thank you for this :)
…Sent from my iPhone
On Jul 8, 2018, at 2:02 AM, tectiv3 <notifications@github.com<mailto:notifications@github.com>> wrote:
done :)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub<#144 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ACkWe6qmtQqz-qwy9ZEOGht_JRodanC9ks5uEcqtgaJpZM4VGSlN>.
|
What this PR does ?
goapp now can build and bundle apps for macOS 10.11
Fixes
10.11 compatibility