Skip to content

Move the SC constants for macos to an accessible module (fixes #24374 for mac). #25093

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

Merged
merged 1 commit into from
May 7, 2015

Conversation

Ms2ger
Copy link
Contributor

@Ms2ger Ms2ger commented May 4, 2015

These constants were added in 6f54ce9 and
e8fbd1c to a consts module that is behind a
gate.

I have not confirmed that these constants do indeed work on either OSX or iOS.

@rust-highfive
Copy link
Contributor

r? @alexcrichton

(rust_highfive has picked a reviewer for you, use r? to override)

@alexcrichton
Copy link
Member

@bors: r+ 2a46976

Thanks!

@bors
Copy link
Collaborator

bors commented May 5, 2015

⌛ Testing commit 2a46976 with merge 4f9d076...

@bors
Copy link
Collaborator

bors commented May 5, 2015

💔 Test failed - auto-mac-64-opt

@Ms2ger
Copy link
Contributor Author

Ms2ger commented May 5, 2015

r?

@alexcrichton
Copy link
Member

@bors: r+ 4de61c6

@bors
Copy link
Collaborator

bors commented May 6, 2015

⌛ Testing commit 4de61c6 with merge 91cc61d...

@bors
Copy link
Collaborator

bors commented May 6, 2015

💔 Test failed - auto-mac-64-nopt-t

@Manishearth
Copy link
Member

@bors: r=alexcrichton

@bors
Copy link
Collaborator

bors commented May 6, 2015

📌 Commit f77c9c9 has been approved by alexcrichton

@bors
Copy link
Collaborator

bors commented May 6, 2015

⌛ Testing commit f77c9c9 with merge 7d43e55...

@bors
Copy link
Collaborator

bors commented May 6, 2015

💔 Test failed - auto-win-32-nopt-t

@Manishearth
Copy link
Member

@bors: retry

@bors
Copy link
Collaborator

bors commented May 6, 2015

⌛ Testing commit f77c9c9 with merge 79893d7...

@bors
Copy link
Collaborator

bors commented May 6, 2015

💔 Test failed - auto-mac-64-opt

…ang#24374 for mac).

These constants were added in 6f54ce9 and
e8fbd1c to a consts module that is behind a
gate.

I have not confirmed that these constants do indeed work on either OSX or iOS.

It appears that some of the constants may actually belong in a POSIX module,
but I didn't make these changes here because I don't have access to the POSIX
standard.
@alexcrichton
Copy link
Member

@bors: r+ ff89600

bors added a commit that referenced this pull request May 7, 2015
These constants were added in 6f54ce9 and
e8fbd1c to a consts module that is behind a
gate.

I have not confirmed that these constants do indeed work on either OSX or iOS.
@bors
Copy link
Collaborator

bors commented May 7, 2015

⌛ Testing commit ff89600 with merge 055d926...

@bors
Copy link
Collaborator

bors commented May 7, 2015

@bors bors merged commit ff89600 into rust-lang:master May 7, 2015
@Ms2ger Ms2ger deleted the mac-consts branch May 29, 2015 17:31
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.

5 participants