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

context sourcing broken (ResolvedContext.append_sys_path not serialised) #889

Closed
nerdvegas opened this issue May 14, 2020 · 0 comments · Fixed by #890
Closed

context sourcing broken (ResolvedContext.append_sys_path not serialised) #889

nerdvegas opened this issue May 14, 2020 · 0 comments · Fixed by #890

Comments

@nerdvegas
Copy link
Contributor

PR #878 added member ResolvedContext.append_sys_path, however serialization wasn't updated to reflect this. Consequently, any context loaded from file does not work:

$ rez-env -i test.rxt
Traceback (most recent call last):
...
  File "/home/ajohns/rez/2.61.0/local/lib/python2.7/site-packages/rez/resolved_context.py", line 1719, in _execute
    if self.append_sys_path:
AttributeError: 'ResolvedContext' object has no attribute 'append_sys_path'

A basic context serialisation error like this should have been caught by tests also, so part of this ticket should included adding the relevant unit test.

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 a pull request may close this issue.

1 participant