-rw-r--r-- | cgit.c | 3 | ||||
-rw-r--r-- | cgit.h | 7 | ||||
-rw-r--r-- | shared.c | 2 | ||||
-rw-r--r-- | ui-commit.c | 46 | ||||
-rw-r--r-- | ui-diff.c | 64 | ||||
-rw-r--r-- | ui-shared.c | 22 |
6 files changed, 75 insertions, 69 deletions
@@ -90,34 +90,33 @@ static void cgit_print_repo_page(struct cacheitem *item) | |||
90 | 90 | ||
91 | cgit_print_pageheader(cgit_query_page, show_search); | 91 | cgit_print_pageheader(cgit_query_page, show_search); |
92 | 92 | ||
93 | switch(cgit_cmd) { | 93 | switch(cgit_cmd) { |
94 | case CMD_LOG: | 94 | case CMD_LOG: |
95 | cgit_print_log(cgit_query_sha1, cgit_query_ofs, | 95 | cgit_print_log(cgit_query_sha1, cgit_query_ofs, |
96 | cgit_max_commit_count, cgit_query_search, | 96 | cgit_max_commit_count, cgit_query_search, |
97 | cgit_query_path, 1); | 97 | cgit_query_path, 1); |
98 | break; | 98 | break; |
99 | case CMD_TREE: | 99 | case CMD_TREE: |
100 | cgit_print_tree(cgit_query_sha1, cgit_query_path); | 100 | cgit_print_tree(cgit_query_sha1, cgit_query_path); |
101 | break; | 101 | break; |
102 | case CMD_COMMIT: | 102 | case CMD_COMMIT: |
103 | cgit_print_commit(cgit_query_sha1); | 103 | cgit_print_commit(cgit_query_sha1); |
104 | break; | 104 | break; |
105 | case CMD_DIFF: | 105 | case CMD_DIFF: |
106 | cgit_print_diff(cgit_query_head, cgit_query_sha1, cgit_query_sha2, | 106 | cgit_print_diff(cgit_query_sha1, cgit_query_sha2); |
107 | cgit_query_path); | ||
108 | break; | 107 | break; |
109 | default: | 108 | default: |
110 | cgit_print_error("Invalid request"); | 109 | cgit_print_error("Invalid request"); |
111 | } | 110 | } |
112 | cgit_print_docend(); | 111 | cgit_print_docend(); |
113 | } | 112 | } |
114 | 113 | ||
115 | static void cgit_fill_cache(struct cacheitem *item, int use_cache) | 114 | static void cgit_fill_cache(struct cacheitem *item, int use_cache) |
116 | { | 115 | { |
117 | static char buf[PATH_MAX]; | 116 | static char buf[PATH_MAX]; |
118 | int stdout2; | 117 | int stdout2; |
119 | 118 | ||
120 | getcwd(buf, sizeof(buf)); | 119 | getcwd(buf, sizeof(buf)); |
121 | item->st.st_mtime = time(NULL); | 120 | item->st.st_mtime = time(NULL); |
122 | 121 | ||
123 | if (use_cache) { | 122 | if (use_cache) { |
@@ -194,40 +194,41 @@ extern char *cache_safe_filename(const char *unsafe); | |||
194 | extern int cache_lock(struct cacheitem *item); | 194 | extern int cache_lock(struct cacheitem *item); |
195 | extern int cache_unlock(struct cacheitem *item); | 195 | extern int cache_unlock(struct cacheitem *item); |
196 | extern int cache_cancel_lock(struct cacheitem *item); | 196 | extern int cache_cancel_lock(struct cacheitem *item); |
197 | extern int cache_exist(struct cacheitem *item); | 197 | extern int cache_exist(struct cacheitem *item); |
198 | extern int cache_expired(struct cacheitem *item); | 198 | extern int cache_expired(struct cacheitem *item); |
199 | 199 | ||
200 | extern char *cgit_repourl(const char *reponame); | 200 | extern char *cgit_repourl(const char *reponame); |
201 | extern char *cgit_pageurl(const char *reponame, const char *pagename, | 201 | extern char *cgit_pageurl(const char *reponame, const char *pagename, |
202 | const char *query); | 202 | const char *query); |
203 | 203 | ||
204 | extern void cgit_tree_link(char *name, char *title, char *class, char *head, | 204 | extern void cgit_tree_link(char *name, char *title, char *class, char *head, |
205 | char *rev, char *path); | 205 | char *rev, char *path); |
206 | extern void cgit_log_link(char *name, char *title, char *class, char *head, | 206 | extern void cgit_log_link(char *name, char *title, char *class, char *head, |
207 | char *rev, char *path); | 207 | char *rev, char *path); |
208 | extern void cgit_commit_link(char *name, char *title, char *class, char *head, | 208 | extern void cgit_commit_link(char *name, char *title, char *class, char *head, |
209 | char *rev); | 209 | char *rev); |
210 | extern void cgit_diff_link(char *name, char *title, char *class, char *head, | ||
211 | char *new_rev, char *old_rev, char *path); | ||
210 | 212 | ||
211 | extern void cgit_print_error(char *msg); | 213 | extern void cgit_print_error(char *msg); |
212 | extern void cgit_print_date(time_t secs, char *format); | 214 | extern void cgit_print_date(time_t secs, char *format); |
213 | extern void cgit_print_age(time_t t, time_t max_relative, char *format); | 215 | extern void cgit_print_age(time_t t, time_t max_relative, char *format); |
214 | extern void cgit_print_docstart(char *title, struct cacheitem *item); | 216 | extern void cgit_print_docstart(char *title, struct cacheitem *item); |
215 | extern void cgit_print_docend(); | 217 | extern void cgit_print_docend(); |
216 | extern void cgit_print_pageheader(char *title, int show_search); | 218 | extern void cgit_print_pageheader(char *title, int show_search); |
217 | extern void cgit_print_snapshot_start(const char *mimetype, | 219 | extern void cgit_print_snapshot_start(const char *mimetype, |
218 | const char *filename, | 220 | const char *filename, |
219 | struct cacheitem *item); | 221 | struct cacheitem *item); |
220 | 222 | ||
221 | extern void cgit_print_repolist(struct cacheitem *item); | 223 | extern void cgit_print_repolist(struct cacheitem *item); |
222 | extern void cgit_print_summary(); | 224 | extern void cgit_print_summary(); |
223 | extern void cgit_print_log(const char *tip, int ofs, int cnt, char *grep, char *path, int pager); | 225 | extern void cgit_print_log(const char *tip, int ofs, int cnt, char *grep, char *path, int pager); |
224 | extern void cgit_print_blob(struct cacheitem *item, const char *hex, char *path); | 226 | extern void cgit_print_blob(struct cacheitem *item, const char *hex, char *path); |
225 | extern void cgit_print_tree(const char *rev, char *path); | 227 | extern void cgit_print_tree(const char *rev, char *path); |
226 | extern void cgit_print_commit(const char *hex); | 228 | extern void cgit_print_commit(char *hex); |
227 | extern void cgit_print_diff(const char *head, const char *old_hex, const char *new_hex, | 229 | extern void cgit_print_diff(const char *new_hex, const char *old_hex); |
228 | char *path); | ||
229 | extern void cgit_print_snapshot(struct cacheitem *item, const char *hex, | 230 | extern void cgit_print_snapshot(struct cacheitem *item, const char *hex, |
230 | const char *format, const char *prefix, | 231 | const char *format, const char *prefix, |
231 | const char *filename); | 232 | const char *filename); |
232 | 233 | ||
233 | #endif /* CGIT_H */ | 234 | #endif /* CGIT_H */ |
@@ -347,32 +347,32 @@ int cgit_diff_files(const unsigned char *old_sha1, | |||
347 | 347 | ||
348 | void cgit_diff_tree(const unsigned char *old_sha1, | 348 | void cgit_diff_tree(const unsigned char *old_sha1, |
349 | const unsigned char *new_sha1, | 349 | const unsigned char *new_sha1, |
350 | filepair_fn fn) | 350 | filepair_fn fn) |
351 | { | 351 | { |
352 | struct diff_options opt; | 352 | struct diff_options opt; |
353 | int ret; | 353 | int ret; |
354 | 354 | ||
355 | diff_setup(&opt); | 355 | diff_setup(&opt); |
356 | opt.output_format = DIFF_FORMAT_CALLBACK; | 356 | opt.output_format = DIFF_FORMAT_CALLBACK; |
357 | opt.detect_rename = 1; | 357 | opt.detect_rename = 1; |
358 | opt.recursive = 1; | 358 | opt.recursive = 1; |
359 | opt.format_callback = cgit_diff_tree_cb; | 359 | opt.format_callback = cgit_diff_tree_cb; |
360 | opt.format_callback_data = fn; | 360 | opt.format_callback_data = fn; |
361 | diff_setup_done(&opt); | 361 | diff_setup_done(&opt); |
362 | 362 | ||
363 | if (old_sha1) | 363 | if (old_sha1 && !is_null_sha1(old_sha1)) |
364 | ret = diff_tree_sha1(old_sha1, new_sha1, "", &opt); | 364 | ret = diff_tree_sha1(old_sha1, new_sha1, "", &opt); |
365 | else | 365 | else |
366 | ret = diff_root_tree_sha1(new_sha1, "", &opt); | 366 | ret = diff_root_tree_sha1(new_sha1, "", &opt); |
367 | diffcore_std(&opt); | 367 | diffcore_std(&opt); |
368 | diff_flush(&opt); | 368 | diff_flush(&opt); |
369 | } | 369 | } |
370 | 370 | ||
371 | void cgit_diff_commit(struct commit *commit, filepair_fn fn) | 371 | void cgit_diff_commit(struct commit *commit, filepair_fn fn) |
372 | { | 372 | { |
373 | unsigned char *old_sha1 = NULL; | 373 | unsigned char *old_sha1 = NULL; |
374 | 374 | ||
375 | if (commit->parents) | 375 | if (commit->parents) |
376 | old_sha1 = commit->parents->item->object.sha1; | 376 | old_sha1 = commit->parents->item->object.sha1; |
377 | cgit_diff_tree(old_sha1, commit->object.sha1, fn); | 377 | cgit_diff_tree(old_sha1, commit->object.sha1, fn); |
378 | } | 378 | } |
diff --git a/ui-commit.c b/ui-commit.c index d489d7c..2679b59 100644 --- a/ui-commit.c +++ b/ui-commit.c | |||
@@ -1,46 +1,46 @@ | |||
1 | /* ui-commit.c: generate commit view | 1 | /* ui-commit.c: generate commit view |
2 | * | 2 | * |
3 | * Copyright (C) 2006 Lars Hjemli | 3 | * Copyright (C) 2006 Lars Hjemli |
4 | * | 4 | * |
5 | * Licensed under GNU General Public License v2 | 5 | * Licensed under GNU General Public License v2 |
6 | * (see COPYING for full license text) | 6 | * (see COPYING for full license text) |
7 | */ | 7 | */ |
8 | 8 | ||
9 | #include "cgit.h" | 9 | #include "cgit.h" |
10 | 10 | ||
11 | static int files, slots; | 11 | static int files, slots; |
12 | static int total_adds, total_rems, max_changes; | 12 | static int total_adds, total_rems, max_changes; |
13 | static int lines_added, lines_removed; | 13 | static int lines_added, lines_removed; |
14 | static char *curr_rev; | ||
14 | 15 | ||
15 | static struct fileinfo { | 16 | static struct fileinfo { |
16 | char status; | 17 | char status; |
17 | unsigned char old_sha1[20]; | 18 | unsigned char old_sha1[20]; |
18 | unsigned char new_sha1[20]; | 19 | unsigned char new_sha1[20]; |
19 | unsigned short old_mode; | 20 | unsigned short old_mode; |
20 | unsigned short new_mode; | 21 | unsigned short new_mode; |
21 | char *old_path; | 22 | char *old_path; |
22 | char *new_path; | 23 | char *new_path; |
23 | unsigned int added; | 24 | unsigned int added; |
24 | unsigned int removed; | 25 | unsigned int removed; |
25 | } *items; | 26 | } *items; |
26 | 27 | ||
27 | 28 | ||
28 | void print_fileinfo(struct fileinfo *info) | 29 | void print_fileinfo(struct fileinfo *info) |
29 | { | 30 | { |
30 | char *query, *query2; | ||
31 | char *class; | 31 | char *class; |
32 | 32 | ||
33 | switch (info->status) { | 33 | switch (info->status) { |
34 | case DIFF_STATUS_ADDED: | 34 | case DIFF_STATUS_ADDED: |
35 | class = "add"; | 35 | class = "add"; |
36 | break; | 36 | break; |
37 | case DIFF_STATUS_COPIED: | 37 | case DIFF_STATUS_COPIED: |
38 | class = "cpy"; | 38 | class = "cpy"; |
39 | break; | 39 | break; |
40 | case DIFF_STATUS_DELETED: | 40 | case DIFF_STATUS_DELETED: |
41 | class = "del"; | 41 | class = "del"; |
42 | break; | 42 | break; |
43 | case DIFF_STATUS_MODIFIED: | 43 | case DIFF_STATUS_MODIFIED: |
44 | class = "upd"; | 44 | class = "upd"; |
45 | break; | 45 | break; |
46 | case DIFF_STATUS_RENAMED: | 46 | case DIFF_STATUS_RENAMED: |
@@ -62,50 +62,38 @@ void print_fileinfo(struct fileinfo *info) | |||
62 | html("<tr>"); | 62 | html("<tr>"); |
63 | htmlf("<td class='mode'>"); | 63 | htmlf("<td class='mode'>"); |
64 | if (is_null_sha1(info->new_sha1)) { | 64 | if (is_null_sha1(info->new_sha1)) { |
65 | html_filemode(info->old_mode); | 65 | html_filemode(info->old_mode); |
66 | } else { | 66 | } else { |
67 | html_filemode(info->new_mode); | 67 | html_filemode(info->new_mode); |
68 | } | 68 | } |
69 | 69 | ||
70 | if (info->old_mode != info->new_mode && | 70 | if (info->old_mode != info->new_mode && |
71 | !is_null_sha1(info->old_sha1) && | 71 | !is_null_sha1(info->old_sha1) && |
72 | !is_null_sha1(info->new_sha1)) { | 72 | !is_null_sha1(info->new_sha1)) { |
73 | html("<span class='modechange'>["); | 73 | html("<span class='modechange'>["); |
74 | html_filemode(info->old_mode); | 74 | html_filemode(info->old_mode); |
75 | html("]</span>"); | 75 | html("]</span>"); |
76 | } | 76 | } |
77 | htmlf("</td><td class='%s'>", class); | 77 | htmlf("</td><td class='%s'>", class); |
78 | query = fmt("id=%s&id2=%s&path=%s", sha1_to_hex(info->old_sha1), | 78 | cgit_tree_link(info->new_path, NULL, NULL, cgit_query_head, curr_rev, |
79 | sha1_to_hex(info->new_sha1), info->new_path); | 79 | info->new_path); |
80 | html_link_open(cgit_pageurl(cgit_query_repo, "diff", query), | 80 | if (info->status == DIFF_STATUS_COPIED || info->status == DIFF_STATUS_RENAMED) |
81 | NULL, NULL); | 81 | htmlf(" (%s from %s)", |
82 | if (info->status == DIFF_STATUS_COPIED || | 82 | info->status == DIFF_STATUS_COPIED ? "copied" : "renamed", |
83 | info->status == DIFF_STATUS_RENAMED) { | 83 | info->old_path); |
84 | html_txt(info->new_path); | ||
85 | htmlf("</a> (%s from ", info->status == DIFF_STATUS_COPIED ? | ||
86 | "copied" : "renamed"); | ||
87 | query2 = fmt("id=%s", sha1_to_hex(info->old_sha1)); | ||
88 | html_link_open(cgit_pageurl(cgit_query_repo, "view", query2), | ||
89 | NULL, NULL); | ||
90 | html_txt(info->old_path); | ||
91 | html("</a>)"); | ||
92 | } else { | ||
93 | html_txt(info->new_path); | ||
94 | html("</a>"); | ||
95 | } | ||
96 | html("</td><td class='right'>"); | 84 | html("</td><td class='right'>"); |
97 | htmlf("%d", info->added + info->removed); | 85 | htmlf("%d", info->added + info->removed); |
98 | html("</td><td class='graph'>"); | 86 | html("</td><td class='graph'>"); |
99 | htmlf("<table width='%d%%'><tr>", (max_changes > 100 ? 100 : max_changes)); | 87 | htmlf("<table width='%d%%'><tr>", (max_changes > 100 ? 100 : max_changes)); |
100 | htmlf("<td class='add' style='width: %.1f%%;'/>", | 88 | htmlf("<td class='add' style='width: %.1f%%;'/>", |
101 | info->added * 100.0 / max_changes); | 89 | info->added * 100.0 / max_changes); |
102 | htmlf("<td class='rem' style='width: %.1f%%;'/>", | 90 | htmlf("<td class='rem' style='width: %.1f%%;'/>", |
103 | info->removed * 100.0 / max_changes); | 91 | info->removed * 100.0 / max_changes); |
104 | htmlf("<td class='none' style='width: %.1f%%;'/>", | 92 | htmlf("<td class='none' style='width: %.1f%%;'/>", |
105 | (max_changes - info->removed - info->added) * 100.0 / max_changes); | 93 | (max_changes - info->removed - info->added) * 100.0 / max_changes); |
106 | html("</tr></table></td></tr>\n"); | 94 | html("</tr></table></td></tr>\n"); |
107 | } | 95 | } |
108 | 96 | ||
109 | void cgit_count_diff_lines(char *line, int len) | 97 | void cgit_count_diff_lines(char *line, int len) |
110 | { | 98 | { |
111 | if (line && (len > 0)) { | 99 | if (line && (len > 0)) { |
@@ -132,45 +120,45 @@ void inspect_filepair(struct diff_filepair *pair) | |||
132 | items[files-1].status = pair->status; | 120 | items[files-1].status = pair->status; |
133 | hashcpy(items[files-1].old_sha1, pair->one->sha1); | 121 | hashcpy(items[files-1].old_sha1, pair->one->sha1); |
134 | hashcpy(items[files-1].new_sha1, pair->two->sha1); | 122 | hashcpy(items[files-1].new_sha1, pair->two->sha1); |
135 | items[files-1].old_mode = pair->one->mode; | 123 | items[files-1].old_mode = pair->one->mode; |
136 | items[files-1].new_mode = pair->two->mode; | 124 | items[files-1].new_mode = pair->two->mode; |
137 | items[files-1].old_path = xstrdup(pair->one->path); | 125 | items[files-1].old_path = xstrdup(pair->one->path); |
138 | items[files-1].new_path = xstrdup(pair->two->path); | 126 | items[files-1].new_path = xstrdup(pair->two->path); |
139 | items[files-1].added = lines_added; | 127 | items[files-1].added = lines_added; |
140 | items[files-1].removed = lines_removed; | 128 | items[files-1].removed = lines_removed; |
141 | if (lines_added + lines_removed > max_changes) | 129 | if (lines_added + lines_removed > max_changes) |
142 | max_changes = lines_added + lines_removed; | 130 | max_changes = lines_added + lines_removed; |
143 | total_adds += lines_added; | 131 | total_adds += lines_added; |
144 | total_rems += lines_removed; | 132 | total_rems += lines_removed; |
145 | } | 133 | } |
146 | 134 | ||
147 | 135 | ||
148 | void cgit_print_commit(const char *hex) | 136 | void cgit_print_commit(char *hex) |
149 | { | 137 | { |
150 | struct commit *commit, *parent; | 138 | struct commit *commit, *parent; |
151 | struct commitinfo *info; | 139 | struct commitinfo *info; |
152 | struct commit_list *p; | 140 | struct commit_list *p; |
153 | unsigned char sha1[20]; | 141 | unsigned char sha1[20]; |
154 | char *query; | ||
155 | char *filename; | 142 | char *filename; |
156 | char *tmp; | 143 | char *tmp; |
157 | int i; | 144 | int i; |
158 | 145 | ||
159 | if (!hex) | 146 | if (!hex) |
160 | hex = cgit_query_head; | 147 | hex = cgit_query_head; |
148 | curr_rev = hex; | ||
161 | 149 | ||
162 | if (get_sha1(hex, sha1)) { | 150 | if (get_sha1(hex, sha1)) { |
163 | cgit_print_error(fmt("Bad object id: %s", hex)); | 151 | cgit_print_error(fmt("Bad object id: %s", hex)); |
164 | return; | 152 | return; |
165 | } | 153 | } |
166 | commit = lookup_commit_reference(sha1); | 154 | commit = lookup_commit_reference(sha1); |
167 | if (!commit) { | 155 | if (!commit) { |
168 | cgit_print_error(fmt("Bad commit reference: %s", hex)); | 156 | cgit_print_error(fmt("Bad commit reference: %s", hex)); |
169 | return; | 157 | return; |
170 | } | 158 | } |
171 | info = cgit_parse_commit(commit); | 159 | info = cgit_parse_commit(commit); |
172 | 160 | ||
173 | html("<table class='commit-info'>\n"); | 161 | html("<table class='commit-info'>\n"); |
174 | html("<tr><th>author</th><td>"); | 162 | html("<tr><th>author</th><td>"); |
175 | html_txt(info->author); | 163 | html_txt(info->author); |
176 | html(" "); | 164 | html(" "); |
@@ -189,54 +177,52 @@ void cgit_print_commit(const char *hex) | |||
189 | tmp = xstrdup(hex); | 177 | tmp = xstrdup(hex); |
190 | cgit_tree_link(sha1_to_hex(commit->tree->object.sha1), NULL, NULL, | 178 | cgit_tree_link(sha1_to_hex(commit->tree->object.sha1), NULL, NULL, |
191 | cgit_query_head, tmp, NULL); | 179 | cgit_query_head, tmp, NULL); |
192 | html("</td></tr>\n"); | 180 | html("</td></tr>\n"); |
193 | for (p = commit->parents; p ; p = p->next) { | 181 | for (p = commit->parents; p ; p = p->next) { |
194 | parent = lookup_commit_reference(p->item->object.sha1); | 182 | parent = lookup_commit_reference(p->item->object.sha1); |
195 | if (!parent) { | 183 | if (!parent) { |
196 | html("<tr><td colspan='3'>"); | 184 | html("<tr><td colspan='3'>"); |
197 | cgit_print_error("Error reading parent commit"); | 185 | cgit_print_error("Error reading parent commit"); |
198 | html("</td></tr>"); | 186 | html("</td></tr>"); |
199 | continue; | 187 | continue; |
200 | } | 188 | } |
201 | html("<tr><th>parent</th>" | 189 | html("<tr><th>parent</th>" |
202 | "<td colspan='2' class='sha1'>"); | 190 | "<td colspan='2' class='sha1'>"); |
203 | cgit_commit_link(sha1_to_hex(p->item->object.sha1), NULL, NULL, | 191 | cgit_commit_link(sha1_to_hex(p->item->object.sha1), NULL, NULL, |
204 | cgit_query_head, sha1_to_hex(p->item->object.sha1)); | 192 | cgit_query_head, sha1_to_hex(p->item->object.sha1)); |
205 | html(" (<a href='"); | 193 | html(" ("); |
206 | query = fmt("id=%s&id2=%s", sha1_to_hex(parent->tree->object.sha1), | 194 | cgit_diff_link("diff", NULL, NULL, cgit_query_head, hex, |
207 | sha1_to_hex(commit->tree->object.sha1)); | 195 | sha1_to_hex(p->item->object.sha1), NULL); |
208 | html_attr(cgit_pageurl(cgit_query_repo, "diff", query)); | 196 | html(")</td></tr>"); |
209 | html("'>diff</a>)</td></tr>"); | ||
210 | } | 197 | } |
211 | if (cgit_repo->snapshots) { | 198 | if (cgit_repo->snapshots) { |
212 | htmlf("<tr><th>download</th><td colspan='2' class='sha1'><a href='"); | 199 | htmlf("<tr><th>download</th><td colspan='2' class='sha1'><a href='"); |
213 | filename = fmt("%s-%s.zip", cgit_query_repo, hex); | 200 | filename = fmt("%s-%s.zip", cgit_query_repo, hex); |
214 | html_attr(cgit_pageurl(cgit_query_repo, "snapshot", | 201 | html_attr(cgit_pageurl(cgit_query_repo, "snapshot", |
215 | fmt("id=%s&name=%s", hex, filename))); | 202 | fmt("id=%s&name=%s", hex, filename))); |
216 | htmlf("'>%s</a></td></tr>", filename); | 203 | htmlf("'>%s</a></td></tr>", filename); |
217 | } | 204 | } |
218 | html("</table>\n"); | 205 | html("</table>\n"); |
219 | html("<div class='commit-subject'>"); | 206 | html("<div class='commit-subject'>"); |
220 | html_txt(info->subject); | 207 | html_txt(info->subject); |
221 | html("</div>"); | 208 | html("</div>"); |
222 | html("<div class='commit-msg'>"); | 209 | html("<div class='commit-msg'>"); |
223 | html_txt(info->msg); | 210 | html_txt(info->msg); |
224 | html("</div>"); | 211 | html("</div>"); |
225 | if (!(commit->parents && commit->parents->next && commit->parents->next->next)) { | 212 | if (!(commit->parents && commit->parents->next && commit->parents->next->next)) { |
226 | html("<div class='diffstat-header'>Diffstat</div>"); | 213 | html("<div class='diffstat-header'>Diffstat</div>"); |
227 | html("<table class='diffstat'>"); | 214 | html("<table class='diffstat'>"); |
228 | max_changes = 0; | 215 | max_changes = 0; |
229 | cgit_diff_commit(commit, inspect_filepair); | 216 | cgit_diff_commit(commit, inspect_filepair); |
230 | for(i = 0; i<files; i++) | 217 | for(i = 0; i<files; i++) |
231 | print_fileinfo(&items[i]); | 218 | print_fileinfo(&items[i]); |
232 | html("</table>"); | 219 | html("</table>"); |
233 | html("<div class='diffstat-summary'>"); | 220 | html("<div class='diffstat-summary'>"); |
234 | htmlf("%d files changed, %d insertions, %d deletions (", | 221 | htmlf("%d files changed, %d insertions, %d deletions (", |
235 | files, total_adds, total_rems); | 222 | files, total_adds, total_rems); |
236 | query = fmt("h=%s", hex); | 223 | cgit_diff_link("show diff", NULL, NULL, cgit_query_head, hex, |
237 | html_link_open(cgit_pageurl(cgit_query_repo, "diff", query), NULL, NULL); | 224 | NULL, NULL); |
238 | html("show diff</a>)"); | 225 | html(")</div>"); |
239 | html("</div>"); | ||
240 | } | 226 | } |
241 | cgit_free_commitinfo(info); | 227 | cgit_free_commitinfo(info); |
242 | } | 228 | } |
@@ -76,67 +76,65 @@ static void header(unsigned char *sha1, char *path1, int mode1, | |||
76 | 76 | ||
77 | static void filepair_cb(struct diff_filepair *pair) | 77 | static void filepair_cb(struct diff_filepair *pair) |
78 | { | 78 | { |
79 | header(pair->one->sha1, pair->one->path, pair->one->mode, | 79 | header(pair->one->sha1, pair->one->path, pair->one->mode, |
80 | pair->two->sha1, pair->two->path, pair->two->mode); | 80 | pair->two->sha1, pair->two->path, pair->two->mode); |
81 | if (S_ISDIRLNK(pair->one->mode) || S_ISDIRLNK(pair->two->mode)) { | 81 | if (S_ISDIRLNK(pair->one->mode) || S_ISDIRLNK(pair->two->mode)) { |
82 | if (S_ISDIRLNK(pair->one->mode)) | 82 | if (S_ISDIRLNK(pair->one->mode)) |
83 | print_line(fmt("-Subproject %s", sha1_to_hex(pair->one->sha1)), 52); | 83 | print_line(fmt("-Subproject %s", sha1_to_hex(pair->one->sha1)), 52); |
84 | if (S_ISDIRLNK(pair->two->mode)) | 84 | if (S_ISDIRLNK(pair->two->mode)) |
85 | print_line(fmt("+Subproject %s", sha1_to_hex(pair->two->sha1)), 52); | 85 | print_line(fmt("+Subproject %s", sha1_to_hex(pair->two->sha1)), 52); |
86 | return; | 86 | return; |
87 | } | 87 | } |
88 | if (cgit_diff_files(pair->one->sha1, pair->two->sha1, print_line)) | 88 | if (cgit_diff_files(pair->one->sha1, pair->two->sha1, print_line)) |
89 | cgit_print_error("Error running diff"); | 89 | cgit_print_error("Error running diff"); |
90 | } | 90 | } |
91 | 91 | ||
92 | void cgit_print_diff(const char *head, const char *old_hex, const char *new_hex, char *path) | 92 | void cgit_print_diff(const char *new_rev, const char *old_rev) |
93 | { | 93 | { |
94 | unsigned char sha1[20], sha2[20]; | 94 | unsigned char sha1[20], sha2[20]; |
95 | enum object_type type; | 95 | enum object_type type; |
96 | unsigned long size; | 96 | unsigned long size; |
97 | struct commit *commit; | 97 | struct commit *commit, *commit2; |
98 | 98 | ||
99 | html("<table class='diff'>"); | 99 | if (!new_rev) |
100 | html("<tr><td>"); | 100 | new_rev = cgit_query_head; |
101 | 101 | get_sha1(new_rev, sha1); | |
102 | if (head && !old_hex && !new_hex) { | 102 | type = sha1_object_info(sha1, &size); |
103 | get_sha1(head, sha1); | 103 | if (type == OBJ_BAD) { |
104 | commit = lookup_commit_reference(sha1); | 104 | cgit_print_error(fmt("Bad object name: %s", new_rev)); |
105 | if (commit && !parse_commit(commit)) | 105 | return; |
106 | cgit_diff_commit(commit, filepair_cb); | 106 | } |
107 | else | 107 | if (type != OBJ_COMMIT) { |
108 | cgit_print_error(fmt("Bad commit: %s", head)); | 108 | cgit_print_error(fmt("Unhandled object type: %s", |
109 | html("</td></tr>"); | 109 | typename(type))); |
110 | html("</table>"); | ||
111 | return; | 110 | return; |
112 | } | 111 | } |
113 | 112 | ||
114 | get_sha1(old_hex, sha1); | 113 | commit = lookup_commit_reference(sha1); |
115 | get_sha1(new_hex, sha2); | 114 | if (!commit || parse_commit(commit)) |
115 | cgit_print_error(fmt("Bad commit: %s", sha1_to_hex(sha1))); | ||
116 | 116 | ||
117 | type = sha1_object_info(sha1, &size); | 117 | if (old_rev) |
118 | if (type == OBJ_BAD) { | 118 | get_sha1(old_rev, sha2); |
119 | else if (commit->parents && commit->parents->item) | ||
120 | hashcpy(sha2, commit->parents->item->object.sha1); | ||
121 | else | ||
122 | hashclr(sha2); | ||
123 | |||
124 | if (!is_null_sha1(sha2)) { | ||
119 | type = sha1_object_info(sha2, &size); | 125 | type = sha1_object_info(sha2, &size); |
120 | if (type == OBJ_BAD) { | 126 | if (type == OBJ_BAD) { |
121 | cgit_print_error(fmt("Bad object names: %s, %s", old_hex, new_hex)); | 127 | cgit_print_error(fmt("Bad object name: %s", sha1_to_hex(sha2))); |
122 | return; | 128 | return; |
123 | } | 129 | } |
130 | commit2 = lookup_commit_reference(sha2); | ||
131 | if (!commit2 || parse_commit(commit2)) | ||
132 | cgit_print_error(fmt("Bad commit: %s", sha1_to_hex(sha2))); | ||
124 | } | 133 | } |
125 | 134 | ||
126 | switch(type) { | 135 | html("<table class='diff'>"); |
127 | case OBJ_BLOB: | 136 | html("<tr><td>"); |
128 | header(sha1, path, 0644, sha2, path, 0644); | 137 | cgit_diff_tree(sha2, sha1, filepair_cb); |
129 | if (cgit_diff_files(sha1, sha2, print_line)) | ||
130 | cgit_print_error("Error running diff"); | ||
131 | break; | ||
132 | case OBJ_TREE: | ||
133 | cgit_diff_tree(sha1, sha2, filepair_cb); | ||
134 | break; | ||
135 | default: | ||
136 | cgit_print_error(fmt("Unhandled object type: %s", | ||
137 | typename(type))); | ||
138 | break; | ||
139 | } | ||
140 | html("</td></tr>"); | 138 | html("</td></tr>"); |
141 | html("</table>"); | 139 | html("</table>"); |
142 | } | 140 | } |
diff --git a/ui-shared.c b/ui-shared.c index 71c899a..15d8254 100644 --- a/ui-shared.c +++ b/ui-shared.c | |||
@@ -163,32 +163,54 @@ void cgit_log_link(char *name, char *title, char *class, char *head, | |||
163 | { | 163 | { |
164 | reporevlink("log", name, title, class, head, rev, path); | 164 | reporevlink("log", name, title, class, head, rev, path); |
165 | } | 165 | } |
166 | 166 | ||
167 | void cgit_commit_link(char *name, char *title, char *class, char *head, | 167 | void cgit_commit_link(char *name, char *title, char *class, char *head, |
168 | char *rev) | 168 | char *rev) |
169 | { | 169 | { |
170 | if (strlen(name) > cgit_max_msg_len && cgit_max_msg_len >= 15) { | 170 | if (strlen(name) > cgit_max_msg_len && cgit_max_msg_len >= 15) { |
171 | name[cgit_max_msg_len] = '\0'; | 171 | name[cgit_max_msg_len] = '\0'; |
172 | name[cgit_max_msg_len - 1] = '.'; | 172 | name[cgit_max_msg_len - 1] = '.'; |
173 | name[cgit_max_msg_len - 2] = '.'; | 173 | name[cgit_max_msg_len - 2] = '.'; |
174 | name[cgit_max_msg_len - 3] = '.'; | 174 | name[cgit_max_msg_len - 3] = '.'; |
175 | } | 175 | } |
176 | reporevlink("commit", name, title, class, head, rev, NULL); | 176 | reporevlink("commit", name, title, class, head, rev, NULL); |
177 | } | 177 | } |
178 | 178 | ||
179 | void cgit_diff_link(char *name, char *title, char *class, char *head, | ||
180 | char *new_rev, char *old_rev, char *path) | ||
181 | { | ||
182 | char *delim; | ||
183 | |||
184 | delim = repolink(title, class, "diff", head, path); | ||
185 | if (new_rev && strcmp(new_rev, cgit_query_head)) { | ||
186 | html(delim); | ||
187 | html("id="); | ||
188 | html_attr(new_rev); | ||
189 | delim = "&"; | ||
190 | } | ||
191 | if (old_rev) { | ||
192 | html(delim); | ||
193 | html("id2="); | ||
194 | html_attr(old_rev); | ||
195 | } | ||
196 | html("'>"); | ||
197 | html_txt(name); | ||
198 | html("</a>"); | ||
199 | } | ||
200 | |||
179 | void cgit_print_date(time_t secs, char *format) | 201 | void cgit_print_date(time_t secs, char *format) |
180 | { | 202 | { |
181 | char buf[64]; | 203 | char buf[64]; |
182 | struct tm *time; | 204 | struct tm *time; |
183 | 205 | ||
184 | time = gmtime(&secs); | 206 | time = gmtime(&secs); |
185 | strftime(buf, sizeof(buf)-1, format, time); | 207 | strftime(buf, sizeof(buf)-1, format, time); |
186 | html_txt(buf); | 208 | html_txt(buf); |
187 | } | 209 | } |
188 | 210 | ||
189 | void cgit_print_age(time_t t, time_t max_relative, char *format) | 211 | void cgit_print_age(time_t t, time_t max_relative, char *format) |
190 | { | 212 | { |
191 | time_t now, secs; | 213 | time_t now, secs; |
192 | 214 | ||
193 | time(&now); | 215 | time(&now); |
194 | secs = now - t; | 216 | secs = now - t; |