AcademySoftwareFoundation / AcademySoftwareFoundation/rez

NotImplemented error with config env-var override is misleading

Open
#782 0 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
Python
Stars
1.1k
Forks
374
Avg merge
9d 12h
Merged PRs (30d)
5

Description

This gives a NotImplementedError that is not helpful:
```
]$ REZ_PACKAGE_FILTER='[]' rez-build
```
It should instead do something like this:
```
]$ REZ_PACKAGE_FILTER='[]' rez-build
Error: The "package_filter" setting does not support override via $REZ_PACKAGE_FILTER. You can try using REZ_PACKAGE_FILTER_JSON instead - you must set this to a json-compatible string.
```

Contributor guide

Open the contributing guide

Research direction

Start by reproducing the REZ_PACKAGE_FILTER='[]' rez-build command, then trace the package_filter environment-variable override handling from the rez-build entry point. Done means the unsupported override produces a helpful error and suggests REZ_PACKAGE_FILTER_JSON with its JSON-string requirement.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
cli, tooling
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.