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.

Sign up for GitHub

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

Jump to bottom

clean: suggest using core.longPaths if paths are too long to remove #3817

Merged
merged 1 commit into from
Apr 22, 2022
Merged

clean: suggest using core.longPaths if paths are too long to remove #3817

merged 1 commit into from
Apr 22, 2022

Conversation

mathstuf
Copy link

@mathstuf mathstuf commented Apr 22, 2022 •
edited
Loading

On Windows, git repositories may have extra files which need cleaned
(e.g., a build directory) that may be arbitrarily deep. Suggest using
core.longPaths if such situations are encountered.

Fixes: #2715
Signed-off-by: Ben Boeckel [email protected]

@mathstuf
clean: suggest using core.longPaths if paths are too long to remove
6fc4995 
On Windows, git repositories may have extra files which need cleaned
(e.g., a build directory) that may be arbitrarily deep. Suggest using
`core.longPaths` if such situations are encountered.

Fixes: #2715
Signed-off-by: Ben Boeckel <[email protected]>
@mathstuf mathstuf force-pushed the name-too-long-advice branch from b6df087 to 6fc4995 Compare April 22, 2022 13:13
dscho
dscho approved these changes Apr 22, 2022
View reviewed changes
Copy link
Member

@dscho dscho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thank you very much!

