Skip to content

Various build updates#12

Merged
d-torrance merged 6 commits intoMacaulay2:masterfrom
d-torrance:build-updates
Feb 3, 2026
Merged

Various build updates#12
d-torrance merged 6 commits intoMacaulay2:masterfrom
d-torrance:build-updates

Conversation

@d-torrance
Copy link
Member

@d-torrance d-torrance commented Feb 3, 2026

  • Remove some unnecessary build flags:
    • C++17 is now handled in the configure script, so we don't need to do it in the Makefile
    • automake already takes care of adding the -I flags for the source directory, so we don't need to worry about adding it. Plus CXXFLAGS is the wrong variable for that anyway.
    • Set the I$(srcdir)/src build flags in configure
  • Allow running autogen.sh from the build directory
  • Update the GitHub workflow -- cmake & macOS builds now supported
  • Go back to hard-coding the version number since the M2 cmake build only uses the src directory.

CXXFLAGS was the wrong variable anyway
* Use BUILD directory since build/autotools is really meant for m4
  macros, etc.

* Running "make" and "make distcheck" is redundant since the latter
  checks the former.  So we just run "make distcheck".
@d-torrance d-torrance force-pushed the build-updates branch 6 times, most recently from c794878 to 3a9f77a Compare February 3, 2026 15:18
@d-torrance d-torrance merged commit da33a8f into Macaulay2:master Feb 3, 2026
4 checks passed
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.

1 participant