fix path in cp command #4221

Closed
opened 2025-12-29 18:33:56 +01:00 by adam · 4 comments
Owner

Originally created by @systeembeheerder on GitHub (Oct 27, 2020).

Originally assigned to: @systeembeheerder on GitHub.

Change Type

[ ] Addition
[X ] Correction
[ ] Deprecation
[ ] Cleanup (formatting, typos, etc.)

Area

[ ] Installation instructions
[ ] Configuration parameters
[ ] Functionality/features
[ ] REST API
[ ] Administration/development
[ ] Other

Proposed Changes

# cp -r /opt/netbox-X.Y.Z/netbox/scripts /opt/netbox/netbox/scripts/
# cp -r /opt/netbox-X.Y.Z/netbox/reports /opt/netbox/netbox/reports/

will create a directory
/opt/netbox/netbox/scripts/scripts and /opt/netbox/netbox/reports/reports

Originally created by @systeembeheerder on GitHub (Oct 27, 2020). Originally assigned to: @systeembeheerder on GitHub. <!-- NOTE: IF YOUR ISSUE DOES NOT FOLLOW THIS TEMPLATE, IT WILL BE CLOSED. Please indicate the nature of the change by placing an X in one of the boxes below. --> ### Change Type [ ] Addition [X ] Correction [ ] Deprecation [ ] Cleanup (formatting, typos, etc.) ### Area [ ] Installation instructions [ ] Configuration parameters [ ] Functionality/features [ ] REST API [ ] Administration/development [ ] Other <!-- Describe the proposed change(s). --> ### Proposed Changes ```` # cp -r /opt/netbox-X.Y.Z/netbox/scripts /opt/netbox/netbox/scripts/ # cp -r /opt/netbox-X.Y.Z/netbox/reports /opt/netbox/netbox/reports/ ```` will create a directory ``/opt/netbox/netbox/scripts/scripts`` and `` /opt/netbox/netbox/reports/reports``
adam added the status: acceptedtype: documentation labels 2025-12-29 18:33:56 +01:00
adam closed this issue 2025-12-29 18:33:56 +01:00
Author
Owner

@DanSheps commented on GitHub (Oct 27, 2020):

What is your proposed change?

@DanSheps commented on GitHub (Oct 27, 2020): What is your proposed change?
Author
Owner

@chicks-net commented on GitHub (Oct 27, 2020):

#5276 looks like the fix.

@chicks-net commented on GitHub (Oct 27, 2020): #5276 looks like the fix.
Author
Owner

@DanSheps commented on GitHub (Oct 27, 2020):

Unfortunately, that is not a workable solution (we are going to try and stay away from relative paths so that everything is explicit, if you deviate from the recommended configuration it is on you to resolve).

@DanSheps commented on GitHub (Oct 27, 2020): Unfortunately, that is not a workable solution (we are going to try and stay away from relative paths so that everything is explicit, if you deviate from the recommended configuration it is on you to resolve).
Author
Owner

@systeembeheerder commented on GitHub (Oct 27, 2020):

@chicks-net yes, thank you
@DanSheps ok, I can rewrite the patch to absolute paths, no problem. Can we agree the current cp command does copy to the wrong directory?

https://github.com/netbox-community/netbox/pull/5277

@systeembeheerder commented on GitHub (Oct 27, 2020): @chicks-net yes, thank you @DanSheps ok, I can rewrite the patch to absolute paths, no problem. Can we agree the current cp command does copy to the wrong directory? https://github.com/netbox-community/netbox/pull/5277
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#4221