Skip to content

Revert "Revert "Add a few forgotten API_SUFFIX wrappers in the recently changed LAPACKE ?TFSM""#1059

Merged
langou merged 1 commit intomasterfrom
revert-1058-revert-1046-fixup-1042
Oct 2, 2024
Merged

Revert "Revert "Add a few forgotten API_SUFFIX wrappers in the recently changed LAPACKE ?TFSM""#1059
langou merged 1 commit intomasterfrom
revert-1058-revert-1046-fixup-1042

Conversation

@weslleyspereira
Copy link
Copy Markdown
Collaborator

Reverts #1058.

That got complicated as @martin-frbg predicted in #1057 (comment). Sorry for that.

@martin-frbg
Copy link
Copy Markdown
Collaborator

Actually I'm a bit unsure what the problem is now, since there were no intervening commits ? My suggestion was simply that merging the second of my original PRs after inadvertently merging the first should be equivalent to, and less noisy in the git history than reverting one and then merging the PR that combined them. Now we're looking at revert&revert-the-revert which is bound to be even more noisy ?

@langou langou merged commit db1bcd2 into master Oct 2, 2024
@weslleyspereira
Copy link
Copy Markdown
Collaborator Author

Actually I'm a bit unsure what the problem is now, since there were no intervening commits ? My suggestion was simply that merging the second of my original PRs after inadvertently merging the first should be equivalent to, and less noisy in the git history than reverting one and then merging the PR that combined them. Now we're looking at revert&revert-the-revert which is bound to be even more noisy ?

Github didn't let us merge your PR #1057 after we accepted #1046. I suspect it is because they shared same commit hashes, if that is really the case. Reverting the merge didn't help.

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.

3 participants