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

NameError: name 'popen' is not defined (typo in rezplugins.shell.cmd) (rez-2.47.7) #771

Closed
willjp opened this issue Oct 23, 2019 · 3 comments

Comments

@willjp
Copy link
Contributor

willjp commented Oct 23, 2019

Hey guys, this one's just a typo.

C:\Users\willjp>rez-env rez
Traceback (most recent call last):
  File "C:\Python-3.6.5\Lib\runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "C:\Python-3.6.5\Lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "C:\Users\willjp\Documents\rez\install\Scripts\rez\rez-env.exe\__main__.py", line 7, in <module>
  File "c:\users\willjp\documents\rez\install\lib\site-packages\rez\cli\_entry_points.py", line 152, in run_rez_env
    return run("env")
  File "c:\users\willjp\documents\rez\install\lib\site-packages\rez\cli\_main.py", line 152, in run
    returncode = run_cmd()
  File "c:\users\willjp\documents\rez\install\lib\site-packages\rez\cli\_main.py", line 144, in run_cmd
    return opts.func(opts, opts.parser, extra_arg_groups)
  File "c:\users\willjp\documents\rez\install\lib\site-packages\rez\cli\env.py", line 250, in command
    block=True)
  File "c:\users\willjp\documents\rez\install\lib\site-packages\rez\resolved_context.py", line 877, in _check
    return fn(self, *nargs, **kwargs)
  File "c:\users\willjp\documents\rez\install\lib\site-packages\rez\resolved_context.py", line 1251, in execute_shell
    self._execute(executor)
  File "c:\users\willjp\documents\rez\install\lib\site-packages\rez\utils\memcached.py", line 254, in wrapper
    return func(*nargs, **kwargs)
  File "c:\users\willjp\documents\rez\install\lib\site-packages\rez\resolved_context.py", line 1697, in _execute
    executor.append_system_paths()
  File "c:\users\willjp\documents\rez\install\lib\site-packages\rez\rex.py", line 1152, in append_system_paths
    paths = sh.get_syspaths()
  File "c:\users\willjp\documents\rez\install\lib\site-packages\rezplugins\shell\cmd.py", line 125, in get_syspaths
    p = popen(cmd, stdout=subprocess.PIPE,
NameError: name 'popen' is not defined
@JeanChristopheMorinPerso
Copy link
Member

This should get fixed by #770 I think. @bfloch can you confirm?

@willjp
Copy link
Contributor Author

willjp commented Oct 23, 2019

I checked the PR you mentioned, it is the same line number. That would fix it.

@willjp willjp closed this as completed Oct 23, 2019
@bfloch
Copy link
Contributor

bfloch commented Oct 23, 2019

Same bug.

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

No branches or pull requests

3 participants