Skip to content

bump index state to get new hackage-security #11024

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
Jun 25, 2025

Conversation

geekosaur
Copy link
Collaborator

Template B: This PR does not modify behaviour or interface

E.g. the PR only touches documentation or tests, does refactorings, etc.

Include the following checklist in your PR:

  • Patches conform to the coding conventions.
  • Is this a PR that fixes CI? If so, it will need to be backported to older cabal release branches (ask maintainers for directions).

@geekosaur
Copy link
Collaborator Author

splitmix is unhappy on Macs because of missing kSecRandomDefault. Apple docs say it's been there since MacOS 10.7. Any clues?

@geekosaur
Copy link
Collaborator Author

Change in splitmix 0.1.3 apparently. Is it okay if for now we block that version?

@ulysses4ever
Copy link
Collaborator

Change in splitmix 0.1.3 apparently. Is it okay if for now we block that version?

Makes sense to me. Maybe we could ask for their advice though?

@geekosaur geekosaur force-pushed the r3.16-C.3.7-index-state branch from 50faa21 to dfb93a0 Compare June 25, 2025 20:23
@geekosaur
Copy link
Collaborator Author

@phadej do you know why GitHub's Mac runners claim not to know about kSecRandomDefault?

<command line>: dlopen(/Users/runner/.cabal/store/ghc-9.2.8/lib/libHSspltmx-0.1.3-0cd0680e-ghc9.2.8.dylib, 0x0005): symbol not found in flat namespace '_kSecRandomDefault'

@geekosaur geekosaur added merge me Tell Mergify Bot to merge and removed attention: needs-review labels Jun 25, 2025
@mergify mergify bot added merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days ready and waiting Mergify is waiting out the cooldown period labels Jun 25, 2025
Copy link
Contributor

mergify bot commented Jun 25, 2025

This pull request has been removed from the queue for the following reason: pull request branch update failed.

The pull request can't be updated.

You should update or rebase your pull request manually. If you do, this pull request will automatically be requeued once the queue conditions match again.
If you think this was a flaky issue, you can requeue the pull request, without updating it, by posting a @mergifyio requeue comment.

@geekosaur
Copy link
Collaborator Author

o.O

@geekosaur geekosaur force-pushed the r3.16-C.3.7-index-state branch from dfb93a0 to 65594f9 Compare June 25, 2025 22:19
@geekosaur
Copy link
Collaborator Author

No problems with any of the others the past two days….

@ulysses4ever
Copy link
Collaborator

Probably because you forked a branch and got it to a mergeable state (approvals, labels, CI) before master advanced. Generally, I think bot has completely lost the ability to rebase. No matter what the account of the PR author / branch owner is.

@mergify mergify bot merged commit 372a197 into haskell:master Jun 25, 2025
54 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days merge me Tell Mergify Bot to merge priority: high 🔥 ready and waiting Mergify is waiting out the cooldown period
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants