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

Match longest repository path towards the shortest #950

Merged
merged 1 commit into from
Nov 18, 2015

Conversation

saper
Copy link
Contributor

@saper saper commented Nov 11, 2015

Try to to match longest repository paths first:

mediawiki/extensions/SemanticMediaWiki
mediawiki/extensions
mediawiki

Bug: #949

@saper
Copy link
Contributor Author

saper commented Nov 11, 2015

I think the better fix would be to move all the logic to something like RepositoryManager.getRepositoryFromURLPath(String), but I'm too unfamiliar with the code to propose a patch.

Try to to match longest repository paths first:

mediawiki/extensions/SemanticMediaWiki
mediawiki/extensions
mediawiki

Bug: gitblit-org#949
@paladox
Copy link
Contributor

paladox commented Nov 15, 2015

+1 I haven't tested but looks good.

And gitblit really does need support for these paths.

gitblit added a commit that referenced this pull request Nov 18, 2015
Match longest repository path towards the shortest
@gitblit gitblit merged commit a5cf407 into gitblit-org:master Nov 18, 2015
@gitblit
Copy link
Collaborator

gitblit commented Nov 18, 2015

@saper Thanks. Looks good, works fine.

@saper
Copy link
Contributor Author

saper commented Nov 18, 2015

Thank you!

paladox added a commit to paladox/gitblit that referenced this pull request Nov 23, 2015
gitblit pushed a commit that referenced this pull request Nov 23, 2015
This was originally from #950 but seems to have been reverted when develop branch was merged with master.
@flaix flaix modified the milestone: 1.8.0 Mar 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants