Upgrade script should check for contents in local_requirements.txt before referencing it #3905

Closed
opened 2025-12-29 18:31:54 +01:00 by adam · 0 comments
Owner

Originally created by @jeremystretch on GitHub (Jul 27, 2020).

Proposed Changes

The upgrade script (upgrade.sh) should check for contents within local_requirements.txt before attempting to reference it with pip install.

Justification

Currently, if the upgrade script is run with an empty local_requirements.txt file, it produces the following output:

Installing local dependencies (pip3 install -r local_requirements.txt)...
You must give at least one requirement to install (see "pip help install")

Although the upgrade script completes without issue, the second line implies an error. Skipping this step when no local dependencies have been defined avoids potentially confusing the user.

Originally created by @jeremystretch on GitHub (Jul 27, 2020). ### Proposed Changes The upgrade script (`upgrade.sh`) should check for contents within `local_requirements.txt` before attempting to reference it with `pip install`. ### Justification Currently, if the upgrade script is run with an empty `local_requirements.txt` file, it produces the following output: ``` Installing local dependencies (pip3 install -r local_requirements.txt)... You must give at least one requirement to install (see "pip help install") ``` Although the upgrade script completes without issue, the second line implies an error. Skipping this step when no local dependencies have been defined avoids potentially confusing the user.
adam added the status: acceptedtype: housekeeping labels 2025-12-29 18:31:54 +01:00
adam closed this issue 2025-12-29 18:31:54 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#3905