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

If cpp_path is relative, make it relative to the current theory. #676

Merged
merged 2 commits into from
Jan 27, 2024

Conversation

adbrucker
Copy link
Contributor

If the path configures as "cpp_path" is not an absolute path, convert it to an absolute path by prepending the directory in which the "install_C_file" is used. This makes it easier to share the same configuration across a team using different operating systems.

Copy link
Member

@lsf37 lsf37 left a comment

Choose a reason for hiding this comment

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

Thanks, I think this makes sense.

Can you please add a sign-off line to the commit message for the Developer Certificate of Origin and remove any trailing whitespace from the new lines to placate the whitespace check.

@lsf37
Copy link
Member

lsf37 commented Jan 17, 2024

@adbrucker ping -- without the sign-off I can't merge your PR.

@adbrucker adbrucker force-pushed the cpp_path_relative branch 5 times, most recently from 44dc9f7 to 809e485 Compare January 26, 2024 04:25
@lsf37 lsf37 self-assigned this Jan 27, 2024
@lsf37 lsf37 added the C-parser anything about the C/Simpl parser label Jan 27, 2024
@lsf37
Copy link
Member

lsf37 commented Jan 27, 2024

(rebased)

@lsf37 lsf37 merged commit c71c31d into seL4:master Jan 27, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-parser anything about the C/Simpl parser
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants