Skip to content

Conversation

mkoeppe
Copy link
Contributor

@mkoeppe mkoeppe commented May 7, 2024

As reported in https://groups.google.com/g/sage-devel/c/8ZsBcHGmn6s @culler

📝 Checklist

  • The title is concise and informative.
  • The description explains in detail what this PR is about.
  • I have linked a relevant issue or discussion.
  • I have created tests covering the changes.
  • I have updated the documentation and checked the documentation preview.

⌛ Dependencies

Copy link

Documentation preview for this PR (built with commit c243404; changes) is ready! 🎉
This preview will update shortly after each push to this PR.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented May 17, 2024

@orlitzky Please review

@orlitzky
Copy link
Contributor

I'm not wasting an hour testing packages that do nothing. Just delete them already.

@culler
Copy link
Contributor

culler commented May 18, 2024

Using this PR I ran ./bootstrap and ./configure on an Intel mac mini running 14.4.1 with autoconf installed. There were no "command not found" messages.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented May 18, 2024

Thanks for testing!

@vbraun vbraun merged commit 1de533d into sagemath:develop May 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants