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

feat: add a check-c run option for a custom include path #240

Merged
merged 1 commit into from
Jan 11, 2024

Conversation

JFreegman
Copy link
Member

@JFreegman JFreegman commented Jan 11, 2024

This change is Reviewable

@JFreegman JFreegman added the enhancement New feature for the user, not a new feature for build script label Jan 11, 2024
@JFreegman JFreegman added this to the v0.0.9 milestone Jan 11, 2024
@JFreegman JFreegman force-pushed the ccheck-include-option branch from 3a2a6df to 18c438a Compare January 11, 2024 15:30
@toktok-releaser toktok-releaser merged commit 18c438a into master Jan 11, 2024
13 checks passed
@toktok-releaser toktok-releaser deleted the ccheck-include-option branch January 11, 2024 20:10
@@ -42,7 +42,7 @@ main = do
let (opts, rest) = partition (isPrefixOf "--") args
let (cppOpts, files) = partition (isPrefixOf "-") rest
let cc = Maybe.fromMaybe "clang" $ getFlag "--cc=" opts
let sysInclude = "/src/workspace/hs-tokstyle/include"
let sysInclude = Maybe.fromMaybe "src/workspace/hs-tokstyle/include" $ getFlag "--include=" opts
Copy link
Member

Choose a reason for hiding this comment

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

Was removing the leading / intentional?

Copy link
Member Author

Choose a reason for hiding this comment

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

Nope

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature for the user, not a new feature for build script
Development

Successfully merging this pull request may close these issues.

4 participants