Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add option to generate an include wrapper for files that do not yet exist #2296

Closed
jphickey opened this issue Apr 17, 2023 · 0 comments · Fixed by #2300
Closed

Add option to generate an include wrapper for files that do not yet exist #2296

jphickey opened this issue Apr 17, 2023 · 0 comments · Fixed by #2300

Comments

@jphickey
Copy link
Contributor

Is your feature request related to a problem? Please describe.
A side effect of a recent change to generate_config_includefile is that if the name specified FALLBACK_FILE option does not exist, it now reports that as an error, whereas previously it would create a wrapper to the (nonexistent) file.

The error is correct for most use cases, but if the underlying file is also generated, then there is an initial condition issue when running the build for the first time from a clean slate.

Describe the solution you'd like
If the source for generate_config_includefile is in turn also a generated file, should blindly create the wrapper and not check for existence. It shouldn't even do the normal search in this case, as it would be incorrect to override a generated file with a static one.

Requester Info
Joseph Hickey, Vantage Systems, Inc.

@jphickey jphickey self-assigned this Apr 17, 2023
dzbaker added a commit that referenced this issue Apr 25, 2023
Fix #2296, add option to link to generated files
@dmknutsen dmknutsen added this to the Equuleus milestone May 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants