mdebray/tvl-depot
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

chore(web/cgit-taz): Remove patches which have been integrated

Browse source 
These patches now live as commits at //third_party/cgit
This commit is contained in:
Vincent Ambo 2020-01-11 23:46:20 +00:00
parent be2f51e1dd
commit 9e67451750
3 changed files with 0 additions and 210 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

114
web/cgit-taz/0001-cgit_monorepo_urls.patch
View file

@ -1,114 +0,0 @@
From f6646e5a6da29da979d6954feba9d85556bc6936 Mon Sep 17 00:00:00 2001
From: Vincent Ambo <tazjin@google.com>
Date: Sat, 21 Dec 2019 18:41:45 +0000
Subject: [PATCH 1/3] feat: Generate monorepo compatible URLs
Generates URLs that do not include the repository name.
On git.tazj.in, only one repository (depot) is served - hence URLs
generated by cgit need not include the name.
---
cmd.c | 24 +-----------------------
ui-shared.c | 29 +++++++++--------------------
2 files changed, 10 insertions(+), 43 deletions(-)
diff --git a/cmd.c b/cmd.c
index 63f0ae5..b37b79d 100644
--- a/cmd.c
+++ b/cmd.c
@@ -39,29 +39,7 @@ static void atom_fn(void)
static void about_fn(void)
{
- if (ctx.repo) {
- size_t path_info_len = ctx.env.path_info ? strlen(ctx.env.path_info) : 0;
- if (!ctx.qry.path &&
- ctx.qry.url[strlen(ctx.qry.url) - 1] != '/' &&
- (!path_info_len || ctx.env.path_info[path_info_len - 1] != '/')) {
- char *currenturl = cgit_currenturl();
- char *redirect = fmtalloc("%s/", currenturl);
- cgit_redirect(redirect, true);
- free(currenturl);
- free(redirect);
- } else if (ctx.repo->readme.nr)
- cgit_print_repo_readme(ctx.qry.path);
- else if (ctx.repo->homepage)
- cgit_redirect(ctx.repo->homepage, false);
- else {
- char *currenturl = cgit_currenturl();
- char *redirect = fmtalloc("%s../", currenturl);
- cgit_redirect(redirect, false);
- free(currenturl);
- free(redirect);
- }
- } else
- cgit_print_site_readme();
+ cgit_print_repo_readme(ctx.qry.path);
}
static void blame_fn(void)
diff --git a/ui-shared.c b/ui-shared.c
index 739505a..c7c3754 100644
--- a/ui-shared.c
+++ b/ui-shared.c
@@ -95,29 +95,23 @@ const char *cgit_loginurl(void)
char *cgit_repourl(const char *reponame)
{
- if (ctx.cfg.virtual_root)
- return fmtalloc("%s%s/", ctx.cfg.virtual_root, reponame);
- else
- return fmtalloc("?r=%s", reponame);
+ // my cgit instance *only* serves the depot, hence that's the only value ever
+ // needed.
+ return fmtalloc("/");
}
char *cgit_fileurl(const char *reponame, const char *pagename,
const char *filename, const char *query)
{
struct strbuf sb = STRBUF_INIT;
- char *delim;
- if (ctx.cfg.virtual_root) {
- strbuf_addf(&sb, "%s%s/%s/%s", ctx.cfg.virtual_root, reponame,
- pagename, (filename ? filename:""));
- delim = "?";
- } else {
- strbuf_addf(&sb, "?url=%s/%s/%s", reponame, pagename,
- (filename ? filename : ""));
- delim = "&amp;";
+ strbuf_addf(&sb, "%s%s/%s", ctx.cfg.virtual_root,
+ pagename, (filename ? filename:""));
+
+ if (query) {
+ strbuf_addf(&sb, "%s%s", "?", query);
}
- if (query)
- strbuf_addf(&sb, "%s%s", delim, query);
+
return strbuf_detach(&sb, NULL);
}
@@ -245,9 +239,6 @@ static char *repolink(const char *title, const char *class, const char *page,
html(" href='");
if (ctx.cfg.virtual_root) {
html_url_path(ctx.cfg.virtual_root);
- html_url_path(ctx.repo->url);
- if (ctx.repo->url[strlen(ctx.repo->url) - 1] != '/')
- html("/");
if (page) {
html_url_path(page);
html("/");
@@ -957,8 +948,6 @@ static void print_header(void)
html("<td class='main'>");
if (ctx.repo) {
- cgit_index_link("index", NULL, NULL, NULL, NULL, 0, 1);
- html(" : ");
cgit_summary_link(ctx.repo->name, ctx.repo->name, NULL, NULL);
if (ctx.env.authenticated) {
html("</td><td class='form'>");
--
2.24.1.735.g03f4e72817-goog

46
web/cgit-taz/0002-cgit_subtree_readmes.patch
View file

@ -1,46 +0,0 @@
From 61500898c7d1363f88b763c7778cf1a8dfd13aca Mon Sep 17 00:00:00 2001
From: Vincent Ambo <tazjin@google.com>
Date: Sat, 21 Dec 2019 22:58:19 +0000
Subject: [PATCH 2/3] feat(ui-summary): Attempt to use README at each subtree
This means that individual subtrees of a repository will also have
their READMEs rendered on the about page, for example:
/foo/bar/README.md
Will render on:
/about/foo/bar/
This is useful for monorepo setups in which subtrees represent
individual projects.
---
ui-summary.c | 12 ++++++++++++
1 file changed, 12 insertions(+)
diff --git a/ui-summary.c b/ui-summary.c
index 8e81ac4..34ce4e9 100644
--- a/ui-summary.c
+++ b/ui-summary.c
@@ -128,6 +128,18 @@ void cgit_print_repo_readme(char *path)
goto done;
}
+ /* Determine which file to serve by checking whether the given filename is
+ * already a valid file and otherwise appending the expected file name of
+ * the readme.
+ *
+ * If neither yield a valid file, the user gets a blank page. Could probably
+ * do with an error message in between there, but whatever.
+ */
+ if (path && ref && !cgit_ref_path_exists(filename, ref, 1)) {
+ filename = fmtalloc("%s/%s", path, ctx.repo->readme.items[0].string);
+ free_filename = 1;
+ }
+
/* Print the calculated readme, either from the git repo or from the
* filesystem, while applying the about-filter.
*/
--
2.24.1.735.g03f4e72817-goog

50
web/cgit-taz/0003-cgit_subtree_about_links.patch
View file

@ -1,50 +0,0 @@
From 531b55dc96bb7ee2ce52a3612021e1c1f4ddac8a Mon Sep 17 00:00:00 2001
From: Vincent Ambo <tazjin@google.com>
Date: Sat, 21 Dec 2019 23:27:28 +0000
Subject: [PATCH 3/3] feat(ui-shared): Generate links to about pages from
subtrees
If you're on tree/foo/bar, the about link will now point to
about/foo/bar.
Currently the annoying thing about this is that it will also do it for
files.
---
ui-shared.c | 14 ++++++++++----
1 file changed, 10 insertions(+), 4 deletions(-)
diff --git a/ui-shared.c b/ui-shared.c
index c7c3754..c37835a 100644
--- a/ui-shared.c
+++ b/ui-shared.c
@@ -297,6 +297,12 @@ void cgit_tag_link(const char *name, const char *title, const char *class,
reporevlink("tag", name, title, class, tag, NULL, NULL);
}
+void cgit_about_link(const char *name, const char *title, const char *class,
+ const char *head, const char *rev, const char *path)
+{
+ reporevlink("about", name, title, class, head, rev, path);
+}
+
void cgit_tree_link(const char *name, const char *title, const char *class,
const char *head, const char *rev, const char *path)
{
@@ -985,10 +991,10 @@ void cgit_print_pageheader(void)
html("<table class='tabs'><tr><td>\n");
if (ctx.env.authenticated && ctx.repo) {
- if (ctx.repo->readme.nr)
- reporevlink("about", "about", NULL,
- hc("about"), ctx.qry.head, NULL,
- NULL);
+ if (ctx.repo->readme.nr) {
+ cgit_about_link("about", NULL, hc("about"), ctx.qry.head,
+ ctx.qry.sha1, ctx.qry.vpath);
+ }
cgit_summary_link("summary", NULL, hc("summary"),
ctx.qry.head);
cgit_refs_link("refs", NULL, hc("refs"), ctx.qry.head,
--
2.24.1.735.g03f4e72817-goog