Fix an error that occurred when using conan.tools.scm.Git.fetch_commit() in a subfolder

[bkarasm]

Changelog: Fix: Fixed an error that occurred when using conan.tools.scm.Git.fetch_commit() in a subfolder.
Docs: Omit

Fixes #17295

[CLAassistant]

All committers have signed the CLA.

[memsharded]

Looks good, thanks very much @bkarasm

[memsharded]

Please don't forget to sign the CLA above (or if you signed, sometimes when not using the same email as the Github one, Github doesn't recognize the signature), so we can merge it for next release. Thanks again for your contribution!

[memsharded]

Seems tests are broken in Linux only (work in Windows and Mac):

TestGitShallowClone.test_clone_to_subfolder

onan.errors.ConanException: Command 'git fetch --depth 1 origin 52bdb239b6917d3fb2d01849c78e07985f1722b3' failed with errorcode '1'

E           b'error: Server does not allow request for unadvertised object 52bdb239b6917d3fb2d01849c78e07985f1722b3\n'

[bkarasm]

@memsharded I'll take a look at the failing tests later today. Regarding the CLA I already signed it using the same account that I contributed with. Not sure why it still shows it is pending.

[bkarasm]

A different test that verifies git.fetch_commit() has this pytest marker: @pytest.mark.skipif(platform.system() == "Linux", reason="Git version in Linux not support it"). I added similar marker for the failing test.

[memsharded]

Thanks for following up!

The CI is now green, ready to be merged.
Still the CLA says it has not been approved. I have checked the git log output, and I'd say it is possible that the commits are signed with other email different the one you are using for your Github account, could you please double check?

[bkarasm]

Thanks @memsharded . Indeed that was the case. It should be fixed now.

[memsharded]

Merged, it will be in next Conan 2.10. Thanks very much for your contribution!