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

sysconf interception fix for solarish systems. #4052

Merged
merged 1 commit into from
Nov 23, 2024

Conversation

devnexen
Copy link
Contributor

also adding the _SC_PAGESIZE alias _SC_PAGE_SIZE supported by Linux, macOS and FreeBSD.

close #4050

@devnexen devnexen marked this pull request as draft November 22, 2024 11:29
also adding the `_SC_PAGESIZE` alias `_SC_PAGE_SIZE` supported by
Linux, macOS and FreeBSD.

close rust-lang#4050
@RalfJung
Copy link
Member

LGTM! Why is this marked as a draft?

@rustbot author

@rustbot rustbot added the S-waiting-on-author Status: Waiting for the PR author to address review comments label Nov 23, 2024
@RalfJung
Copy link
Member

Ah, please also change ci.sh to re-enable Solaris on CI.

@RalfJung
Copy link
Member

Please squash whenever you think the PR is ready (and mark it as ready, and do rustbot ready as usual).

@RalfJung
Copy link
Member

Also your system clock seems to be completely off, github says you committed that "enable solaris CI" change 19 hours ago. Would be nice if you could fix your clock so that things are less confusing. :)

@devnexen
Copy link
Contributor Author

@rustbot ready

@rustbot rustbot removed the S-waiting-on-author Status: Waiting for the PR author to address review comments label Nov 23, 2024
@devnexen devnexen marked this pull request as ready for review November 23, 2024 07:46
@rustbot rustbot added the S-waiting-on-review Status: Waiting for a review to complete label Nov 23, 2024
@RalfJung RalfJung enabled auto-merge November 23, 2024 08:04
@RalfJung
Copy link
Member

Thanks for the quick fix :)

@RalfJung RalfJung added this pull request to the merge queue Nov 23, 2024
Merged via the queue into rust-lang:master with commit 2382a20 Nov 23, 2024
7 checks passed
devnexen added a commit to devnexen/miri that referenced this pull request Nov 23, 2024
devnexen added a commit to devnexen/miri that referenced this pull request Nov 24, 2024
devnexen added a commit to devnexen/miri that referenced this pull request Nov 24, 2024
devnexen added a commit to devnexen/miri that referenced this pull request Nov 24, 2024
github-merge-queue bot pushed a commit that referenced this pull request Nov 24, 2024
follow-up on #4052, making a miri evaluation context fn for strerror_r.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Waiting for a review to complete
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Miri broken on Solaris
3 participants