Add a .git for git repositories
Closed, ResolvedPublic
Actions

Assigned To

None

Authored By

	enko
	Nov 28 2013, 4:02 PM

Description

I don't know if it sounds reasonable, but it would be cool if you could add an alias /diffusion/<callsign>.git for git repositories, as some software require it.

I tried adding this:

diff --git a/src/applications/diffusion/application/PhabricatorApplicationDiffusion.php b/src/applications/diffusion/application/PhabricatorApplicationDiffusion.php
index 9ca2c6a..0677eac 100644
--- a/src/applications/diffusion/application/PhabricatorApplicationDiffusion.php
+++ b/src/applications/diffusion/application/PhabricatorApplicationDiffusion.php
@@ -46,6 +46,7 @@ final class PhabricatorApplicationDiffusion extends PhabricatorApplication {
         'new/' => 'DiffusionRepositoryNewController',
         '(?P<edit>create)/' => 'DiffusionRepositoryCreateController',
         '(?P<edit>import)/' => 'DiffusionRepositoryCreateController',
+       '(?P<callsign>[A-Z]+)\.git/' => 'DiffusionRepositoryDefaultController',
         '(?P<callsign>[A-Z]+)/' => array(
           '' => 'DiffusionRepositoryController',

But that was not so much of a success.

Revisions and Commits

rPHU libphutil
		D8099	rPHUe60dca3649f4 Add "%R" to csprintf(), for printing more readable commands
rP Phabricator
	Abandoned		D8090 Show full clone commands instead of URIs
		D8100	rP3bfa54819e45 Use new "%R" escape for csprintf() to produce slightly nicer clone/checkout…
		D8098	rPc41b4cfac0a8 Allow Git and Mercurial repositories to be cloned with names in the URI
		D8097	rPffeee3781019 Add "Clone As" to repositories and generate full clone commands in UI

Related Objects

Duplicates Merged Here: T4201: Need friendlier names for hosted repositories

Event Timeline

enko created this task.Nov 28 2013, 4:02 PM

enko raised the priority of this task from to Wishlist.

enko updated the task description. (Show Details)

enko added a project: Diffusion.

enko added a subscriber: enko.

enko assigned this task to epriestley.Nov 29 2013, 10:13 AM

chad removed epriestley as the assignee of this task.Nov 29 2013, 3:56 PM

chad added subscribers: epriestley, chad.

A slightly more flexible approach might be:

/diffusion/CALLSIGN/anything.git

...which would let you provide something for "anything" to name the default working copy something human-readable (that is, when you do a clone right now you get "CALLSIGN/", but it would be slightly nicer to clone "/diffusion/CALLSIGN/project.git" and get "project/").

I held off on this initially because I wasn't sure we could do it without breaking stuff, but now that we've implemented things I believe it's possible (albeit somewhat more involved than the one liner above).

One downside is that having unlimited canonical URIs for the project is kind of sketchy. We can hard-code the Phabricator equivalence rules into Phabricator, but we can't reasonably expect other tools to know that "/CALLSIGN/anything.git" is the same for any value of "anything".

packagist's behavior here also seems a bit dumb. If the only motivation in the wild is "packagist is written dumb", maybe a cleaner fix is in the packagist upstream. Or you could work around it by registering git.derp.blarp.com and pointing that at your install as a secondary URI to trick packagist.