@dscho dscho merged commit a8150f0 into git-for-windows:main Apr 22, 2022
dscho added a commit to git-for-windows/build-extra that referenced this pull request Apr 22, 2022
@dscho
Mention New Feature in release notes
3b3d601 
When `git clean` fails due to long
paths, [Git now advises the user to set
`core.longPaths`](git-for-windows/git#3817).

Signed-off-by: Johannes Schindelin <[email protected]>
@dscho dscho added this to the Next release milestone Apr 22, 2022
@mathstuf mathstuf deleted the name-too-long-advice branch April 22, 2022 23:10
dscho added a commit that referenced this pull request Apr 26, 2022
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
08c9192 
clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit that referenced this pull request Apr 26, 2022
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
5594ffb 
clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit that referenced this pull request Apr 26, 2022
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
f846b33 
clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit that referenced this pull request Apr 26, 2022
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
8ea2563 
clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit that referenced this pull request Apr 28, 2022
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
c22f8e9 
clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit that referenced this pull request Apr 28, 2022
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
4f06c9c 
clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit that referenced this pull request Apr 28, 2022
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
805c71f 
clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit that referenced this pull request Apr 29, 2022
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
11e06aa 
clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit that referenced this pull request Apr 29, 2022
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
e340cb2 
clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit that referenced this pull request Apr 29, 2022
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
f635b89 
clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit that referenced this pull request Apr 29, 2022
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
ae7c254 
clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit that referenced this pull request Apr 29, 2022
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
33d1578 
clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit that referenced this pull request May 2, 2022
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
5abc407 
clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit that referenced this pull request May 3, 2022
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
a021781 
clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit that referenced this pull request May 3, 2022
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
0d2d568 
clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit that referenced this pull request May 3, 2022
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
5905a02 
clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit that referenced this pull request May 5, 2022
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
e490c75 
clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit that referenced this pull request May 5, 2022
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
be9e85e 
clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit that referenced this pull request May 5, 2022
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
698114f 
clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit that referenced this pull request May 9, 2022
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
75119ca 
clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit that referenced this pull request May 9, 2022
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
4576196 
clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit that referenced this pull request May 9, 2022
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
61c2154 
clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit that referenced this pull request May 9, 2022
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
405378a 
clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit to dscho/git that referenced this pull request Dec 30, 2024
@dscho
Merge pull request git-for-windows#3817 from mathstuf/name-too-long-a…
9fad607 
…dvice

clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit to dscho/git that referenced this pull request Dec 30, 2024
@dscho
Merge pull request git-for-windows#3817 from mathstuf/name-too-long-a…
9f7e930 
…dvice

clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit to dscho/git that referenced this pull request Dec 30, 2024
@dscho
Merge pull request git-for-windows#3817 from mathstuf/name-too-long-a…
b7ef73a 
…dvice

clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit to dscho/git that referenced this pull request Dec 30, 2024
@dscho
Merge pull request git-for-windows#3817 from mathstuf/name-too-long-a…
7dd492b 
…dvice

clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit to dscho/git that referenced this pull request Dec 30, 2024
@dscho
Merge pull request git-for-windows#3817 from mathstuf/name-too-long-a…
9b0258a 
…dvice

clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit to dscho/git that referenced this pull request Dec 30, 2024
@dscho
Merge pull request git-for-windows#3817 from mathstuf/name-too-long-a…
08d4169 
…dvice

clean: suggest using `core.longPaths` if paths are too long to remove
git-for-windows-ci pushed a commit that referenced this pull request Dec 31, 2024
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
0abd589 
clean: suggest using `core.longPaths` if paths are too long to remove
git-for-windows-ci pushed a commit that referenced this pull request Dec 31, 2024
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
e65391e 
clean: suggest using `core.longPaths` if paths are too long to remove
git-for-windows-ci pushed a commit that referenced this pull request Jan 1, 2025
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
06745b0 
clean: suggest using `core.longPaths` if paths are too long to remove
git-for-windows-ci pushed a commit that referenced this pull request Jan 1, 2025
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
f2992ae 
clean: suggest using `core.longPaths` if paths are too long to remove
git-for-windows-ci pushed a commit that referenced this pull request Jan 1, 2025
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
9436898 
clean: suggest using `core.longPaths` if paths are too long to remove
git-for-windows-ci pushed a commit that referenced this pull request Jan 1, 2025
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
58734e5 
clean: suggest using `core.longPaths` if paths are too long to remove
git-for-windows-ci pushed a commit that referenced this pull request Jan 1, 2025
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
0ee2ad7 
clean: suggest using `core.longPaths` if paths are too long to remove
git-for-windows-ci pushed a commit that referenced this pull request Jan 2, 2025
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
40f52c1 
clean: suggest using `core.longPaths` if paths are too long to remove
git-for-windows-ci pushed a commit that referenced this pull request Jan 2, 2025
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
847b9b2 
clean: suggest using `core.longPaths` if paths are too long to remove
git-for-windows-ci pushed a commit that referenced this pull request Jan 2, 2025
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
0d79454 
clean: suggest using `core.longPaths` if paths are too long to remove
git-for-windows-ci pushed a commit that referenced this pull request Jan 3, 2025
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
b15cedb 
clean: suggest using `core.longPaths` if paths are too long to remove
git-for-windows-ci pushed a commit that referenced this pull request Jan 6, 2025
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
478156a 
clean: suggest using `core.longPaths` if paths are too long to remove
git-for-windows-ci pushed a commit that referenced this pull request Jan 6, 2025
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
e3a0ced 
clean: suggest using `core.longPaths` if paths are too long to remove
git-for-windows-ci pushed a commit that referenced this pull request Jan 7, 2025
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
a541096 
clean: suggest using `core.longPaths` if paths are too long to remove
git-for-windows-ci pushed a commit that referenced this pull request Jan 7, 2025
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
41a642f 
clean: suggest using `core.longPaths` if paths are too long to remove
@dscho dscho mentioned this pull request Jan 7, 2025
Open
dscho added a commit to dscho/git that referenced this pull request Jan 7, 2025
@dscho
Merge pull request git-for-windows#3817 from mathstuf/name-too-long-a…
9f071d6 
…dvice

clean: suggest using `core.longPaths` if paths are too long to remove
dscho added a commit to dscho/git that referenced this pull request Jan 7, 2025
@dscho
Merge pull request git-for-windows#3817 from mathstuf/name-too-long-a…
635b3f1 
…dvice

clean: suggest using `core.longPaths` if paths are too long to remove
git-for-windows-ci pushed a commit that referenced this pull request Jan 7, 2025
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
b8c7485 
clean: suggest using `core.longPaths` if paths are too long to remove
git-for-windows-ci pushed a commit that referenced this pull request Jan 7, 2025
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
1ffe853 
clean: suggest using `core.longPaths` if paths are too long to remove
git-for-windows-ci pushed a commit that referenced this pull request Jan 8, 2025
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
7aa1d5c 
clean: suggest using `core.longPaths` if paths are too long to remove
git-for-windows-ci pushed a commit that referenced this pull request Jan 8, 2025
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
604a199 
clean: suggest using `core.longPaths` if paths are too long to remove
git-for-windows-ci pushed a commit that referenced this pull request Jan 8, 2025
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
870b88b 
clean: suggest using `core.longPaths` if paths are too long to remove
git-for-windows-ci pushed a commit that referenced this pull request Jan 8, 2025
@dscho
Merge pull request #3817 from mathstuf/name-too-long-advice
8303513 
clean: suggest using `core.longPaths` if paths are too long to remove
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dscho dscho dscho approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v2.36.1
Development

Successfully merging this pull request may close these issues.

clean cannot remove paths with long names
2 participants
@mathstuf @dscho