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

Fix #9405 - cfg and nims run in sync #13472

Closed
wants to merge 1 commit into from
Closed

Fix #9405 - cfg and nims run in sync #13472

wants to merge 1 commit into from

Conversation

genotrance
Copy link
Contributor

No description provided.

@genotrance
Copy link
Contributor Author

Test is failing in devel itself.

testament run tests/osproc/texitcode.nim

@genotrance
Copy link
Contributor Author

#13449

@Araq
Copy link
Member

Araq commented Feb 23, 2020

I fixed it too here, #13461

Not sure which version I like better though. Yours seem to avoid some code duplication.

@genotrance
Copy link
Contributor Author

I just moved most of the code into loadConfigs() since I felt it gets used in many more places and simplified loadConfigsAndRunMainCommand(). But until CI passes, I have no idea if I did this right.

@timotheecour
Copy link
Member

timotheecour commented Feb 25, 2020

But until CI passes, I have no idea if I did this right.

for now just close/reopen (#13449 is flaky, not failing always) EDIT: I've sent a PR to fix it #13487

Not sure which version I like better though. Yours seem to avoid some code duplication.

@disruptek
Copy link
Contributor

Is there a way for nimph to not run the NimScript while parsing with loadConfigs? This seems like a breaking change for me...

@timotheecour
Copy link
Member

Is there a way for nimph to not run the NimScript while parsing with loadConfigs? This seems like a breaking change for me...

nim --fullhelp:
--skipCfg:on|off do not read the nim installation's configuration file
--skipUserCfg:on|off do not read the user's configuration file
--skipParentCfg:on|off do not read the parent dirs' configuration files
--skipProjCfg:on|off do not read the project's configuration file

@disruptek
Copy link
Contributor

Sure, but nimph is running loadConfigs directly from the compiler/... After talking to @genotrance it sounds like this is obsolescence by design, so we'll make it work no matter how slow/broken it is. 👍

@genotrance
Copy link
Contributor Author

Superseded by #13488

@genotrance genotrance closed this Feb 25, 2020
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.

4 participants