googlecode doesn't seem to support shallow clients at all and we have already one conversion that broke silently
once the failed fetch has been done, the user has to remove the directory in git3-src/ manually before a non-shallow clone can work
It's not my fault if people commit stuff without testing. I think that I explicitly mentioned Google Code in most of my mails about it, and I've just updated the docstring to make that clear.
That proves that your eclass DEFAULT is not sane.
No. It only proves that Google can't sustain a compliant git server. Default is supposed to represent the *most common* case. Then, you can disable them per-ebuild as necessary. Going the other way is not that simple.
Is it really sane to shallow by default and expect people to disable it on "non-compliant git servers"? this also seems to break on local shallow checkouts as it tries to write to the repo (???)
* Fetching HEAD from /home/eroen/projects/git-annex-wrapper ...
git fetch --no-tags /home/eroen/projects/git-annex-wrapper --depth 1 -f HEAD:dev-python/ga_wrapper/0/__main__
* ACCESS DENIED: mkstemp64: /home/eroen/projects/git-annex-wrapper/.git/shallow_XXXXXX
fatal: Unable to create temporary file '/home/eroen/projects/git-annex-wrapper/.git/shallow_XXXXXX': Permission denied
(In reply to Rick Farina (Zero_Chaos) from comment #5)
> Is it really sane to shallow by default and expect people to disable it on
> "non-compliant git servers"?
It is better than expecting people to enable it globally and then disable per-package on packages that got broken.
> this also seems to break on local shallow
> checkouts as it tries to write to the repo (???)
> * Fetching HEAD from /home/eroen/projects/git-annex-wrapper ...
> git fetch --no-tags /home/eroen/projects/git-annex-wrapper --depth 1 -f
> * ACCESS DENIED: mkstemp64:
> fatal: Unable to create temporary file
> '/home/eroen/projects/git-annex-wrapper/.git/shallow_XXXXXX': Permission
Please report a bug.
(In reply to Michał Górny from comment #6)
> Please report a bug.
The eclass is going to get non-shallow support removed so we can close this.