🔧 Allow use of a minimal Config.h #27338
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
thinkyhead
force-pushed
the
bf2_minimal_config_PR
branch
28 times, most recently
from
32f2168 to
1b3d729
Compare
thinkyhead
force-pushed
the
bf2_minimal_config_PR
branch
6 times, most recently
from
96c75ed to
efcbb23
Compare
thinkyhead
force-pushed
the
bf2_minimal_config_PR
branch
4 times, most recently
from
b6f3896 to
2b868fd
Compare
thinkyhead
force-pushed
the
bugfix-2.1.x
branch
from
c792921 to
37fb26b
Compare
thinkyhead
force-pushed
the
bf2_minimal_config_PR
branch
from
2b868fd to
f0fe9b5
Compare
|
Just a heads up: I'm running the monthly "build all examples" script and have run across an issue with recent Conditionals_post.h:899:44: error: 'X_MAX_ENDSTOP_HIT_STATE' was not declared in this scope; did you mean 'X_MIN_ENDSTOP_HIT_STATE'?
899 | #define Z2_MIN_ENDSTOP_HIT_STATE X_MAX_ENDSTOP_HIT_STATE...or... Conditionals_post.h:907:44: error: 'Z_MAX_ENDSTOP_HIT_STATE' was not declared in this scope; did you mean 'Z2_MAX_ENDSTOP_HIT_STATE'?
907 | #define Z2_MIN_ENDSTOP_HIT_STATE Z_MAX_ENDSTOP_HIT_STATEThe affected configs so far:
Looks like it's only those three: There's also a report of
|
thinkyhead
force-pushed
the
bf2_minimal_config_PR
branch
2 times, most recently
from
40c5279 to
ecbff8f
Compare
thinkyhead
added a commit
to MarlinFirmware/Configurations
that referenced
this pull request
Aug 15, 2024
thinkyhead
force-pushed
the
bf2_minimal_config_PR
branch
from
ecbff8f to
5b5fbc4
Compare
thinkyhead
force-pushed
the
bf2_minimal_config_PR
branch
from
5b5fbc4 to
ae31871
Compare
thinkyhead
force-pushed
the
bf2_minimal_config_PR
branch
from
e301d6b to
9fa25f2
Compare
1 task
rngkll
pushed a commit
to rngkll/Configurations
that referenced
this pull request
Oct 24, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
By popular demand…
Users have long wanted a way to use a minimal configuration file containing only their enabled options without having to migrate their settings from the old configurations to the new formats. Some vendors maintain unified configuration files based on this basic idea.
We balked at this idea because we didn't want to wrap every option in the config files with
#ifndefas this would only make the files harder to read. We did put together aconfig.inimethod, but it is somewhat obscure and we don't distribute our examples in this format, so users and vendors have been slow to adopt. (It also requires PlatformIO.)This PR provides…
CONFIG_EXPORT 5andCONFIG_EXPORT 105options. On build this export option creates aConfig-export.hfile containing only the defined options, split up by file or by section. This file should be renamed toConfig.hfor use.Config.hfile so it can act as the sole configuration file, overriding the base default config files. TheConfiguration.handConfiguration_adv.hfiles will be ignored, with a warning.