8000 failed path do not match error · Issue #253 · CEXT-Dan/PyRx · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

failed path do not match error #253

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

Jump to bottom
Closed
CEXT-Dan opened this issue Apr 8, 2025 · 1 comment
Closed

failed path do not match error #253

CEXT-Dan opened this issue Apr 8, 2025 · 1 comment

Comments

@CEXT-Dan
Copy link
Owner
CEXT-Dan commented Apr 8, 2025

No description provided.

@CEXT-Dan
Copy link
Owner Author
CEXT-Dan commented Apr 8, 2025

To reproduce

if two paths, i.e.
M:\Dev\Projects\PyRxGit\PySamples
M:\Dev\Projects\PyRxGit\PySamples\Play
Have a module called _doit.py

If I load another module from M:\Dev\Projects\PyRxGit\PySamples\Play, say
M:\Dev\Projects\PyRxGit\PySamples\Play\offset.py

Then try to load
M:\Dev\Projects\PyRxGit\PySamples\_doit.py

Then python sees M:\Dev\Projects\PyRxGit\PySamples\Play\_doit.py first,

Because it was added to sys.path first. I’m not sure the best course of action.

I could:

  • try to move the path of the module being loaded to the front of sys.path, then move them back
  • only add modules paths to the search path, then pop after the module is loaded
  • Or don’t do anything and just advise people to use unique names

CEXT-Dan added a commit that referenced this issue Apr 9, 2025
@CEXT-Dan
[#253] potential fix
0ae13e0
CEXT-Dan added a commit that referenced this issue Apr 9, 2025
@CEXT-Dan
[#253] change search order
30b2937
CEXT-Dan added a commit that referenced this issue Apr 9, 2025
@CEXT-Dan
[#253] check null
524324d
CEXT-Dan added a commit that referenced this issue Apr 9, 2025
@CEXT-Dan
[#253] add test
fc10954
CEXT-Dan added a commit that referenced this issue Apr 9, 2025
@CEXT-Dan
[#253] add debug warning
84a3b91
CEXT-Dan added a commit that referenced this issue Apr 9, 2025
@CEXT-Dan
[#253] check list size
c46a057
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@CEXT-Dan
0