search

racket / racket-pkg-website

A frontend for the Racket Package Catalog.
Other
11 stars 16 forks source link

Constructed package source URLs don't fit the grammar #7

Closed tonyg closed 8 years ago

tonyg commented 8 years ago

Bug report from Matthew:

[When adding a package,] The default options generated the URL

   git://github.com/mflatt/glu-tessellate/master

at least on the package's info page, but that's not a valid package source. It should be

   git://github.com/mflatt/glu-tessellate#master

or, better in the case of master, just

   git://github.com/mflatt/glu-tessellate

The short form correctly showed up in the preview, but if I type a branch name, the format is wrong there, too.

tonyg commented 8 years ago

We also need to ensure good support for repo URLs like

    git://github.com/tonyg/racket-operational-transformation?path=operational-transformation-lib

and, of course, combinations of path and branch and so forth.

tonyg commented 8 years ago

Closed by 4776595e7977d559cd775b4e66be4cc5ede61813. Hopefully.