Skip to content

Fix clang flags.#5

Open
RecursiveDescent wants to merge 2 commits into
volantlang:mainfrom
RecursiveDescent:main
Open

Fix clang flags.#5
RecursiveDescent wants to merge 2 commits into
volantlang:mainfrom
RecursiveDescent:main

Conversation

@RecursiveDescent
Copy link
Copy Markdown

Fixes the clang flag to be able to accept multiple arguments.

Copy link
Copy Markdown

@ode ode left a comment

Choose a reason for hiding this comment

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

using /bin/bash is more of a workaround than a fix, also, it destroys windows compatibility [sadly, we have to care about that too]

@RecursiveDescent
Copy link
Copy Markdown
Author

RecursiveDescent commented Sep 3, 2020

using /bin/bash is more of a workaround than a fix, also, it destroys windows compatibility [sadly, we have to care about that too]

It wasn't really that windows compatible in the first place, how many windows users do you know of that have a C compiler installed in the first place?

@ode
Copy link
Copy Markdown

ode commented Sep 3, 2020

using /bin/bash is more of a workaround than a fix, also, it destroys windows compatibility [sadly, we have to care about that too]

It wasn't really that windows compatible in the first place, how many windows users do you know of that have a C compiler installed in the first place?

@apoorvsingal does. so that's one. besides, we list the clang compiler as a dependency so that users can get it if they want to use volant. As much as I love linux, we cannot just not support windows.

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