mottosso / bleeding-rez

Rez - Reproducible software environments for Windows, Linux and MacOS
GNU Lesser General Public License v3.0
72 stars 10 forks source link

config.nonlocal_packages_path does not account for varying slashes #49

Closed mottosso closed 5 years ago

mottosso commented 5 years ago

It's too literal, without normalising or removing duplicate slashes etc.

https://github.com/mottosso/bleeding-rez/blob/45d4d34409c84f5ec2bd8fd0d02f552c7569364e/src/rez/config.py#L539

To reproduce

PS> env:REZ_LOCAL_PACKAGES_PATH="c:\users\marcus/packages"
PS> env:REZ_PACKAGES_PATH="$env:REZ_LOCAL_PACKAGES_PATH;\\server\rez\packages"
PS> python
>>> from rez.config import config
>>> config.nonlocal_packages_path
# ['c:\\users\\marcus/packages', '\\server\rez\packages']

Expected

>>> config.nonlocal_packages_path
# 'c:\\users\\marcus/packages'
>>> config.local_packages_path
# ['\\server\rez\packages']