Fix Git.get_url_and_commit raising for some Git configs

[Solviana]

Changelog: Bugfix: Fix Git.get_url_and_commit raising for some Git configs.
Docs: Omit

• Refer to the issue that supports this Pull Request.
• If the issue has missing info, explain the purpose/use case/pain/need that covers this Pull Request.
• I've read the Contributing guide.
• I've followed the PEP8 style guides for Python code.
• I've opened another PR in the Conan docs repo to the develop branch, documenting this one.

[CLAassistant]

All committers have signed the CLA.

[memsharded]

It would be necessary to add a test that covers the need for this, so if someone sometime removes it, the test will fail.

There are a bunch of tests in test_git.py that you could use as a base, but let us know if you need help, we can even try to contribute the test if necessary.

[Solviana]

It would be necessary to add a test that covers the need for this, so if someone sometime removes it, the test will fail.

There are a bunch of tests in test_git.py that you could use as a base, but let us know if you need help, we can even try to contribute the test if necessary.

I'll get familiar with your setup and I will create the tests, no prob!

[memsharded]

Hi @Solviana

I am submitting a very related PR to fix a Git.is_dirty() bug, see #15289, you can see a unit test there, I hope it helps.

If you'd like to include this fix in 2.0.15, it will be most likely branched tomorrow, if you could manage to do the test, please let me know, and I'll try to include it.

[Solviana]

@memsharded

I was afk for a while but now I updated the PR with some tests. Can you review this again?

[memsharded]

Look great, thanks very much!

[AbrilRBS]

The new flags are 10 and 15 years old, respectively - no need for us to worry about that I think heh.

I've now modified the PR message, and will merge this to be released for 2.1, thanks a lot for your contribution @Solviana, we really appreciate it :)