Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Transition repository from absolute to relative paths #36

Merged
merged 5 commits into from
Aug 2, 2022

Conversation

aisgbnok
Copy link
Owner

Summary

This PR transitions the repository from absolute paths to relative paths.

Why

Previously this repository had to be cloned in the exact same spot. It was required to have a path of C:\provisioning\. This was somewhat annoying. Moving to relative paths means that this repository can be stored anywhere. Executing package builds or scripts will now use relative paths and are no longer constricted to an absolute path of C:\provisioning\....

Drawbacks

The only main annoyance is that now instead of requiring the path to be absolute, you must execute commands and scripts at the root of the packages directory. Using relative paths means their source or start has to be the same or they don't know how to get to their destination. I prefer this over being forced to use absolute paths everywhere, but it's something to be aware of.

@aisgbnok aisgbnok added the Enhancement New feature or request label Jul 31, 2022
@aisgbnok aisgbnok self-assigned this Jul 31, 2022
@aisgbnok aisgbnok added this to the 4.0 milestone Jul 31, 2022
# Conflicts:
#	packages/Generate New Packages.bat
#	packages/building.md
#	packages/gci_air_gap/README.md
#	packages/netlab_core/README.md
#	packages/netlab_core/netlab_core_customizations.xml
@aisgbnok aisgbnok changed the title Add relative paths Transition repository from absolute to relative paths Aug 2, 2022
@aisgbnok aisgbnok merged commit 442c693 into main Aug 2, 2022
@aisgbnok aisgbnok deleted the add-relative-paths branch August 2, 2022 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Look into relative paths
1 participant