Skip to content

Match longest repository path towards the shortest - #950

Merged
gitblit merged 1 commit into
gitblit-org:masterfrom
saper:searchlast
Nov 18, 2015
Merged

gitblit merged 1 commit into
gitblit-org:masterfrom
saper:searchlast

Conversation

@saper

@saper saper commented Nov 11, 2015

Copy link
Copy Markdown
Contributor

Try to to match longest repository paths first:

mediawiki/extensions/SemanticMediaWiki
mediawiki/extensions
mediawiki

Bug: #949

@saper

saper commented Nov 11, 2015

Copy link
Copy Markdown
Contributor Author

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

paladox commented Nov 15, 2015

Copy link
Copy Markdown
Contributor

+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

gitblit commented Nov 18, 2015

Copy link
Copy Markdown
Collaborator

@saper Thanks. Looks good, works fine.

@saper

saper commented Nov 18, 2015

Copy link
Copy Markdown
Contributor Author

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.

4 participants