Modify include guards in cmd.c so that Windows compilers don't compil… #291
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
…e Unix-only code.
In 7fd4b01, functions like
alphasort
andscandir
that are not available on Windows were added to a codepath that Windows compilers can see (when compiling, for instance, using a C++ compiler). This leads to errors such as:src/base/cmd/cmd.c:1904:21: error: 'scandir' was not declared in this scope; did you mean 'srand'?
.I've updated the include guards to work around those codepaths, and confirmed a successful compile and test run (
synth_ice40
on a non-empty design) withyosys
. These changes took a while to propagate toyosys
, which is why it took me a while to notice.