diff options
-rw-r--r-- | Makefile | 6 | ||||
-rw-r--r-- | cgit.c | 42 | ||||
-rw-r--r-- | cgit.css | 27 | ||||
-rw-r--r-- | cgit.h | 8 | ||||
-rw-r--r-- | cgitrc.5.txt | 67 | ||||
-rw-r--r-- | cmd.c | 44 | ||||
-rw-r--r-- | cmd.h | 3 | ||||
-rwxr-xr-x | filters/commit-links.sh | 18 | ||||
-rwxr-xr-x | filters/syntax-highlighting.sh | 11 | ||||
-rw-r--r-- | html.c | 8 | ||||
-rw-r--r-- | html.h | 1 | ||||
-rw-r--r-- | shared.c | 29 | ||||
-rwxr-xr-x | tests/setup.sh | 1 | ||||
-rwxr-xr-x | tests/t0102-summary.sh | 6 | ||||
-rw-r--r-- | ui-commit.c | 10 | ||||
-rw-r--r-- | ui-diff.c | 83 | ||||
-rw-r--r-- | ui-diff.h | 4 | ||||
-rw-r--r-- | ui-log.c | 29 | ||||
-rw-r--r-- | ui-repolist.c | 15 | ||||
-rw-r--r-- | ui-stats.c | 51 | ||||
-rw-r--r-- | ui-summary.c | 2 |
21 files changed, 341 insertions, 124 deletions
@@ -198,9 +198,9 @@ install-pdf: doc-pdf | |||
198 | $(INSTALL) -m 0644 $(DOC_PDF) $(DESTDIR)$(pdfdir) | 198 | $(INSTALL) -m 0644 $(DOC_PDF) $(DESTDIR)$(pdfdir) |
199 | 199 | ||
200 | uninstall: | 200 | uninstall: |
201 | rm -f $(CGIT_SCRIPT_PATH)/$(CGIT_SCRIPT_NAME) | 201 | rm -f $(DESTDIR)$(CGIT_SCRIPT_PATH)/$(CGIT_SCRIPT_NAME) |
202 | rm -f $(CGIT_DATA_PATH)/cgit.css | 202 | rm -f $(DESTDIR)$(CGIT_DATA_PATH)/cgit.css |
203 | rm -f $(CGIT_DATA_PATH)/cgit.png | 203 | rm -f $(DESTDIR)$(CGIT_DATA_PATH)/cgit.png |
204 | 204 | ||
205 | uninstall-doc: uninstall-man uninstall-html uninstall-pdf | 205 | uninstall-doc: uninstall-man uninstall-html uninstall-pdf |
206 | 206 | ||
@@ -26,14 +26,27 @@ void add_mimetype(const char *name, const char *value) | |||
26 | item->util = xstrdup(value); | 26 | item->util = xstrdup(value); |
27 | } | 27 | } |
28 | 28 | ||
29 | struct cgit_filter *new_filter(const char *cmd, int extra_args) | 29 | struct cgit_filter *new_filter(const char *cmd, filter_type filtertype) |
30 | { | 30 | { |
31 | struct cgit_filter *f; | 31 | struct cgit_filter *f; |
32 | int args_size = 0; | 32 | int args_size = 0; |
33 | int extra_args; | ||
33 | 34 | ||
34 | if (!cmd || !cmd[0]) | 35 | if (!cmd || !cmd[0]) |
35 | return NULL; | 36 | return NULL; |
36 | 37 | ||
38 | switch (filtertype) { | ||
39 | case SOURCE: | ||
40 | extra_args = 1; | ||
41 | break; | ||
42 | |||
43 | case ABOUT: | ||
44 | case COMMIT: | ||
45 | default: | ||
46 | extra_args = 0; | ||
47 | break; | ||
48 | } | ||
49 | |||
37 | f = xmalloc(sizeof(struct cgit_filter)); | 50 | f = xmalloc(sizeof(struct cgit_filter)); |
38 | f->cmd = xstrdup(cmd); | 51 | f->cmd = xstrdup(cmd); |
39 | args_size = (2 + extra_args) * sizeof(char *); | 52 | args_size = (2 + extra_args) * sizeof(char *); |
@@ -83,11 +96,11 @@ void repo_config(struct cgit_repo *repo, const char *name, const char *value) | |||
83 | repo->logo_link = xstrdup(value); | 96 | repo->logo_link = xstrdup(value); |
84 | else if (ctx.cfg.enable_filter_overrides) { | 97 | else if (ctx.cfg.enable_filter_overrides) { |
85 | if (!strcmp(name, "about-filter")) | 98 | if (!strcmp(name, "about-filter")) |
86 | repo->about_filter = new_filter(value, 0); | 99 | repo->about_filter = new_filter(value, ABOUT); |
87 | else if (!strcmp(name, "commit-filter")) | 100 | else if (!strcmp(name, "commit-filter")) |
88 | repo->commit_filter = new_filter(value, 0); | 101 | repo->commit_filter = new_filter(value, COMMIT); |
89 | else if (!strcmp(name, "source-filter")) | 102 | else if (!strcmp(name, "source-filter")) |
90 | repo->source_filter = new_filter(value, 1); | 103 | repo->source_filter = new_filter(value, SOURCE); |
91 | } | 104 | } |
92 | } | 105 | } |
93 | 106 | ||
@@ -147,6 +160,8 @@ void config_cb(const char *name, const char *value) | |||
147 | ctx.cfg.enable_filter_overrides = atoi(value); | 160 | ctx.cfg.enable_filter_overrides = atoi(value); |
148 | else if (!strcmp(name, "enable-gitweb-owner")) | 161 | else if (!strcmp(name, "enable-gitweb-owner")) |
149 | ctx.cfg.enable_gitweb_owner = atoi(value); | 162 | ctx.cfg.enable_gitweb_owner = atoi(value); |
163 | else if (!strcmp(name, "enable-http-clone")) | ||
164 | ctx.cfg.enable_http_clone = atoi(value); | ||
150 | else if (!strcmp(name, "enable-index-links")) | 165 | else if (!strcmp(name, "enable-index-links")) |
151 | ctx.cfg.enable_index_links = atoi(value); | 166 | ctx.cfg.enable_index_links = atoi(value); |
152 | else if (!strcmp(name, "enable-commit-graph")) | 167 | else if (!strcmp(name, "enable-commit-graph")) |
@@ -178,9 +193,9 @@ void config_cb(const char *name, const char *value) | |||
178 | else if (!strcmp(name, "cache-dynamic-ttl")) | 193 | else if (!strcmp(name, "cache-dynamic-ttl")) |
179 | ctx.cfg.cache_dynamic_ttl = atoi(value); | 194 | ctx.cfg.cache_dynamic_ttl = atoi(value); |
180 | else if (!strcmp(name, "about-filter")) | 195 | else if (!strcmp(name, "about-filter")) |
181 | ctx.cfg.about_filter = new_filter(value, 0); | 196 | ctx.cfg.about_filter = new_filter(value, ABOUT); |
182 | else if (!strcmp(name, "commit-filter")) | 197 | else if (!strcmp(name, "commit-filter")) |
183 | ctx.cfg.commit_filter = new_filter(value, 0); | 198 | ctx.cfg.commit_filter = new_filter(value, COMMIT); |
184 | else if (!strcmp(name, "embedded")) | 199 | else if (!strcmp(name, "embedded")) |
185 | ctx.cfg.embedded = atoi(value); | 200 | ctx.cfg.embedded = atoi(value); |
186 | else if (!strcmp(name, "max-atom-items")) | 201 | else if (!strcmp(name, "max-atom-items")) |
@@ -210,7 +225,7 @@ void config_cb(const char *name, const char *value) | |||
210 | else if (!strcmp(name, "section-from-path")) | 225 | else if (!strcmp(name, "section-from-path")) |
211 | ctx.cfg.section_from_path = atoi(value); | 226 | ctx.cfg.section_from_path = atoi(value); |
212 | else if (!strcmp(name, "source-filter")) | 227 | else if (!strcmp(name, "source-filter")) |
213 | ctx.cfg.source_filter = new_filter(value, 1); | 228 | ctx.cfg.source_filter = new_filter(value, SOURCE); |
214 | else if (!strcmp(name, "summary-log")) | 229 | else if (!strcmp(name, "summary-log")) |
215 | ctx.cfg.summary_log = atoi(value); | 230 | ctx.cfg.summary_log = atoi(value); |
216 | else if (!strcmp(name, "summary-branches")) | 231 | else if (!strcmp(name, "summary-branches")) |
@@ -229,6 +244,8 @@ void config_cb(const char *name, const char *value) | |||
229 | ctx.cfg.robots = xstrdup(value); | 244 | ctx.cfg.robots = xstrdup(value); |
230 | else if (!strcmp(name, "clone-prefix")) | 245 | else if (!strcmp(name, "clone-prefix")) |
231 | ctx.cfg.clone_prefix = xstrdup(value); | 246 | ctx.cfg.clone_prefix = xstrdup(value); |
247 | else if (!strcmp(name, "clone-url")) | ||
248 | ctx.cfg.clone_url = xstrdup(value); | ||
232 | else if (!strcmp(name, "local-time")) | 249 | else if (!strcmp(name, "local-time")) |
233 | ctx.cfg.local_time = atoi(value); | 250 | ctx.cfg.local_time = atoi(value); |
234 | else if (!prefixcmp(name, "mimetype.")) | 251 | else if (!prefixcmp(name, "mimetype.")) |
@@ -313,6 +330,7 @@ static void prepare_context(struct cgit_context *ctx) | |||
313 | ctx->cfg.logo = "/cgit.png"; | 330 | ctx->cfg.logo = "/cgit.png"; |
314 | ctx->cfg.local_time = 0; | 331 | ctx->cfg.local_time = 0; |
315 | ctx->cfg.enable_gitweb_owner = 1; | 332 | ctx->cfg.enable_gitweb_owner = 1; |
333 | ctx->cfg.enable_http_clone = 1; | ||
316 | ctx->cfg.enable_tree_linenumbers = 1; | 334 | ctx->cfg.enable_tree_linenumbers = 1; |
317 | ctx->cfg.max_repo_count = 50; | 335 | ctx->cfg.max_repo_count = 50; |
318 | ctx->cfg.max_commit_count = 50; | 336 | ctx->cfg.max_commit_count = 50; |
@@ -444,7 +462,7 @@ static int prepare_repo_cmd(struct cgit_context *ctx) | |||
444 | tmp = xstrdup(ctx->qry.head); | 462 | tmp = xstrdup(ctx->qry.head); |
445 | ctx->qry.head = ctx->repo->defbranch; | 463 | ctx->qry.head = ctx->repo->defbranch; |
446 | ctx->page.status = 404; | 464 | ctx->page.status = 404; |
447 | ctx->page.statusmsg = "not found"; | 465 | ctx->page.statusmsg = "Not found"; |
448 | cgit_print_http_headers(ctx); | 466 | cgit_print_http_headers(ctx); |
449 | cgit_print_docstart(ctx); | 467 | cgit_print_docstart(ctx); |
450 | cgit_print_pageheader(ctx); | 468 | cgit_print_pageheader(ctx); |
@@ -452,6 +470,7 @@ static int prepare_repo_cmd(struct cgit_context *ctx) | |||
452 | cgit_print_docend(); | 470 | cgit_print_docend(); |
453 | return 1; | 471 | return 1; |
454 | } | 472 | } |
473 | cgit_prepare_repo_env(ctx->repo); | ||
455 | return 0; | 474 | return 0; |
456 | } | 475 | } |
457 | 476 | ||
@@ -463,6 +482,8 @@ static void process_request(void *cbdata) | |||
463 | cmd = cgit_get_cmd(ctx); | 482 | cmd = cgit_get_cmd(ctx); |
464 | if (!cmd) { | 483 | if (!cmd) { |
465 | ctx->page.title = "cgit error"; | 484 | ctx->page.title = "cgit error"; |
485 | ctx->page.status = 404; | ||
486 | ctx->page.statusmsg = "Not found"; | ||
466 | cgit_print_http_headers(ctx); | 487 | cgit_print_http_headers(ctx); |
467 | cgit_print_docstart(ctx); | 488 | cgit_print_docstart(ctx); |
468 | cgit_print_pageheader(ctx); | 489 | cgit_print_pageheader(ctx); |
@@ -471,6 +492,11 @@ static void process_request(void *cbdata) | |||
471 | return; | 492 | return; |
472 | } | 493 | } |
473 | 494 | ||
495 | if (!ctx->cfg.enable_http_clone && cmd->is_clone) { | ||
496 | html_status(404, "Not found", 0); | ||
497 | return; | ||
498 | } | ||
499 | |||
474 | /* If cmd->want_vpath is set, assume ctx->qry.path contains a "virtual" | 500 | /* If cmd->want_vpath is set, assume ctx->qry.path contains a "virtual" |
475 | * in-project path limit to be made available at ctx->qry.vpath. | 501 | * in-project path limit to be made available at ctx->qry.vpath. |
476 | * Otherwise, no path limit is in effect (ctx->qry.vpath = NULL). | 502 | * Otherwise, no path limit is in effect (ctx->qry.vpath = NULL). |
@@ -331,6 +331,33 @@ table.commit-info { | |||
331 | margin-top: 1.5em; | 331 | margin-top: 1.5em; |
332 | } | 332 | } |
333 | 333 | ||
334 | div.cgit-panel { | ||
335 | float: right; | ||
336 | margin-top: 1.5em; | ||
337 | } | ||
338 | |||
339 | div.cgit-panel table { | ||
340 | border-collapse: collapse; | ||
341 | border: solid 1px #aaa; | ||
342 | background-color: #eee; | ||
343 | } | ||
344 | |||
345 | div.cgit-panel th { | ||
346 | text-align: center; | ||
347 | } | ||
348 | |||
349 | div.cgit-panel td { | ||
350 | padding: 0.25em 0.5em; | ||
351 | } | ||
352 | |||
353 | div.cgit-panel td.label { | ||
354 | padding-right: 0.5em; | ||
355 | } | ||
356 | |||
357 | div.cgit-panel td.ctrl { | ||
358 | padding-left: 0.5em; | ||
359 | } | ||
360 | |||
334 | table.commit-info th { | 361 | table.commit-info th { |
335 | text-align: left; | 362 | text-align: left; |
336 | font-weight: normal; | 363 | font-weight: normal; |
@@ -51,6 +51,10 @@ typedef void (*configfn)(const char *name, const char *value); | |||
51 | typedef void (*filepair_fn)(struct diff_filepair *pair); | 51 | typedef void (*filepair_fn)(struct diff_filepair *pair); |
52 | typedef void (*linediff_fn)(char *line, int len); | 52 | typedef void (*linediff_fn)(char *line, int len); |
53 | 53 | ||
54 | typedef enum { | ||
55 | ABOUT, COMMIT, SOURCE | ||
56 | } filter_type; | ||
57 | |||
54 | struct cgit_filter { | 58 | struct cgit_filter { |
55 | char *cmd; | 59 | char *cmd; |
56 | char **argv; | 60 | char **argv; |
@@ -162,6 +166,7 @@ struct cgit_config { | |||
162 | char *agefile; | 166 | char *agefile; |
163 | char *cache_root; | 167 | char *cache_root; |
164 | char *clone_prefix; | 168 | char *clone_prefix; |
169 | char *clone_url; | ||
165 | char *css; | 170 | char *css; |
166 | char *favicon; | 171 | char *favicon; |
167 | char *footer; | 172 | char *footer; |
@@ -192,6 +197,7 @@ struct cgit_config { | |||
192 | int embedded; | 197 | int embedded; |
193 | int enable_filter_overrides; | 198 | int enable_filter_overrides; |
194 | int enable_gitweb_owner; | 199 | int enable_gitweb_owner; |
200 | int enable_http_clone; | ||
195 | int enable_index_links; | 201 | int enable_index_links; |
196 | int enable_commit_graph; | 202 | int enable_commit_graph; |
197 | int enable_log_filecount; | 203 | int enable_log_filecount; |
@@ -318,6 +324,8 @@ extern int cgit_parse_snapshots_mask(const char *str); | |||
318 | extern int cgit_open_filter(struct cgit_filter *filter); | 324 | extern int cgit_open_filter(struct cgit_filter *filter); |
319 | extern int cgit_close_filter(struct cgit_filter *filter); | 325 | extern int cgit_close_filter(struct cgit_filter *filter); |
320 | 326 | ||
327 | extern void cgit_prepare_repo_env(struct cgit_repo * repo); | ||
328 | |||
321 | extern int readfile(const char *path, char **buf, size_t *size); | 329 | extern int readfile(const char *path, char **buf, size_t *size); |
322 | 330 | ||
323 | extern char *expand_macros(const char *txt); | 331 | extern char *expand_macros(const char *txt); |
diff --git a/cgitrc.5.txt b/cgitrc.5.txt index 26c4bfc..a22423b 100644 --- a/cgitrc.5.txt +++ b/cgitrc.5.txt | |||
@@ -31,7 +31,7 @@ about-filter:: | |||
31 | about pages (both top-level and for each repository). The command will | 31 | about pages (both top-level and for each repository). The command will |
32 | get the content of the about-file on its STDIN, and the STDOUT from the | 32 | get the content of the about-file on its STDIN, and the STDOUT from the |
33 | command will be included verbatim on the about page. Default value: | 33 | command will be included verbatim on the about page. Default value: |
34 | none. | 34 | none. See also: "FILTER API". |
35 | 35 | ||
36 | agefile:: | 36 | agefile:: |
37 | Specifies a path, relative to each repository path, which can be used | 37 | Specifies a path, relative to each repository path, which can be used |
@@ -76,11 +76,17 @@ clone-prefix:: | |||
76 | setting is only used if `repo.clone-url` is unspecified. Default value: | 76 | setting is only used if `repo.clone-url` is unspecified. Default value: |
77 | none. | 77 | none. |
78 | 78 | ||
79 | clone-url:: | ||
80 | Space-separated list of clone-url templates. This setting is only | ||
81 | used if `repo.clone-url` is unspecified. Default value: none. See | ||
82 | also: "MACRO EXPANSION", "FILTER API". | ||
83 | |||
79 | commit-filter:: | 84 | commit-filter:: |
80 | Specifies a command which will be invoked to format commit messages. | 85 | Specifies a command which will be invoked to format commit messages. |
81 | The command will get the message on its STDIN, and the STDOUT from the | 86 | The command will get the message on its STDIN, and the STDOUT from the |
82 | command will be included verbatim as the commit message, i.e. this can | 87 | command will be included verbatim as the commit message, i.e. this can |
83 | be used to implement bugtracker integration. Default value: none. | 88 | be used to implement bugtracker integration. Default value: none. |
89 | See also: "FILTER API". | ||
84 | 90 | ||
85 | css:: | 91 | css:: |
86 | Url which specifies the css document to include in all cgit pages. | 92 | Url which specifies the css document to include in all cgit pages. |
@@ -105,6 +111,11 @@ enable-gitweb-owner:: | |||
105 | for the git config value "gitweb.owner" to determine the owner. | 111 | for the git config value "gitweb.owner" to determine the owner. |
106 | Default value: "1". See also: scan-path. | 112 | Default value: "1". See also: scan-path. |
107 | 113 | ||
114 | enable-http-clone:: | ||
115 | If set to "1", cgit will act as an dumb HTTP endpoint for git clones. | ||
116 | If you use an alternate way of serving git repositories, you may wish | ||
117 | to disable this. Default value: "1". | ||
118 | |||
108 | enable-index-links:: | 119 | enable-index-links:: |
109 | Flag which, when set to "1", will make cgit generate extra links for | 120 | Flag which, when set to "1", will make cgit generate extra links for |
110 | each repo in the repository index (specifically, to the "summary", | 121 | each repo in the repository index (specifically, to the "summary", |
@@ -320,7 +331,7 @@ source-filter:: | |||
320 | and the name of the blob as its only command line argument. The STDOUT | 331 | and the name of the blob as its only command line argument. The STDOUT |
321 | from the command will be included verbatim as the blob contents, i.e. | 332 | from the command will be included verbatim as the blob contents, i.e. |
322 | this can be used to implement e.g. syntax highlighting. Default value: | 333 | this can be used to implement e.g. syntax highlighting. Default value: |
323 | none. | 334 | none. See also: "FILTER API". |
324 | 335 | ||
325 | summary-branches:: | 336 | summary-branches:: |
326 | Specifies the number of branches to display in the repository "summary" | 337 | Specifies the number of branches to display in the repository "summary" |
@@ -353,15 +364,15 @@ REPOSITORY SETTINGS | |||
353 | ------------------- | 364 | ------------------- |
354 | repo.about-filter:: | 365 | repo.about-filter:: |
355 | Override the default about-filter. Default value: none. See also: | 366 | Override the default about-filter. Default value: none. See also: |
356 | "enable-filter-overrides". | 367 | "enable-filter-overrides". See also: "FILTER API". |
357 | 368 | ||
358 | repo.clone-url:: | 369 | repo.clone-url:: |
359 | A list of space-separated urls which can be used to clone this repo. | 370 | A list of space-separated urls which can be used to clone this repo. |
360 | Default value: none. | 371 | Default value: none. See also: "MACRO EXPANSION". |
361 | 372 | ||
362 | repo.commit-filter:: | 373 | repo.commit-filter:: |
363 | Override the default commit-filter. Default value: none. See also: | 374 | Override the default commit-filter. Default value: none. See also: |
364 | "enable-filter-overrides". | 375 | "enable-filter-overrides". See also: "FILTER API". |
365 | 376 | ||
366 | repo.defbranch:: | 377 | repo.defbranch:: |
367 | The name of the default branch for this repository. If no such branch | 378 | The name of the default branch for this repository. If no such branch |
@@ -438,7 +449,7 @@ repo.section:: | |||
438 | 449 | ||
439 | repo.source-filter:: | 450 | repo.source-filter:: |
440 | Override the default source-filter. Default value: none. See also: | 451 | Override the default source-filter. Default value: none. See also: |
441 | "enable-filter-overrides". | 452 | "enable-filter-overrides". See also: "FILTER API". |
442 | 453 | ||
443 | repo.url:: | 454 | repo.url:: |
444 | The relative url used to access the repository. This must be the first | 455 | The relative url used to access the repository. This must be the first |
@@ -458,6 +469,40 @@ Note: the "repo." prefix is dropped from the option names in repo-specific | |||
458 | config files, e.g. "repo.desc" becomes "desc". | 469 | config files, e.g. "repo.desc" becomes "desc". |
459 | 470 | ||
460 | 471 | ||
472 | FILTER API | ||
473 | ---------- | ||
474 | about filter:: | ||
475 | This filter is given no arguments. The about text that is to be | ||
476 | filtered is available on standard input and the filtered text is | ||
477 | expected on standard output. | ||
478 | |||
479 | commit filter:: | ||
480 | This filter is given no arguments. The commit message text that is to | ||
481 | be filtered is available on standard input and the filtered text is | ||
482 | expected on standard output. | ||
483 | |||
484 | source filter:: | ||
485 | This filter is given a single parameter: the filename of the source | ||
486 | file to filter. The filter can use the filename to determine (for | ||
487 | example) the syntax highlighting mode. The contents of the source | ||
488 | file that is to be filtered is available on standard input and the | ||
489 | filtered contents is expected on standard output. | ||
490 | |||
491 | Also, all filters are handed the following environment variables: | ||
492 | |||
493 | - CGIT_REPO_URL (from repo.url) | ||
494 | - CGIT_REPO_NAME (from repo.name) | ||
495 | - CGIT_REPO_PATH (from repo.path) | ||
496 | - CGIT_REPO_OWNER (from repo.owner) | ||
497 | - CGIT_REPO_DEFBRANCH (from repo.defbranch) | ||
498 | - CGIT_REPO_SECTION (from repo.section) | ||
499 | - CGIT_REPO_CLONE_URL (from repo.clone-url) | ||
500 | |||
501 | If a setting is not defined for a repository and the corresponding global | ||
502 | setting is also not defined (if applicable), then the corresponding | ||
503 | environment variable will be unset. | ||
504 | |||
505 | |||
461 | MACRO EXPANSION | 506 | MACRO EXPANSION |
462 | --------------- | 507 | --------------- |
463 | The following cgitrc options supports a simple macro expansion feature, | 508 | The following cgitrc options supports a simple macro expansion feature, |
@@ -477,6 +522,12 @@ can be accomplished by adding the following line to /etc/cgitrc: | |||
477 | 522 | ||
478 | include=/etc/cgitrc.d/$HTTP_HOST | 523 | include=/etc/cgitrc.d/$HTTP_HOST |
479 | 524 | ||
525 | The following options are expanded during request processing, and support | ||
526 | the environment variables defined in "FILTER API": | ||
527 | |||
528 | - clone-url | ||
529 | - repo.clone-url | ||
530 | |||
480 | 531 | ||
481 | EXAMPLE CGITRC FILE | 532 | EXAMPLE CGITRC FILE |
482 | ------------------- | 533 | ------------------- |
@@ -486,8 +537,8 @@ EXAMPLE CGITRC FILE | |||
486 | cache-size=1000 | 537 | cache-size=1000 |
487 | 538 | ||
488 | 539 | ||
489 | # Specify some default clone prefixes | 540 | # Specify some default clone urls using macro expansion |
490 | clone-prefix=git://example.com ssh://example.com/pub/git http://example.com/git | 541 | clone-url=git://foo.org/$CGIT_REPO_URL git@foo.org:$CGIT_REPO_URL |
491 | 542 | ||
492 | # Specify the css url | 543 | # Specify the css url |
493 | css=/css/cgit.css | 544 | css=/css/cgit.css |
@@ -56,7 +56,7 @@ static void commit_fn(struct cgit_context *ctx) | |||
56 | 56 | ||
57 | static void diff_fn(struct cgit_context *ctx) | 57 | static void diff_fn(struct cgit_context *ctx) |
58 | { | 58 | { |
59 | cgit_print_diff(ctx->qry.sha1, ctx->qry.sha2, ctx->qry.path); | 59 | cgit_print_diff(ctx->qry.sha1, ctx->qry.sha2, ctx->qry.path, 1); |
60 | } | 60 | } |
61 | 61 | ||
62 | static void info_fn(struct cgit_context *ctx) | 62 | static void info_fn(struct cgit_context *ctx) |
@@ -130,31 +130,31 @@ static void tree_fn(struct cgit_context *ctx) | |||
130 | cgit_print_tree(ctx->qry.sha1, ctx->qry.path); | 130 | cgit_print_tree(ctx->qry.sha1, ctx->qry.path); |
131 | } | 131 | } |
132 | 132 | ||
133 | #define def_cmd(name, want_repo, want_layout, want_vpath) \ | 133 | #define def_cmd(name, want_repo, want_layout, want_vpath, is_clone) \ |
134 | {#name, name##_fn, want_repo, want_layout, want_vpath} | 134 | {#name, name##_fn, want_repo, want_layout, want_vpath, is_clone} |
135 | 135 | ||
136 | struct cgit_cmd *cgit_get_cmd(struct cgit_context *ctx) | 136 | struct cgit_cmd *cgit_get_cmd(struct cgit_context *ctx) |
137 | { | 137 | { |
138 | static struct cgit_cmd cmds[] = { | 138 | static struct cgit_cmd cmds[] = { |
139 | def_cmd(HEAD, 1, 0, 0), | 139 | def_cmd(HEAD, 1, 0, 0, 1), |
140 | def_cmd(atom, 1, 0, 0), | 140 | def_cmd(atom, 1, 0, 0, 0), |
141 | def_cmd(about, 0, 1, 0), | 141 | def_cmd(about, 0, 1, 0, 0), |
142 | def_cmd(blob, 1, 0, 0), | 142 | def_cmd(blob, 1, 0, 0, 0), |
143 | def_cmd(commit, 1, 1, 1), | 143 | def_cmd(commit, 1, 1, 1, 0), |
144 | def_cmd(diff, 1, 1, 1), | 144 | def_cmd(diff, 1, 1, 1, 0), |
145 | def_cmd(info, 1, 0, 0), | 145 | def_cmd(info, 1, 0, 0, 1), |
146 | def_cmd(log, 1, 1, 1), | 146 | def_cmd(log, 1, 1, 1, 0), |
147 | def_cmd(ls_cache, 0, 0, 0), | 147 | def_cmd(ls_cache, 0, 0, 0, 0), |
148 | def_cmd(objects, 1, 0, 0), | 148 | def_cmd(objects, 1, 0, 0, 1), |
149 | def_cmd(patch, 1, 0, 1), | 149 | def_cmd(patch, 1, 0, 1, 0), |
150 | def_cmd(plain, 1, 0, 0), | 150 | def_cmd(plain, 1, 0, 0, 0), |
151 | def_cmd(refs, 1, 1, 0), | 151 | def_cmd(refs, 1, 1, 0, 0), |
152 | def_cmd(repolist, 0, 0, 0), | 152 | def_cmd(repolist, 0, 0, 0, 0), |
153 | def_cmd(snapshot, 1, 0, 0), | 153 | def_cmd(snapshot, 1, 0, 0, 0), |
154 | def_cmd(stats, 1, 1, 1), | 154 | def_cmd(stats, 1, 1, 1, 0), |
155 | def_cmd(summary, 1, 1, 0), | 155 | def_cmd(summary, 1, 1, 0, 0), |
156 | def_cmd(tag, 1, 1, 0), | 156 | def_cmd(tag, 1, 1, 0, 0), |
157 | def_cmd(tree, 1, 1, 1), | 157 | def_cmd(tree, 1, 1, 1, 0), |
158 | }; | 158 | }; |
159 | int i; | 159 | int i; |
160 | 160 | ||
@@ -8,7 +8,8 @@ struct cgit_cmd { | |||
8 | cgit_cmd_fn fn; | 8 | cgit_cmd_fn fn; |
9 | unsigned int want_repo:1, | 9 | unsigned int want_repo:1, |
10 | want_layout:1, | 10 | want_layout:1, |
11 | want_vpath:1; | 11 | want_vpath:1, |
12 | is_clone:1; | ||
12 | }; | 13 | }; |
13 | 14 | ||
14 | extern struct cgit_cmd *cgit_get_cmd(struct cgit_context *ctx); | 15 | extern struct cgit_cmd *cgit_get_cmd(struct cgit_context *ctx); |
diff --git a/filters/commit-links.sh b/filters/commit-links.sh index 110c609..5881952 100755 --- a/filters/commit-links.sh +++ b/filters/commit-links.sh | |||
@@ -3,12 +3,26 @@ | |||
3 | # | 3 | # |
4 | # To use this script, refer to this file with either the commit-filter or the | 4 | # To use this script, refer to this file with either the commit-filter or the |
5 | # repo.commit-filter options in cgitrc. | 5 | # repo.commit-filter options in cgitrc. |
6 | # | ||
7 | # The following environment variables can be used to retrieve the configuration | ||
8 | # of the repository for which this script is called: | ||
9 | # CGIT_REPO_URL ( = repo.url setting ) | ||
10 | # CGIT_REPO_NAME ( = repo.name setting ) | ||
11 | # CGIT_REPO_PATH ( = repo.path setting ) | ||
12 | # CGIT_REPO_OWNER ( = repo.owner setting ) | ||
13 | # CGIT_REPO_DEFBRANCH ( = repo.defbranch setting ) | ||
14 | # CGIT_REPO_SECTION ( = section setting ) | ||
15 | # CGIT_REPO_CLONE_URL ( = repo.clone-url setting ) | ||
16 | # | ||
17 | |||
18 | regex='' | ||
6 | 19 | ||
7 | # This expression generates links to commits referenced by their SHA1. | 20 | # This expression generates links to commits referenced by their SHA1. |
8 | regex=$regex' | 21 | regex=$regex' |
9 | s|\b([0-9a-fA-F]{8,40})\b|<a href="./?id=\1">\1</a>|g' | 22 | s|\b([0-9a-fA-F]{7,40})\b|<a href="./?id=\1">\1</a>|g' |
23 | |||
10 | # This expression generates links to a fictional bugtracker. | 24 | # This expression generates links to a fictional bugtracker. |
11 | regex=$regex' | 25 | regex=$regex' |
12 | s| #([0-9]+)\b|<a href="http://bugs.example.com/?bug=\1">#\1</a>|g' | 26 | s|#([0-9]+)\b|<a href="http://bugs.example.com/?bug=\1">#\1</a>|g' |
13 | 27 | ||
14 | sed -re "$regex" | 28 | sed -re "$regex" |
diff --git a/filters/syntax-highlighting.sh b/filters/syntax-highlighting.sh index 6b1c576..6283ce9 100755 --- a/filters/syntax-highlighting.sh +++ b/filters/syntax-highlighting.sh | |||
@@ -23,6 +23,17 @@ | |||
23 | # table.blob .kwb { color:#830000; } | 23 | # table.blob .kwb { color:#830000; } |
24 | # table.blob .kwc { color:#000000; font-weight:bold; } | 24 | # table.blob .kwc { color:#000000; font-weight:bold; } |
25 | # table.blob .kwd { color:#010181; } | 25 | # table.blob .kwd { color:#010181; } |
26 | # | ||
27 | # The following environment variables can be used to retrieve the configuration | ||
28 | # of the repository for which this script is called: | ||
29 | # CGIT_REPO_URL ( = repo.url setting ) | ||
30 | # CGIT_REPO_NAME ( = repo.name setting ) | ||
31 | # CGIT_REPO_PATH ( = repo.path setting ) | ||
32 | # CGIT_REPO_OWNER ( = repo.owner setting ) | ||
33 | # CGIT_REPO_DEFBRANCH ( = repo.defbranch setting ) | ||
34 | # CGIT_REPO_SECTION ( = section setting ) | ||
35 | # CGIT_REPO_CLONE_URL ( = repo.clone-url setting ) | ||
36 | # | ||
26 | 37 | ||
27 | # store filename and extension in local vars | 38 | # store filename and extension in local vars |
28 | BASENAME="$1" | 39 | BASENAME="$1" |
@@ -215,6 +215,14 @@ void html_option(const char *value, const char *text, const char *selected_value | |||
215 | html("</option>\n"); | 215 | html("</option>\n"); |
216 | } | 216 | } |
217 | 217 | ||
218 | void html_intoption(int value, const char *text, int selected_value) | ||
219 | { | ||
220 | htmlf("<option value='%d'%s>", value, | ||
221 | value == selected_value ? " selected='selected'" : ""); | ||
222 | html_txt(text); | ||
223 | html("</option>"); | ||
224 | } | ||
225 | |||
218 | void html_link_open(const char *url, const char *title, const char *class) | 226 | void html_link_open(const char *url, const char *title, const char *class) |
219 | { | 227 | { |
220 | html("<a href='"); | 228 | html("<a href='"); |
@@ -17,6 +17,7 @@ extern void html_url_path(const char *txt); | |||
17 | extern void html_url_arg(const char *txt); | 17 | extern void html_url_arg(const char *txt); |
18 | extern void html_hidden(const char *name, const char *value); | 18 | extern void html_hidden(const char *name, const char *value); |
19 | extern void html_option(const char *value, const char *text, const char *selected_value); | 19 | extern void html_option(const char *value, const char *text, const char *selected_value); |
20 | extern void html_intoption(int value, const char *text, int selected_value); | ||
20 | extern void html_link_open(const char *url, const char *title, const char *class); | 21 | extern void html_link_open(const char *url, const char *title, const char *class); |
21 | extern void html_link_close(void); | 22 | extern void html_link_close(void); |
22 | extern void html_fileperm(unsigned short mode); | 23 | extern void html_fileperm(unsigned short mode); |
@@ -7,6 +7,7 @@ | |||
7 | */ | 7 | */ |
8 | 8 | ||
9 | #include "cgit.h" | 9 | #include "cgit.h" |
10 | #include <stdio.h> | ||
10 | 11 | ||
11 | struct cgit_repolist cgit_repolist; | 12 | struct cgit_repolist cgit_repolist; |
12 | struct cgit_context ctx; | 13 | struct cgit_context ctx; |
@@ -68,6 +69,7 @@ struct cgit_repo *cgit_add_repo(const char *url) | |||
68 | ret->about_filter = ctx.cfg.about_filter; | 69 | ret->about_filter = ctx.cfg.about_filter; |
69 | ret->commit_filter = ctx.cfg.commit_filter; | 70 | ret->commit_filter = ctx.cfg.commit_filter; |
70 | ret->source_filter = ctx.cfg.source_filter; | 71 | ret->source_filter = ctx.cfg.source_filter; |
72 | ret->clone_url = ctx.cfg.clone_url; | ||
71 | return ret; | 73 | return ret; |
72 | } | 74 | } |
73 | 75 | ||
@@ -367,6 +369,33 @@ int cgit_parse_snapshots_mask(const char *str) | |||
367 | return rv; | 369 | return rv; |
368 | } | 370 | } |
369 | 371 | ||
372 | typedef struct { | ||
373 | char * name; | ||
374 | char * value; | ||
375 | } cgit_env_var; | ||
376 | |||
377 | void cgit_prepare_repo_env(struct cgit_repo * repo) | ||
378 | { | ||
379 | cgit_env_var env_vars[] = { | ||
380 | { .name = "CGIT_REPO_URL", .value = repo->url }, | ||
381 | { .name = "CGIT_REPO_NAME", .value = repo->name }, | ||
382 | { .name = "CGIT_REPO_PATH", .value = repo->path }, | ||
383 | { .name = "CGIT_REPO_OWNER", .value = repo->owner }, | ||
384 | { .name = "CGIT_REPO_DEFBRANCH", .value = repo->defbranch }, | ||
385 | { .name = "CGIT_REPO_SECTION", .value = repo->section }, | ||
386 | { .name = "CGIT_REPO_CLONE_URL", .value = repo->clone_url } | ||
387 | }; | ||
388 | int env_var_count = ARRAY_SIZE(env_vars); | ||
389 | cgit_env_var *p, *q; | ||
390 | static char *warn = "cgit warning: failed to set env: %s=%s\n"; | ||
391 | |||
392 | p = env_vars; | ||
393 | q = p + env_var_count; | ||
394 | for (; p < q; p++) | ||
395 | if (p->value && setenv(p->name, p->value, 1)) | ||
396 | fprintf(stderr, warn, p->name, p->value); | ||
397 | } | ||
398 | |||
370 | int cgit_open_filter(struct cgit_filter *filter) | 399 | int cgit_open_filter(struct cgit_filter *filter) |
371 | { | 400 | { |
372 | 401 | ||
diff --git a/tests/setup.sh b/tests/setup.sh index b9475a3..b21908d 100755 --- a/tests/setup.sh +++ b/tests/setup.sh | |||
@@ -63,6 +63,7 @@ enable-log-linecount=1 | |||
63 | summary-log=5 | 63 | summary-log=5 |
64 | summary-branches=5 | 64 | summary-branches=5 |
65 | summary-tags=5 | 65 | summary-tags=5 |
66 | clone-url=git://example.org/\$CGIT_REPO_URL.git | ||
66 | 67 | ||
67 | repo.url=foo | 68 | repo.url=foo |
68 | repo.path=$PWD/trash/repos/foo/.git | 69 | repo.path=$PWD/trash/repos/foo/.git |
diff --git a/tests/t0102-summary.sh b/tests/t0102-summary.sh index f0b0d9a..f299c5a 100755 --- a/tests/t0102-summary.sh +++ b/tests/t0102-summary.sh | |||
@@ -9,6 +9,9 @@ run_test 'find commit 1' 'grep -e "commit 1" trash/tmp' | |||
9 | run_test 'find commit 5' 'grep -e "commit 5" trash/tmp' | 9 | run_test 'find commit 5' 'grep -e "commit 5" trash/tmp' |
10 | run_test 'find branch master' 'grep -e "master" trash/tmp' | 10 | run_test 'find branch master' 'grep -e "master" trash/tmp' |
11 | run_test 'no tags' '! grep -e "tags" trash/tmp' | 11 | run_test 'no tags' '! grep -e "tags" trash/tmp' |
12 | run_test 'clone-url expanded correctly' ' | ||
13 | grep -e "git://example.org/foo.git" trash/tmp | ||
14 | ' | ||
12 | 15 | ||
13 | run_test 'generate bar summary' 'cgit_url "bar" >trash/tmp' | 16 | run_test 'generate bar summary' 'cgit_url "bar" >trash/tmp' |
14 | run_test 'no commit 45' '! grep -e "commit 45" trash/tmp' | 17 | run_test 'no commit 45' '! grep -e "commit 45" trash/tmp' |
@@ -16,5 +19,8 @@ run_test 'find commit 46' 'grep -e "commit 46" trash/tmp' | |||
16 | run_test 'find commit 50' 'grep -e "commit 50" trash/tmp' | 19 | run_test 'find commit 50' 'grep -e "commit 50" trash/tmp' |
17 | run_test 'find branch master' 'grep -e "master" trash/tmp' | 20 | run_test 'find branch master' 'grep -e "master" trash/tmp' |
18 | run_test 'no tags' '! grep -e "tags" trash/tmp' | 21 | run_test 'no tags' '! grep -e "tags" trash/tmp' |
22 | run_test 'clone-url expanded correctly' ' | ||
23 | grep -e "git://example.org/bar.git" trash/tmp | ||
24 | ' | ||
19 | 25 | ||
20 | tests_done | 26 | tests_done |
diff --git a/ui-commit.c b/ui-commit.c index 2b4f677..536a8e8 100644 --- a/ui-commit.c +++ b/ui-commit.c | |||
@@ -39,7 +39,8 @@ void cgit_print_commit(char *hex, const char *prefix) | |||
39 | format_note(NULL, sha1, ¬es, PAGE_ENCODING, 0); | 39 | format_note(NULL, sha1, ¬es, PAGE_ENCODING, 0); |
40 | 40 | ||
41 | load_ref_decorations(DECORATE_FULL_REFS); | 41 | load_ref_decorations(DECORATE_FULL_REFS); |
42 | 42 | ||
43 | cgit_print_diff_ctrls(); | ||
43 | html("<table summary='commit info' class='commit-info'>\n"); | 44 | html("<table summary='commit info' class='commit-info'>\n"); |
44 | html("<tr><th>author</th><td>"); | 45 | html("<tr><th>author</th><td>"); |
45 | html_txt(info->author); | 46 | html_txt(info->author); |
@@ -64,11 +65,6 @@ void cgit_print_commit(char *hex, const char *prefix) | |||
64 | cgit_commit_link(tmp, NULL, NULL, ctx.qry.head, tmp, prefix, 0); | 65 | cgit_commit_link(tmp, NULL, NULL, ctx.qry.head, tmp, prefix, 0); |
65 | html(" ("); | 66 | html(" ("); |
66 | cgit_patch_link("patch", NULL, NULL, NULL, tmp, prefix); | 67 | cgit_patch_link("patch", NULL, NULL, NULL, tmp, prefix); |
67 | html(") ("); | ||
68 | if ((ctx.qry.ssdiff && !ctx.cfg.ssdiff) || (!ctx.qry.ssdiff && ctx.cfg.ssdiff)) | ||
69 | cgit_commit_link("unidiff", NULL, NULL, ctx.qry.head, tmp, prefix, 1); | ||
70 | else | ||
71 | cgit_commit_link("side-by-side diff", NULL, NULL, ctx.qry.head, tmp, prefix, 1); | ||
72 | html(")</td></tr>\n"); | 68 | html(")</td></tr>\n"); |
73 | html("<tr><th>tree</th><td colspan='2' class='sha1'>"); | 69 | html("<tr><th>tree</th><td colspan='2' class='sha1'>"); |
74 | tmp = xstrdup(hex); | 70 | tmp = xstrdup(hex); |
@@ -139,7 +135,7 @@ void cgit_print_commit(char *hex, const char *prefix) | |||
139 | tmp = sha1_to_hex(commit->parents->item->object.sha1); | 135 | tmp = sha1_to_hex(commit->parents->item->object.sha1); |
140 | else | 136 | else |
141 | tmp = NULL; | 137 | tmp = NULL; |
142 | cgit_print_diff(ctx.qry.sha1, tmp, prefix); | 138 | cgit_print_diff(ctx.qry.sha1, tmp, prefix, 0); |
143 | } | 139 | } |
144 | strbuf_release(¬es); | 140 | strbuf_release(¬es); |
145 | cgit_free_commitinfo(info); | 141 | cgit_free_commitinfo(info); |
@@ -169,7 +169,7 @@ static void inspect_filepair(struct diff_filepair *pair) | |||
169 | void cgit_print_diffstat(const unsigned char *old_sha1, | 169 | void cgit_print_diffstat(const unsigned char *old_sha1, |
170 | const unsigned char *new_sha1, const char *prefix) | 170 | const unsigned char *new_sha1, const char *prefix) |
171 | { | 171 | { |
172 | int i, save_context = ctx.qry.context; | 172 | int i; |
173 | 173 | ||
174 | html("<div class='diffstat-header'>"); | 174 | html("<div class='diffstat-header'>"); |
175 | cgit_diff_link("Diffstat", NULL, NULL, ctx.qry.head, ctx.qry.sha1, | 175 | cgit_diff_link("Diffstat", NULL, NULL, ctx.qry.head, ctx.qry.sha1, |
@@ -179,19 +179,6 @@ void cgit_print_diffstat(const unsigned char *old_sha1, | |||
179 | html_txt(prefix); | 179 | html_txt(prefix); |
180 | html("')"); | 180 | html("')"); |
181 | } | 181 | } |
182 | html(" ("); | ||
183 | ctx.qry.context = (save_context > 0 ? save_context : 3) << 1; | ||
184 | cgit_self_link("more", NULL, NULL, &ctx); | ||
185 | html("/"); | ||
186 | ctx.qry.context = (save_context > 3 ? save_context : 3) >> 1; | ||
187 | cgit_self_link("less", NULL, NULL, &ctx); | ||
188 | ctx.qry.context = save_context; | ||
189 | html(" context)"); | ||
190 | html(" ("); | ||
191 | ctx.qry.ignorews = (ctx.qry.ignorews + 1) % 2; | ||
192 | cgit_self_link(ctx.qry.ignorews ? "ignore" : "show", NULL, NULL, &ctx); | ||
193 | ctx.qry.ignorews = (ctx.qry.ignorews + 1) % 2; | ||
194 | html(" whitespace changes)"); | ||
195 | html("</div>"); | 182 | html("</div>"); |
196 | html("<table summary='diffstat' class='diffstat'>"); | 183 | html("<table summary='diffstat' class='diffstat'>"); |
197 | max_changes = 0; | 184 | max_changes = 0; |
@@ -280,19 +267,6 @@ static void header(unsigned char *sha1, char *path1, int mode1, | |||
280 | html("</div>"); | 267 | html("</div>"); |
281 | } | 268 | } |
282 | 269 | ||
283 | static void print_ssdiff_link() | ||
284 | { | ||
285 | if (!strcmp(ctx.qry.page, "diff")) { | ||
286 | if (use_ssdiff) | ||
287 | cgit_diff_link("Unidiff", NULL, NULL, ctx.qry.head, | ||
288 | ctx.qry.sha1, ctx.qry.sha2, ctx.qry.path, 1); | ||
289 | else | ||
290 | cgit_diff_link("Side-by-side diff", NULL, NULL, | ||
291 | ctx.qry.head, ctx.qry.sha1, | ||
292 | ctx.qry.sha2, ctx.qry.path, 1); | ||
293 | } | ||
294 | } | ||
295 | |||
296 | static void filepair_cb(struct diff_filepair *pair) | 270 | static void filepair_cb(struct diff_filepair *pair) |
297 | { | 271 | { |
298 | unsigned long old_size = 0; | 272 | unsigned long old_size = 0; |
@@ -332,7 +306,56 @@ static void filepair_cb(struct diff_filepair *pair) | |||
332 | cgit_ssdiff_footer(); | 306 | cgit_ssdiff_footer(); |
333 | } | 307 | } |
334 | 308 | ||
335 | void cgit_print_diff(const char *new_rev, const char *old_rev, const char *prefix) | 309 | void cgit_print_diff_ctrls() |
310 | { | ||
311 | int i, curr; | ||
312 | |||
313 | html("<div class='cgit-panel'>"); | ||
314 | html("<b>diff options</b>"); | ||
315 | html("<form method='get' action='.'>"); | ||
316 | cgit_add_hidden_formfields(1, 0, ctx.qry.page); | ||
317 | html("<table>"); | ||
318 | html("<tr><td colspan='2'/></tr>"); | ||
319 | html("<tr>"); | ||
320 | html("<td class='label'>context:</td>"); | ||
321 | html("<td class='ctrl'>"); | ||
322 | html("<select name='context' onchange='this.form.submit();'>"); | ||
323 | curr = ctx.qry.context; | ||
324 | if (!curr) | ||
325 | curr = 3; | ||
326 | for (i = 1; i <= 10; i++) | ||
327 | html_intoption(i, fmt("%d", i), curr); | ||
328 | for (i = 15; i <= 40; i += 5) | ||
329 | html_intoption(i, fmt("%d", i), curr); | ||
330 | html("</select>"); | ||
331 | html("</td>"); | ||
332 | html("</tr><tr>"); | ||
333 | html("<td class='label'>space:</td>"); | ||
334 | html("<td class='ctrl'>"); | ||
335 | html("<select name='ignorews' onchange='this.form.submit();'>"); | ||
336 | html_intoption(0, "include", ctx.qry.ignorews); | ||
337 | html_intoption(1, "ignore", ctx.qry.ignorews); | ||
338 | html("</select>"); | ||
339 | html("</td>"); | ||
340 | html("</tr><tr>"); | ||
341 | html("<td class='label'>mode:</td>"); | ||
342 | html("<td class='ctrl'>"); | ||
343 | html("<select name='ss' onchange='this.form.submit();'>"); | ||
344 | curr = ctx.qry.ssdiff; | ||
345 | if (!curr && ctx.cfg.ssdiff) | ||
346 | curr = 1; | ||
347 | html_intoption(0, "unified", curr); | ||
348 | html_intoption(1, "ssdiff", curr); | ||
349 | html("</select></td></tr>"); | ||
350 | html("<tr><td/><td class='ctrl'>"); | ||
351 | html("<noscript><input type='submit' value='reload'/></noscript>"); | ||
352 | html("</td></tr></table>"); | ||
353 | html("</form>"); | ||
354 | html("</div>"); | ||
355 | } | ||
356 | |||
357 | void cgit_print_diff(const char *new_rev, const char *old_rev, | ||
358 | const char *prefix, int show_ctrls) | ||
336 | { | 359 | { |
337 | enum object_type type; | 360 | enum object_type type; |
338 | unsigned long size; | 361 | unsigned long size; |
@@ -374,7 +397,9 @@ void cgit_print_diff(const char *new_rev, const char *old_rev, const char *prefi | |||
374 | 397 | ||
375 | use_ssdiff = ctx.qry.has_ssdiff ? ctx.qry.ssdiff : ctx.cfg.ssdiff; | 398 | use_ssdiff = ctx.qry.has_ssdiff ? ctx.qry.ssdiff : ctx.cfg.ssdiff; |
376 | 399 | ||
377 | print_ssdiff_link(); | 400 | if (show_ctrls) |
401 | cgit_print_diff_ctrls(); | ||
402 | |||
378 | cgit_print_diffstat(old_rev_sha1, new_rev_sha1, prefix); | 403 | cgit_print_diffstat(old_rev_sha1, new_rev_sha1, prefix); |
379 | 404 | ||
380 | if (use_ssdiff) { | 405 | if (use_ssdiff) { |
@@ -1,11 +1,13 @@ | |||
1 | #ifndef UI_DIFF_H | 1 | #ifndef UI_DIFF_H |
2 | #define UI_DIFF_H | 2 | #define UI_DIFF_H |
3 | 3 | ||
4 | extern void cgit_print_diff_ctrls(); | ||
5 | |||
4 | extern void cgit_print_diffstat(const unsigned char *old_sha1, | 6 | extern void cgit_print_diffstat(const unsigned char *old_sha1, |
5 | const unsigned char *new_sha1); | 7 | const unsigned char *new_sha1); |
6 | 8 | ||
7 | extern void cgit_print_diff(const char *new_hex, const char *old_hex, | 9 | extern void cgit_print_diff(const char *new_hex, const char *old_hex, |
8 | const char *prefix); | 10 | const char *prefix, int show_ctrls); |
9 | 11 | ||
10 | extern struct diff_filespec *cgit_get_current_old_file(void); | 12 | extern struct diff_filespec *cgit_get_current_old_file(void); |
11 | extern struct diff_filespec *cgit_get_current_new_file(void); | 13 | extern struct diff_filespec *cgit_get_current_new_file(void); |
@@ -102,11 +102,10 @@ void print_commit(struct commit *commit, struct rev_info *revs) | |||
102 | struct strbuf graphbuf = STRBUF_INIT; | 102 | struct strbuf graphbuf = STRBUF_INIT; |
103 | struct strbuf msgbuf = STRBUF_INIT; | 103 | struct strbuf msgbuf = STRBUF_INIT; |
104 | 104 | ||
105 | if (ctx.repo->enable_log_filecount) { | 105 | if (ctx.repo->enable_log_filecount) |
106 | cols++; | ||
107 | if (ctx.repo->enable_log_linecount) | ||
106 | cols++; | 108 | cols++; |
107 | if (ctx.repo->enable_log_linecount) | ||
108 | cols++; | ||
109 | } | ||
110 | 109 | ||
111 | if (revs->graph) { | 110 | if (revs->graph) { |
112 | /* Advance graph until current commit */ | 111 | /* Advance graph until current commit */ |
@@ -173,18 +172,18 @@ void print_commit(struct commit *commit, struct rev_info *revs) | |||
173 | cgit_print_age(commit->date, TM_WEEK * 2, FMT_SHORTDATE); | 172 | cgit_print_age(commit->date, TM_WEEK * 2, FMT_SHORTDATE); |
174 | } | 173 | } |
175 | 174 | ||
176 | if (ctx.repo->enable_log_filecount) { | 175 | if (ctx.repo->enable_log_filecount || ctx.repo->enable_log_linecount) { |
177 | files = 0; | 176 | files = 0; |
178 | add_lines = 0; | 177 | add_lines = 0; |
179 | rem_lines = 0; | 178 | rem_lines = 0; |
180 | cgit_diff_commit(commit, inspect_files, ctx.qry.vpath); | 179 | cgit_diff_commit(commit, inspect_files, ctx.qry.vpath); |
181 | html("</td><td>"); | ||
182 | htmlf("%d", files); | ||
183 | if (ctx.repo->enable_log_linecount) { | ||
184 | html("</td><td>"); | ||
185 | htmlf("-%d/+%d", rem_lines, add_lines); | ||
186 | } | ||
187 | } | 180 | } |
181 | |||
182 | if (ctx.repo->enable_log_filecount) | ||
183 | htmlf("</td><td>%d", files); | ||
184 | if (ctx.repo->enable_log_linecount) | ||
185 | htmlf("</td><td>-%d/+%d", rem_lines, add_lines); | ||
186 | |||
188 | html("</td></tr>\n"); | 187 | html("</td></tr>\n"); |
189 | 188 | ||
190 | if (revs->graph || ctx.qry.showmsg) { /* Print a second table row */ | 189 | if (revs->graph || ctx.qry.showmsg) { /* Print a second table row */ |
@@ -373,10 +372,10 @@ void cgit_print_log(const char *tip, int ofs, int cnt, char *grep, char *pattern | |||
373 | if (ctx.repo->enable_log_filecount) { | 372 | if (ctx.repo->enable_log_filecount) { |
374 | html("<th class='left'>Files</th>"); | 373 | html("<th class='left'>Files</th>"); |
375 | columns++; | 374 | columns++; |
376 | if (ctx.repo->enable_log_linecount) { | 375 | } |
377 | html("<th class='left'>Lines</th>"); | 376 | if (ctx.repo->enable_log_linecount) { |
378 | columns++; | 377 | html("<th class='left'>Lines</th>"); |
379 | } | 378 | columns++; |
380 | } | 379 | } |
381 | html("</tr>\n"); | 380 | html("</tr>\n"); |
382 | 381 | ||
diff --git a/ui-repolist.c b/ui-repolist.c index 4b21e2f..a09a689 100644 --- a/ui-repolist.c +++ b/ui-repolist.c | |||
@@ -46,11 +46,20 @@ static int get_repo_modtime(const struct cgit_repo *repo, time_t *mtime) | |||
46 | } | 46 | } |
47 | 47 | ||
48 | path = fmt("%s/refs/heads/%s", repo->path, repo->defbranch); | 48 | path = fmt("%s/refs/heads/%s", repo->path, repo->defbranch); |
49 | if (stat(path, &s) == 0) | 49 | if (stat(path, &s) == 0) { |
50 | *mtime = s.st_mtime; | 50 | *mtime = s.st_mtime; |
51 | else | 51 | r->mtime = *mtime; |
52 | *mtime = 0; | 52 | return 1; |
53 | } | ||
54 | |||
55 | path = fmt("%s/%s", repo->path, "packed-refs"); | ||
56 | if (stat(path, &s) == 0) { | ||
57 | *mtime = s.st_mtime; | ||
58 | r->mtime = *mtime; | ||
59 | return 1; | ||
60 | } | ||
53 | 61 | ||
62 | *mtime = 0; | ||
54 | r->mtime = *mtime; | 63 | r->mtime = *mtime; |
55 | return (r->mtime != 0); | 64 | return (r->mtime != 0); |
56 | } | 65 | } |
@@ -386,6 +386,33 @@ void cgit_show_stats(struct cgit_context *ctx) | |||
386 | top = ctx->qry.ofs; | 386 | top = ctx->qry.ofs; |
387 | if (!top) | 387 | if (!top) |
388 | top = 10; | 388 | top = 10; |
389 | |||
390 | html("<div class='cgit-panel'>"); | ||
391 | html("<b>stat options</b>"); | ||
392 | html("<form method='get' action=''>"); | ||
393 | cgit_add_hidden_formfields(1, 0, "stats"); | ||
394 | html("<table><tr><td colspan='2'/></tr>"); | ||
395 | if (ctx->repo->max_stats > 1) { | ||
396 | html("<tr><td class='label'>Period:</td>"); | ||
397 | html("<td class='ctrl'><select name='period' onchange='this.form.submit();'>"); | ||
398 | for (i = 0; i < ctx->repo->max_stats; i++) | ||
399 | html_option(fmt("%c", periods[i].code), | ||
400 | periods[i].name, fmt("%c", period->code)); | ||
401 | html("</select></td></tr>"); | ||
402 | } | ||
403 | html("<tr><td class='label'>Authors:</td>"); | ||
404 | html("<td class='ctrl'><select name='ofs' onchange='this.form.submit();'>"); | ||
405 | html_intoption(10, "10", top); | ||
406 | html_intoption(25, "25", top); | ||
407 | html_intoption(50, "50", top); | ||
408 | html_intoption(100, "100", top); | ||
409 | html_intoption(-1, "all", top); | ||
410 | html("</select></td></tr>"); | ||
411 | html("<tr><td/><td class='ctrl'>"); | ||
412 | html("<noscript><input type='submit' value='Reload'/></noscript>"); | ||
413 | html("</td></tr></table>"); | ||
414 | html("</form>"); | ||
415 | html("</div>"); | ||
389 | htmlf("<h2>Commits per author per %s", period->name); | 416 | htmlf("<h2>Commits per author per %s", period->name); |
390 | if (ctx->qry.path) { | 417 | if (ctx->qry.path) { |
391 | html(" (path '"); | 418 | html(" (path '"); |
@@ -393,30 +420,6 @@ void cgit_show_stats(struct cgit_context *ctx) | |||
393 | html("')"); | 420 | html("')"); |
394 | } | 421 | } |
395 | html("</h2>"); | 422 | html("</h2>"); |
396 | |||
397 | html("<form method='get' action='' style='float: right; text-align: right;'>"); | ||
398 | cgit_add_hidden_formfields(1, 0, "stats"); | ||
399 | if (ctx->repo->max_stats > 1) { | ||
400 | html("Period: "); | ||
401 | html("<select name='period' onchange='this.form.submit();'>"); | ||
402 | for (i = 0; i < ctx->repo->max_stats; i++) | ||
403 | htmlf("<option value='%c'%s>%s</option>", | ||
404 | periods[i].code, | ||
405 | period == &periods[i] ? " selected" : "", | ||
406 | periods[i].name); | ||
407 | html("</select><br/><br/>"); | ||
408 | } | ||
409 | html("Authors: "); | ||
410 | html(""); | ||
411 | html("<select name='ofs' onchange='this.form.submit();'>"); | ||
412 | htmlf("<option value='10'%s>10</option>", top == 10 ? " selected" : ""); | ||
413 | htmlf("<option value='25'%s>25</option>", top == 25 ? " selected" : ""); | ||
414 | htmlf("<option value='50'%s>50</option>", top == 50 ? " selected" : ""); | ||
415 | htmlf("<option value='100'%s>100</option>", top == 100 ? " selected" : ""); | ||
416 | htmlf("<option value='-1'%s>All</option>", top == -1 ? " selected" : ""); | ||
417 | html("</select>"); | ||
418 | html("<noscript> <input type='submit' value='Reload'/></noscript>"); | ||
419 | html("</form>"); | ||
420 | print_authors(&authors, top, period); | 423 | print_authors(&authors, top, period); |
421 | } | 424 | } |
422 | 425 | ||
diff --git a/ui-summary.c b/ui-summary.c index 5be2545..227ed27 100644 --- a/ui-summary.c +++ b/ui-summary.c | |||
@@ -62,7 +62,7 @@ void cgit_print_summary() | |||
62 | NULL, NULL, 0, 0); | 62 | NULL, NULL, 0, 0); |
63 | } | 63 | } |
64 | if (ctx.repo->clone_url) | 64 | if (ctx.repo->clone_url) |
65 | print_urls(ctx.repo->clone_url, NULL); | 65 | print_urls(expand_macros(ctx.repo->clone_url), NULL); |
66 | else if (ctx.cfg.clone_prefix) | 66 | else if (ctx.cfg.clone_prefix) |
67 | print_urls(ctx.cfg.clone_prefix, ctx.repo->url); | 67 | print_urls(ctx.cfg.clone_prefix, ctx.repo->url); |
68 | html("</table>"); | 68 | html("</table>"); |