Skip to content

Conversation

@Atlante45
Copy link
Contributor

When calling configure_file() on namespace.h.in, cmake will create a temporary file at the target location called namespace.h.tmp and when done generating move it to namespace.h

This causes an issue when building multiple release types at the same time. They will both generate the same .tmp file and overwrite each other. The first that completes will move the shared file causing the other instance to fail with the error No such file or directory

This is a proposed solution that generates the namespace header in the binary directory as intended by configure_file and then makes sure all relevant steps load the header from the appropriate directory

@bifurcation bifurcation mentioned this pull request Apr 24, 2025
Copy link
Contributor

@bifurcation bifurcation left a comment

Choose a reason for hiding this comment

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

Thanks for the PR @Atlante45, and sorry for the very delayed review.

FYI, I filed #448 with a couple of minor follow-ups, so as not to delay merging this.

@bifurcation
Copy link
Contributor

I'm not sure why CI is not running on this. Merging on faith, and verifying later.

@bifurcation bifurcation merged commit 7ca371f into cisco:main Apr 24, 2025
bifurcation added a commit that referenced this pull request Apr 24, 2025
bifurcation added a commit that referenced this pull request Apr 24, 2025
* Follow-ups from #439

* Follow-up from #443
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