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

convert svn access to github to git sparse checkout #204

Closed
wants to merge 1 commit into from

Conversation

jedwards4b
Copy link
Collaborator

Reinterprets svn access to github.com as a git sparse checkout.

User interface changes?: Yes
I could not figure out how to maintain the original paths in the checkout so this will require changes in components who have used
svn access to github to do sparse checkout in the past.

Fixes: #203

Testing:
test removed:
unit tests: all pass
system tests: suprepo tests are failing - this is issue #202 test_container_simple_svn also fails (new)
manual testing: Checkout cam using cam6_3_130

@jedwards4b jedwards4b marked this pull request as draft November 15, 2023 13:16
@jedwards4b
Copy link
Collaborator Author

If we can figure out how to maintain the original paths in the checkout this PR could potentially allow us to maintain the current cam structure in the face of the upcoming change in support for svn access to github.com

@gold2718
Copy link
Collaborator

If we can figure out how to maintain the original paths in the checkout this PR could potentially allow us to maintain the current cam structure in the face of the upcoming change in support for svn access to github.com

I'm pretty sure any attempt to do this will break all the git internals. Even the index becomes useless so manage_externals could not even get a status on such a repo.

@jedwards4b jedwards4b closed this Nov 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

sparse checkout using svn no longer works.
2 participants