exceptionfactory commented on PR #8500: URL: https://github.com/apache/nifi/pull/8500#issuecomment-1997574031
Thanks for the reply and reference to the mailing list thread @bobpaulin. Reviewing the [Python Virtual Environments documentation](https://docs.python.org/3/tutorial/venv.html) and a [Python venv tutorial](https://python.land/virtual-environments/virtualenv), `Scripts` seems to be the standard directory for Windows and `bin` seems to be standard for macOS and Linux. Given that background, I think directory-existence checking would be the ideal way forward. It avoids having to evaluate the OS System properties, and also avoids the need for a new NiFi property. If we run into issues down the road, then something along the lines of the filter property you described seems worth considering. For now, however, checking first for `bin` and then for `Scripts` seems like it should cover current behavior. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
