Remove CDN in a Box Makefile directory check #5653
Merged
Conversation
|
|
Fixed the Tomcat RPM target path in f7414d9. |
|
This works now |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
What does this PR (Pull Request) do?
This PR includes various improvements to the CDN in a Box Makefile:
Allows you to use the Makefile even if a parent directory name contains a space
Allows you to run the Makefile without needing to be in the CDN in a Box directory
This PR is not related to any Issue
Which Traffic Control components are affected by this PR?
What is the best way to verify this PR?
${GOPATH}/src/github.com/apache, rename thetrafficcontroldirectory toTraffic Controlwith a space.Traffic Controldirectory, use the CDN in a Box Makefile to build the RPMs:If this is a bug fix, what versions of Traffic Control are affected?
The following criteria are ALL met by this PR