Skip to content

Add automatic build configuration [] - #163

Merged
DarkArc merged 1 commit into
mainfrom
cmake-autodetect
Oct 1, 2026
Merged

DarkArc merged 1 commit into
mainfrom
cmake-autodetect

Conversation

@DarkArc

@DarkArc DarkArc commented Oct 1, 2026 •

Copy link
Copy Markdown
Collaborator

This an update to @mborland's work in #120. This change additionally:

  • disables the in-source build (to prevent conflicts with .envrc),
  • makes EDG_BASE a cached variable more closely associated with the build,
  • and fixes a bug with Unix Makefiles.

In contrast to #120, we also maintain the use of CACHE variables for the macro configurations allowing things to more easily be tweaked and viewed with ccmake.

@DarkArc
DarkArc requested a review from mborland October 1, 2026 12:51

@mborland mborland left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good changes. I will close my PR in favor of this one.

@DarkArc
DarkArc force-pushed the cmake-autodetect branch 2 times, most recently from 4e91dde to 3bd1009 Compare October 1, 2026 13:16
This change additionally:
- disables the in-source build (to prevent conflicts with .envrc),
- makes EDG_BASE a cached variable more closely associated with the build,
- and fixes a bug with Unix Makefiles.
@DarkArc
DarkArc merged commit ca5890b into main Oct 1, 2026
15 checks passed
@mborland
mborland deleted the cmake-autodetect branch October 1, 2026 15:27
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.

2 participants