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

Cpp wrapper #675

Closed
wants to merge 2 commits into from
Closed

Cpp wrapper #675

wants to merge 2 commits into from

Conversation

adbrucker
Copy link
Contributor

A small update to the cpp-script that allows to use the same script on Linux/Windows (with the cpp from gcc) and macOS. This makes it easier to share the same configuration across a team using different operating systems.

@lsf37
Copy link
Member

lsf37 commented Oct 8, 2023

Thanks for drawing our attention to this script -- I don't think it's actually necessary any more, at least I haven't used it any more in ages and I'm one of the main Darwin users of the l4v repo.

I'm tempted to remove it from the repo. Did you find the tool necessary for other applications of the parser?

@adbrucker
Copy link
Contributor Author

The situation is the opposite for me: I seem to the only non-macOS/Darwin user. I just checked with the macOS/Darwin user in my team. And, indeed, the script does not seem to be necessary any longer. Hence, removing it from the repo seems to be a sensible action.

@lsf37
Copy link
Member

lsf37 commented Oct 9, 2023

Perfect, then I'll close this PR and make another one to remove the script.

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