-
Notifications
You must be signed in to change notification settings - Fork 392
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
[JENKINS-47609] Add clone option "core.longpaths" to enable long file names. #523
[JENKINS-47609] Add clone option "core.longpaths" to enable long file names. #523
Conversation
You'll need to resolve the conflicts or rebase the changes on the tip of the upstream master branch. |
You may want to consider installing the
You create a new pull request with a single command:
You evaluate someone else's pull request with a single command:
|
I was actually trying to solve this with hub, I think I've done something wrong. I've resolved the merge conflicts, 72bb31e |
src/test/java/org/jenkinsci/plugins/gitclient/GitAPITestCase.java
Outdated
Show resolved
Hide resolved
src/test/java/org/jenkinsci/plugins/gitclient/GitAPITestCase.java
Outdated
Show resolved
Hide resolved
src/test/java/org/jenkinsci/plugins/gitclient/GitAPITestCase.java
Outdated
Show resolved
Hide resolved
src/test/java/org/jenkinsci/plugins/gitclient/GitAPITestCase.java
Outdated
Show resolved
Hide resolved
src/test/java/org/jenkinsci/plugins/gitclient/GitAPITestCase.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few relatively minor things for you to consider.
src/test/java/org/jenkinsci/plugins/gitclient/GitClientCloneTest.java
Outdated
Show resolved
Hide resolved
src/test/java/org/jenkinsci/plugins/gitclient/GitClientCloneTest.java
Outdated
Show resolved
Hide resolved
src/test/java/org/jenkinsci/plugins/gitclient/GitClientCloneTest.java
Outdated
Show resolved
Hide resolved
src/test/java/org/jenkinsci/plugins/gitclient/GitClientCloneTest.java
Outdated
Show resolved
Hide resolved
src/test/java/org/jenkinsci/plugins/gitclient/GitClientCloneTest.java
Outdated
Show resolved
Hide resolved
src/test/java/org/jenkinsci/plugins/gitclient/GitClientCloneTest.java
Outdated
Show resolved
Hide resolved
src/test/java/org/jenkinsci/plugins/gitclient/GitClientCloneTest.java
Outdated
Show resolved
Hide resolved
@MarkEWaite thanks for the review, I have updated the PR with the relevant changes (I'll try to make the code cleaner and semantically better next time). |
The new commit: add3695 address the issue of workspace directory not being able to be renamed to a 260 char file path. (Build was potentially failing because of this issue) |
@MarkEWaite I am facing an issue on Windows platform, This is happening while initializing the workspace for git clone. |
Probably indicates that the directory does not actually exist. Likely the request to create the directory failed because the name of the directory was too long. I believe Windows (and POSIX and Linux and BSD) limit the name of an individual file or directory to 255 characters. At least that is what I observed on my Windows 10 computer. I can create a directory with 255 characters in the name. I can't create a directory with a 256 character name. You may need to create two directories, one with a 200 character name, then a subdirectory with a name that is more than 60 characters. |
@rishabhBudhouliya thanks again for this pull request. In my CI infrastructure I had a test failing on windows due to not having long paths enabled. I look forward to this extension so that the submodule test in GitClient can use the extension to avoid the problem. |
…r long directory name
…ory path length>255char
@MarkEWaite thanks for the suggestions. While I took your advice to create two directories in (200 char + 60 char) format, I am still facing issues.
|
…o directory path length>255char" This is not possible since git init creates the workspace. This reverts commit 3b1f412.
This time I divided the directory structure, still I receive this issue: |
JENKINS-47609
Please refer to this PR: jenkinsci/git-plugin#856
This PR includes the changes in CloneCommand, CliGitAPIImpl and JGitAPIImpl to implement a new clone behavior which enables "core.longpaths".