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

t5580: verify that alternates can be UNC paths #140

Closed
wants to merge 1 commit into from

Conversation

dscho
Copy link
Member

@dscho dscho commented Feb 24, 2019

This was fixed in git.git's master some time ago, unfortunately not upstreamed from Git for Windows' patch thicket (my fault, or better put: the fault of the twenty-four hour days not being longer). We carried a regression test on top of the fix, and I can at least contribute that.

On Windows, UNC paths are a very convenient way to share data, and
alternates are all about sharing data.

We fixed a bug where alternates specifying UNC paths were not handled
properly, and it is high time that we add a regression test to ensure
that this bug is not reintroduced.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
@dscho dscho added the ready to submit Has commits that have not been submitted yet label Feb 24, 2019
@dscho
Copy link
Member Author

dscho commented Feb 25, 2019

/submit

@dscho dscho removed the ready to submit Has commits that have not been submitted yet label Feb 25, 2019
@gitgitgadget
Copy link

gitgitgadget bot commented Feb 25, 2019

Submitted as pull.140.git.gitgitgadget@gmail.com

@gitgitgadget
Copy link

gitgitgadget bot commented May 7, 2019

This branch is now known as js/t5580-unc-alternate-test.

@gitgitgadget
Copy link

gitgitgadget bot commented May 7, 2019

This patch series was integrated into pu via git@f0c0adb.

@gitgitgadget gitgitgadget bot added the pu label May 7, 2019
@gitgitgadget
Copy link

gitgitgadget bot commented May 8, 2019

This patch series was integrated into pu via git@ae31419.

@gitgitgadget
Copy link

gitgitgadget bot commented May 8, 2019

This patch series was integrated into pu via git@d616b47.

@gitgitgadget
Copy link

gitgitgadget bot commented May 9, 2019

This patch series was integrated into pu via git@60839fe.

@gitgitgadget
Copy link

gitgitgadget bot commented May 10, 2019

This patch series was integrated into pu via git@380c54e.

@gitgitgadget
Copy link

gitgitgadget bot commented May 13, 2019

This patch series was integrated into pu via git@ddc59e9.

@gitgitgadget
Copy link

gitgitgadget bot commented May 13, 2019

This patch series was integrated into pu via git@34a0e77.

@gitgitgadget
Copy link

gitgitgadget bot commented May 15, 2019

This patch series was integrated into pu via git@6a538c1.

@gitgitgadget
Copy link

gitgitgadget bot commented May 15, 2019

This patch series was integrated into next via git@dfff4a6.

@gitgitgadget gitgitgadget bot added the next label May 15, 2019
@gitgitgadget
Copy link

gitgitgadget bot commented May 16, 2019

This patch series was integrated into pu via git@0684276.

@gitgitgadget
Copy link

gitgitgadget bot commented May 19, 2019

This patch series was integrated into pu via git@0b076b4.

@gitgitgadget
Copy link

gitgitgadget bot commented May 19, 2019

This patch series was integrated into next via git@0b076b4.

@gitgitgadget
Copy link

gitgitgadget bot commented May 19, 2019

This patch series was integrated into master via git@0b076b4.

@gitgitgadget gitgitgadget bot added the master label May 19, 2019
@gitgitgadget
Copy link

gitgitgadget bot commented May 19, 2019

Closed via 0b076b4.

@gitgitgadget gitgitgadget bot closed this May 19, 2019
@dscho dscho deleted the test-unc-alternates branch May 20, 2019 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant