-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
fixes #22065; do not search path for relative imports #22073
Conversation
btw, something to be aware of is that this kind of "strictness" increase will result in a number of packages failing to compile - this is certainly the case of status' packages which prompted our initial desire for this feature - worst case, this might have to be turned into a warning ("imported from weird location") that can be turned into an error |
As to important packages, only arraymancer uses |
Thanks for your hard work on this PR! Hint: mm: orc; opt: speed; options: -d:release |
@@ -260,6 +260,8 @@ | |||
|
|||
- `strutils.split` and `strutils.rsplit` now forbid an empty separator. | |||
|
|||
- Relative imports will not resolve to stdlib anymore, e.g. `import ./tables` now reports an error properly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wait .. only stdlib
or also -p:path
? ie the latter should also not be searched
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, make sense, I will correct the changelog.
…ng#22073) * fixes nim-lang#22065; do not search path for "./" * simplify * fixes * fixes * allow ".." * cleanup * add a test case * slightly modify the import * adds a changelog
fixes #22065