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.

Sign up for GitHub

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

Add the new experimental forloopmacros flag for Stint fix #109 #110

Merged

mratsim merged 2 commits into master from experimental-forloopmacros

Aug 20, 2018

Contributor

mratsim commented Aug 20, 2018 •

edited

Loading

See #109


Add the new experimental forloopmacros flag for Stint

adb049a

Contributor Author

mratsim commented Aug 20, 2018

Unfortunately only Linux tests are working.

Compilation by cloning git repo fails because Nimbus' nim.cfg is contaminating koch compilation and csources doesn't support the {.experimental.} flag:
https://travis-ci.org/status-im/nimbus/jobs/418128454#L647-L800

$ export PATH=$PWD/bin:$PATH
$ nim c koch
Hint: used config file '/Users/travis/build/status-im/nimbus/nim/config/nim.cfg' [Conf]
/Users/travis/build/status-im/nimbus/nim.cfg(1, 3) Error: invalid argument for command line option: '--experimental'


Use a .nimble switch instead of .cfg

a6c3342

mratsim merged commit 18e9390 into master

mratsim added a commit that referenced this pull request


Only install dependencies in Appveyor followup #110, fix #109

986cba5

mratsim mentioned this pull request

Post Mortem for {.experimental: "ForLoopMacros".} #120

Closed

yglukhov deleted the experimental-forloopmacros branch

September 6, 2018 15:58

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment