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

'git clean -fdx' returns errorlevel 1 #521

Closed
talcloudshare opened this issue Nov 9, 2015 · 7 comments
Closed

'git clean -fdx' returns errorlevel 1 #521

talcloudshare opened this issue Nov 9, 2015 · 7 comments

Comments

@talcloudshare
Copy link

Running 'git clean -fd' or 'git clean -fx' works properly (i.e., errorlevel is 0 at the end of the execution)

Seems to only happen with certain (large) repositories

@talcloudshare
Copy link
Author

Before anyone asks, this does not happen with the linux Git cli

@dscho
Copy link
Member

dscho commented Nov 9, 2015

Before anyone asks, this does not happen with the linux Git cli

@talcloudshare that is not at all the first question I have... my first question is: are you absolutely sure you do not want to describe this issue in substantially more detail, including Git for Windows version, exact paste of the output of the command, URL to a repository with which this issue can be reproduced reliably?

@talcloudshare
Copy link
Author

I am using the latest version of git for windows (2.6.2)

When running 'git clean -xfd' there's no output other than the removed folders/files, running 'echo %errorlevel%' returns 1

C:\code\cloudshare>git clean -xfd                                               
Removing ...                                         
...                                                
Removing ...                                           

C:\code\cloudshare>echo %errorlevel%                                            
1

Unfortunately the repository is private and cannot be shared, however I will attempt to debug this myself and try to narrow down the problem so that I can provide more information. Will be helpful if you could guide me, currently following https://github.com/git-for-windows/git/wiki/Debugging-Git

@kgybels
Copy link

kgybels commented Nov 9, 2015

@dscho I can reproduce if I create long paths:

KGybels@PC-KG MINGW64 /c/git/bug
$ ls
steps.sh*

KGybels@PC-KG MINGW64 /c/git/bug
$ cat steps.sh
#!/bin/bash
longname=$(printf '%*s' 100 | tr ' ' a)
git init test
cd test
mkdir -p $longname/$longname/$longname
touch $longname/test.txt
touch $longname/$longname/$longname/test.txt
git clean -xdf
echo $?
KGybels@PC-KG MINGW64 /c/git/bug
$ ./steps.sh
Initialized empty Git repository in C:/git/bug/test/.git/
Removing aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa/test.txt
1

@talcloudshare Can you check for long paths in your case?

@kgybels
Copy link

kgybels commented Nov 9, 2015

@talcloudshare Enabling core.longpaths fixes it for my example.

Can you try with it enabled?
git config core.longpaths true

@talcloudshare
Copy link
Author

Thanks, this works for me
Was suspicious of a long path that cannot be cleaned, but there was no indicative output

@dscho
Copy link
Member

dscho commented Nov 10, 2015

So the issue is the missing error message.

The next steps would be:

  1. turn @kgybels' MCVE (thank you so much, this was the most helpful thing in this entire ticket) into a proper regression test, best would be t/t7300-clean.sh, using test_expect_success MINGW ...
  2. debug into the issue where the failure happens and why no error is shown
  3. add the error message
  4. open a Pull Request

@dscho dscho closed this as completed in fdf7375 Feb 16, 2016
dscho added a commit that referenced this issue Feb 16, 2016
In particular on Windows, where the default maximum path length is quite
small, but there are ways to circumvent that limit in many cases, it is
very important that users be given an indication why their command
failed because of too long paths when it did.

This test case makes sure that a warning is issued that would have
helped the user who reported Git for Windows' issue 521:

	#521

Signed-off-by: Johannes Schindelin <[email protected]>
dscho added a commit that referenced this issue Feb 16, 2016
This addresses #521

Signed-off-by: Johannes Schindelin <[email protected]>
dscho added a commit to git-for-windows/build-extra that referenced this issue Feb 16, 2016
When a too-long path is encountered, `git clean -dfx` [no longer
aborts quietly](git-for-windows/git#521).

Signed-off-by: Johannes Schindelin <[email protected]>
dscho added a commit that referenced this issue Feb 23, 2016
Without an error message when stat() failed, e.g. `git clean` would
abort without an error message, leaving the user quite puzzled.

This fixes #521

Signed-off-by: Johannes Schindelin <[email protected]>
dscho added a commit that referenced this issue Feb 23, 2016
In particular on Windows, where the default maximum path length is quite
small, but there are ways to circumvent that limit in many cases, it is
very important that users be given an indication why their command
failed because of too long paths when it did.

This test case makes sure that a warning is issued that would have
helped the user who reported Git for Windows' issue 521:

	#521

Signed-off-by: Johannes Schindelin <[email protected]>
dscho added a commit that referenced this issue Feb 23, 2016
This addresses #521

Signed-off-by: Johannes Schindelin <[email protected]>
dscho added a commit that referenced this issue Mar 15, 2016
Without an error message when stat() failed, e.g. `git clean` would
abort without an error message, leaving the user quite puzzled.

This fixes #521

Signed-off-by: Johannes Schindelin <[email protected]>
dscho added a commit that referenced this issue Mar 15, 2016
In particular on Windows, where the default maximum path length is quite
small, but there are ways to circumvent that limit in many cases, it is
very important that users be given an indication why their command
failed because of too long paths when it did.

This test case makes sure that a warning is issued that would have
helped the user who reported Git for Windows' issue 521:

	#521

Signed-off-by: Johannes Schindelin <[email protected]>
dscho added a commit that referenced this issue Mar 15, 2016
This addresses #521

Signed-off-by: Johannes Schindelin <[email protected]>
dscho added a commit that referenced this issue Mar 18, 2016
Without an error message when stat() failed, e.g. `git clean` would
abort without an error message, leaving the user quite puzzled.

This fixes #521

Signed-off-by: Johannes Schindelin <[email protected]>
dscho added a commit that referenced this issue Mar 18, 2016
In particular on Windows, where the default maximum path length is quite
small, but there are ways to circumvent that limit in many cases, it is
very important that users be given an indication why their command
failed because of too long paths when it did.

This test case makes sure that a warning is issued that would have
helped the user who reported Git for Windows' issue 521:

	#521

Signed-off-by: Johannes Schindelin <[email protected]>
dscho added a commit that referenced this issue Mar 18, 2016
This addresses #521

Signed-off-by: Johannes Schindelin <[email protected]>
git-for-windows-ci pushed a commit that referenced this issue Jul 15, 2019
Without an error message when stat() failed, e.g. `git clean` would
abort without an error message, leaving the user quite puzzled.

This fixes #521

Signed-off-by: Johannes Schindelin <[email protected]>
git-for-windows-ci pushed a commit that referenced this issue Jul 15, 2019
In particular on Windows, where the default maximum path length is quite
small, but there are ways to circumvent that limit in many cases, it is
very important that users be given an indication why their command
failed because of too long paths when it did.

This test case makes sure that a warning is issued that would have
helped the user who reported Git for Windows' issue 521:

	#521

Signed-off-by: Johannes Schindelin <[email protected]>
git-for-windows-ci pushed a commit that referenced this issue Jul 15, 2019
This addresses #521

Signed-off-by: Johannes Schindelin <[email protected]>
dscho added a commit that referenced this issue Jul 16, 2019
Without an error message when stat() failed, e.g. `git clean` would
abort without an error message, leaving the user quite puzzled.

This fixes #521

Signed-off-by: Johannes Schindelin <[email protected]>
dscho added a commit that referenced this issue Jul 16, 2019
In particular on Windows, where the default maximum path length is quite
small, but there are ways to circumvent that limit in many cases, it is
very important that users be given an indication why their command
failed because of too long paths when it did.

This test case makes sure that a warning is issued that would have
helped the user who reported Git for Windows' issue 521:

	#521

Signed-off-by: Johannes Schindelin <[email protected]>
dscho added a commit that referenced this issue Jul 16, 2019
This addresses #521

Signed-off-by: Johannes Schindelin <[email protected]>
git-for-windows-ci pushed a commit that referenced this issue Jul 16, 2019
Without an error message when stat() failed, e.g. `git clean` would
abort without an error message, leaving the user quite puzzled.

This fixes #521

Signed-off-by: Johannes Schindelin <[email protected]>
git-for-windows-ci pushed a commit that referenced this issue Jul 16, 2019
In particular on Windows, where the default maximum path length is quite
small, but there are ways to circumvent that limit in many cases, it is
very important that users be given an indication why their command
failed because of too long paths when it did.

This test case makes sure that a warning is issued that would have
helped the user who reported Git for Windows' issue 521:

	#521

Signed-off-by: Johannes Schindelin <[email protected]>
git-for-windows-ci pushed a commit that referenced this issue Jul 16, 2019
This addresses #521

Signed-off-by: Johannes Schindelin <[email protected]>
dscho added a commit to dscho/git that referenced this issue Jul 18, 2019
Without an error message when `lstat()` failed, `git clean` would
abort without an error message, leaving the user quite puzzled.

In particular on Windows, where the default maximum path length is quite
small (yet there are ways to circumvent that limit in many cases), it is
very important that users be given an indication why their command
failed because of too long paths when it did.

This test case makes sure that a warning is issued that would have
helped the user who reported this issue:

	git-for-windows#521

Note that we temporarily set `core.longpaths = false` in the regression
test; This ensures forward-compatibility with the `core.longpaths`
feature that has not yet been upstreamed from Git for Windows.

Helped-by: René Scharfe <[email protected]>
Helped-by: SZEDER Gábor <[email protected]>
Helped-by: Junio C Hamano <[email protected]>
Signed-off-by: Johannes Schindelin <[email protected]>
gitster pushed a commit to gitster/git that referenced this issue Jul 18, 2019
When `lstat()` failed, `git clean` would abort without an error
message, leaving the user quite puzzled.

In particular on Windows, where the default maximum path length is
quite small (yet there are ways to circumvent that limit in many
cases), it helps to give users an indication why their command
failed because of too long paths when it did.

This test case makes sure that a warning is issued that would have
helped the user who reported this issue:

	git-for-windows#521

Note that we temporarily set `core.longpaths = false` in the regression
test; this ensures forward-compatibility with the `core.longpaths`
feature that has not yet been upstreamed from Git for Windows.

Helped-by: René Scharfe <[email protected]>
Helped-by: SZEDER Gábor <[email protected]>
Helped-by: Junio C Hamano <[email protected]>
Signed-off-by: Johannes Schindelin <[email protected]>
Signed-off-by: Junio C Hamano <[email protected]>
gitster pushed a commit to gitster/git that referenced this issue Jul 19, 2019
When `lstat()` failed, `git clean` would abort without an error
message, leaving the user quite puzzled.

In particular on Windows, where the default maximum path length is
quite small (yet there are ways to circumvent that limit in many
cases), it is very important that users be given an indication why
their command failed because of too long paths when it did.

This test case makes sure that a warning is issued that would have
helped the user who reported this issue:

	git-for-windows#521

Note that we temporarily set `core.longpaths = false` in the regression
test; this ensures forward-compatibility with the `core.longpaths`
feature that has not yet been upstreamed from Git for Windows.

Helped-by: René Scharfe <[email protected]>
Helped-by: SZEDER Gábor <[email protected]>
Helped-by: Junio C Hamano <[email protected]>
Signed-off-by: Johannes Schindelin <[email protected]>
Signed-off-by: Junio C Hamano <[email protected]>
git-for-windows-ci pushed a commit that referenced this issue Jul 23, 2019
Without an error message when stat() failed, e.g. `git clean` would
abort without an error message, leaving the user quite puzzled.

This fixes #521

Signed-off-by: Johannes Schindelin <[email protected]>
git-for-windows-ci pushed a commit that referenced this issue Jul 23, 2019
In particular on Windows, where the default maximum path length is quite
small, but there are ways to circumvent that limit in many cases, it is
very important that users be given an indication why their command
failed because of too long paths when it did.

This test case makes sure that a warning is issued that would have
helped the user who reported Git for Windows' issue 521:

	#521

Signed-off-by: Johannes Schindelin <[email protected]>
git-for-windows-ci pushed a commit that referenced this issue Jul 23, 2019
This addresses #521

Signed-off-by: Johannes Schindelin <[email protected]>
git-for-windows-ci pushed a commit that referenced this issue Jul 23, 2019
Without an error message when stat() failed, e.g. `git clean` would
abort without an error message, leaving the user quite puzzled.

This fixes #521

Signed-off-by: Johannes Schindelin <[email protected]>
git-for-windows-ci pushed a commit that referenced this issue Jul 23, 2019
In particular on Windows, where the default maximum path length is quite
small, but there are ways to circumvent that limit in many cases, it is
very important that users be given an indication why their command
failed because of too long paths when it did.

This test case makes sure that a warning is issued that would have
helped the user who reported Git for Windows' issue 521:

	#521

Signed-off-by: Johannes Schindelin <[email protected]>
git-for-windows-ci pushed a commit that referenced this issue Jul 23, 2019
This addresses #521

Signed-off-by: Johannes Schindelin <[email protected]>
git-for-windows-ci pushed a commit that referenced this issue Jul 25, 2019
Without an error message when stat() failed, e.g. `git clean` would
abort without an error message, leaving the user quite puzzled.

This fixes #521

Signed-off-by: Johannes Schindelin <[email protected]>
git-for-windows-ci pushed a commit that referenced this issue Jul 25, 2019
In particular on Windows, where the default maximum path length is quite
small, but there are ways to circumvent that limit in many cases, it is
very important that users be given an indication why their command
failed because of too long paths when it did.

This test case makes sure that a warning is issued that would have
helped the user who reported Git for Windows' issue 521:

	#521

Signed-off-by: Johannes Schindelin <[email protected]>
git-for-windows-ci pushed a commit that referenced this issue Jul 25, 2019
This addresses #521

Signed-off-by: Johannes Schindelin <[email protected]>
dscho pushed a commit to dscho/git that referenced this issue Jul 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants