Status

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: gps, Assigned: gps)

Tracking

(Blocks 2 bugs)

Details

(Whiteboard: [kanban:engops:https://mozilla.kanbanize.com/ctrl_board/6/4262] )

Attachments

(41 attachments, 1 obsolete attachment)

39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
39 bytes, text/x-review-board-request
bkero
: review+
Details
(Assignee)

Description

4 years ago
We're currently running a fork of the "gitweb" theme on hg.mozilla.org. The theme code appears to be based on some ancient version of Mercurial - likely before the 2.x series.

As part of migrating hg.mozilla.org to paper, let's update the gitweb theme to modern times. This make the transition easier, as it will make a diff of vanilla to Mozilla easier and this in turn will make porting to paper easier.
(Assignee)

Comment 1

4 years ago
Posted file MozReview Request: bz://1117021/gps (obsolete) —
Attachment #8543203 - Flags: review?(bkero)
(Assignee)

Comment 2

4 years ago
/r/1893 - hgtemplates: URL escape {url}
/r/1895 - hgtemplates: add missing branches, bookmarks, and help sections
/r/1897 - hgtemplates: use {logourl}
/r/1899 - hgtemplates: link to latest file revision
/r/1901 - hgtemplates: add file comparison
/r/1903 - hgtemplates: refresh graph from upstream
/r/1905 - hgtemplates: clean up navigation headers
/r/1907 - hgtemplates: expand map entries into multiple lines
/r/1909 - hgtemplates: escape some values
/r/1911 - hgtemplates: move graph and pushlog
/r/1913 - hgtemplates: escape some values
/r/1915 - hgtemplates: add bookmarktag
/r/1917 - hgtemplates: add bookmarks to summary / fix branches link
/r/1919 - hgtemplates: display bookmarks in manifest

Pull down these commits:

hg pull review -r eadbbfa23cde151828b0c81efc451b7cd07d15c7

Updated

4 years ago
Whiteboard: [kanban:engops:https://mozilla.kanbanize.com/ctrl_board/6/4262]
(Assignee)

Comment 3

4 years ago
/r/1893 - hgtemplates: URL escape {url}
/r/1895 - hgtemplates: add missing branches, bookmarks, and help sections
/r/1897 - hgtemplates: use {logourl}
/r/1899 - hgtemplates: link to latest file revision
/r/1901 - hgtemplates: add file comparison
/r/1903 - hgtemplates: refresh graph from upstream
/r/1905 - hgtemplates: clean up navigation headers
/r/1907 - hgtemplates: expand map entries into multiple lines
/r/1909 - hgtemplates: escape some values
/r/1911 - hgtemplates: move graph and pushlog
/r/1913 - hgtemplates: escape some values
/r/1915 - hgtemplates: add bookmarktag
/r/1917 - hgtemplates: add bookmarks to summary / fix branches link
/r/1919 - hgtemplates: display bookmarks in manifest
/r/1925 - hgtemplates: synchronize excanvas.js
/r/1927 - hgtemplates: synchronize some CSS rules
/r/1929 - hgtemplates: synchronize style.css
/r/1931 - hgtemplates: synchronize changeset.tmpl
/r/1933 - hgtemplates: synchronize fileannotate.tmpl
/r/1935 - hgtemplates: synchronize filerevision.tmpl
/r/1937 - hgtemplates: synchronize shortlog.tmpl
/r/1939 - hgtemplates: add comparison link in file links
/r/1941 - hgtemplates: synchronize diff and annotate line entries
/r/1943 - hgtemplates: make template formatting consistent with upstream
/r/1945 - hgtemplates: make filelogrename consistent with upstream
/r/1947 - hgtemplates: render age in tagentry consistent with upstream
/r/1949 - hgtemplates: make branchentry consistent with upstream
/r/1951 - hgtemplates: remove filelogentry_old
/r/1953 - hgtemplates: make navigation header consistent with upstream
/r/1955 - hgtemplates: synchronize hg icon and logo with upstream
/r/1957 - hgtemplates: synchronize gitweb
/r/1959 - hgtemplates: import paper style
/r/1961 - hgtemplates: synchronize raw style
/r/1963 - hgtemplates: remove old .hgtags file
/r/1965 - hgtemplates: URL escape {url} in atom style
/r/1967 - hgtemplates: add missing atom files
/r/1969 - hgtemplates: synchronize atom changelogentry.tmpl with upstream
/r/1971 - hgtemplates: synchronize rss style with upstream
/r/1973 - hgtemplates: synchronize command line map files
/r/1975 - hgtemplates: remove gitweb_ecma style

Pull down these commits:

hg pull review -r 777b8d7078c5adeb92d8c7d71b7219583afcb8b0
(Assignee)

Updated

4 years ago
Blocks: 459727
(Assignee)

Comment 4

4 years ago
Ted: I figure you may want to peruse the changes. I'm confident I accidentally regressed something we purposefully added many years ago. Maybe you can spot it before it rolls out to production :)
https://reviewboard.mozilla.org/r/1887/#review1295

::: hgtemplates/gitweb_mozilla/changeset.tmpl
(Diff revision 2)
> -{desc|strip|escape|firstline|buglink} <span class="logtags">{inbranch%inbranchtag}{branches%branchtag}{tags%tagtag}</span>
> +<a class="title" href="{url|urlescape}raw-rev/{node|short}">{desc|strip|escape|firstline|buglink|nonempty} <span class="logtags">{inbranch%inbranchtag}{branches%branchtag}{tags%tagtag}{bookmarks%bookmarktag}</span></a>

As someone pointed out in bug 1088315 this will break because the buglink filter outputs <a> elements, and nested <a>s aren't going to work. It's probably okay to just remove buglink here since this is just the first line and the whole commit message is below.

::: hgtemplates/gitweb_mozilla/filerevision.tmpl
(Diff revision 2)
> -{desc|strip|escape|addbreaks|buglink}
> +{desc|strip|escape|websub|addbreaks|nonempty}

You dropped buglink here, was that intentional?

::: hgtemplates/static/style-gitweb.css
(Diff revision 2)
> -span.difflineat { color:#990099; }

It looks like these were an intentional change:
http://hg.mozilla.org/hgcustom/version-control-tools/rev/2c355a580af6

We may need to upstream that.
(Assignee)

Comment 6

4 years ago
https://reviewboard.mozilla.org/r/1887/#review1319

> It looks like these were an intentional change:
> http://hg.mozilla.org/hgcustom/version-control-tools/rev/2c355a580af6
> 
> We may need to upstream that.

Nice catch! I submitted the patch to upstream: http://www.selenic.com/pipermail/mercurial-devel/2015-January/065241.html

> You dropped buglink here, was that intentional?

Nope. Will fix.
(Assignee)

Comment 7

4 years ago
/r/1893 - hgtemplates: URL escape {url}
/r/1895 - hgtemplates: add missing branches, bookmarks, and help sections
/r/1897 - hgtemplates: use {logourl}
/r/1899 - hgtemplates: link to latest file revision
/r/1901 - hgtemplates: add file comparison
/r/1903 - hgtemplates: refresh graph from upstream
/r/1905 - hgtemplates: clean up navigation headers
/r/1907 - hgtemplates: expand map entries into multiple lines
/r/1909 - hgtemplates: escape some values
/r/1911 - hgtemplates: move graph and pushlog
/r/1913 - hgtemplates: escape some values
/r/1915 - hgtemplates: add bookmarktag
/r/1917 - hgtemplates: add bookmarks to summary / fix branches link
/r/1919 - hgtemplates: display bookmarks in manifest
/r/1925 - hgtemplates: synchronize excanvas.js
/r/1927 - hgtemplates: synchronize some CSS rules
/r/1929 - hgtemplates: synchronize style.css
/r/1931 - hgtemplates: synchronize changeset.tmpl
/r/1933 - hgtemplates: synchronize fileannotate.tmpl
/r/1935 - hgtemplates: synchronize filerevision.tmpl
/r/1937 - hgtemplates: synchronize shortlog.tmpl
/r/1939 - hgtemplates: add comparison link in file links
/r/1941 - hgtemplates: synchronize diff and annotate line entries
/r/1943 - hgtemplates: make template formatting consistent with upstream
/r/1945 - hgtemplates: make filelogrename consistent with upstream
/r/1947 - hgtemplates: render age in tagentry consistent with upstream
/r/1949 - hgtemplates: make branchentry consistent with upstream
/r/1951 - hgtemplates: remove filelogentry_old
/r/1953 - hgtemplates: make navigation header consistent with upstream
/r/1955 - hgtemplates: synchronize hg icon and logo with upstream
/r/1957 - hgtemplates: synchronize gitweb
/r/1959 - hgtemplates: import paper style
/r/1961 - hgtemplates: synchronize raw style
/r/1963 - hgtemplates: remove old .hgtags file
/r/1965 - hgtemplates: URL escape {url} in atom style
/r/1967 - hgtemplates: add missing atom files
/r/1969 - hgtemplates: synchronize atom changelogentry.tmpl with upstream
/r/1971 - hgtemplates: synchronize rss style with upstream
/r/1973 - hgtemplates: synchronize command line map files
/r/1975 - hgtemplates: remove gitweb_ecma style

Pull down these commits:

hg pull review -r c05d09165bbc9cdb56e2fa07a288e8529ef9b213
(Assignee)

Comment 8

4 years ago
/r/1893 - hgtemplates: URL escape {url}
/r/1895 - hgtemplates: add missing branches, bookmarks, and help sections
/r/1897 - hgtemplates: use {logourl}
/r/1899 - hgtemplates: link to latest file revision
/r/1901 - hgtemplates: add file comparison
/r/1903 - hgtemplates: refresh graph from upstream
/r/1905 - hgtemplates: clean up navigation headers
/r/1907 - hgtemplates: expand map entries into multiple lines
/r/1909 - hgtemplates: escape some values
/r/1911 - hgtemplates: move graph and pushlog
/r/1913 - hgtemplates: escape some values
/r/1915 - hgtemplates: add bookmarktag
/r/1917 - hgtemplates: add bookmarks to summary / fix branches link
/r/1919 - hgtemplates: display bookmarks in manifest
/r/1925 - hgtemplates: synchronize excanvas.js
/r/1927 - hgtemplates: synchronize some CSS rules
/r/1929 - hgtemplates: synchronize style.css
/r/1931 - hgtemplates: synchronize changeset.tmpl
/r/1933 - hgtemplates: synchronize fileannotate.tmpl
/r/1935 - hgtemplates: synchronize filerevision.tmpl
/r/1937 - hgtemplates: synchronize shortlog.tmpl
/r/1939 - hgtemplates: add comparison link in file links
/r/1941 - hgtemplates: synchronize diff and annotate line entries
/r/1943 - hgtemplates: make template formatting consistent with upstream
/r/1945 - hgtemplates: make filelogrename consistent with upstream
/r/1947 - hgtemplates: render age in tagentry consistent with upstream
/r/1949 - hgtemplates: make branchentry consistent with upstream
/r/1951 - hgtemplates: remove filelogentry_old
/r/1953 - hgtemplates: make navigation header consistent with upstream
/r/1955 - hgtemplates: synchronize hg icon and logo with upstream
/r/1957 - hgtemplates: synchronize gitweb
/r/1959 - hgtemplates: import paper style
/r/1961 - hgtemplates: synchronize raw style
/r/1963 - hgtemplates: remove old .hgtags file
/r/1965 - hgtemplates: URL escape {url} in atom style
/r/1967 - hgtemplates: add missing atom files
/r/1969 - hgtemplates: synchronize atom changelogentry.tmpl with upstream
/r/1971 - hgtemplates: synchronize rss style with upstream
/r/1973 - hgtemplates: synchronize command line map files
/r/1975 - hgtemplates: remove gitweb_ecma style
/r/2099 - hgtemplates: move "age" class in shortlogentry

Pull down these commits:

hg pull review -r 3f3d19d5095d31a2c467d008a2e6e9733ab8c3a5

Comment 9

4 years ago
https://reviewboard.mozilla.org/r/1893/#review1353

::: hgtemplates/gitweb_mozilla/summary.tmpl
(Diff revision 2)
> -<form action="{url}log">
> +<form action="{url|urlescape}log">

Does this look right to you? It's not a standard <a> tag, it's the action name of a form.

Other than the question earlier, this looks very cut and dry. Ship it.
https://reviewboard.mozilla.org/r/1895/#review1355

lgtm, lots of replicated code. Are most of these the changes taken from upstream?
https://reviewboard.mozilla.org/r/1897/#review1357

This seems to be reverting the logo link of some pages from MDN back to http://mercurial.selenic.com/ . Is this intentional? Perhaps with this patch we should be adding config.web.logourl = MDN?
(Assignee)

Comment 14

4 years ago
https://reviewboard.mozilla.org/r/1897/#review1363

Yes, we'll want to add the following to the hgrc on these machines:

   [web]
   logourl = https://developer.mozilla.org/en/docs/Mercurial
https://reviewboard.mozilla.org/r/1905/#review1367

::: hgtemplates/gitweb_mozilla/changelog.tmpl
(Diff revision 2)
> -<a href="{url|urlescape}graph/{rev}{sessionvars%urlparameter}">graph</a> |
> +<a href="{url|urlescape}graph{sessionvars%urlparameter}">graph</a> |

Do we care about changing the behavior of these graph/ links? Or will the rev be pased in the sessionvars?

Other than the behavior question earlier, lgtm.
(Assignee)

Comment 18

4 years ago
https://reviewboard.mozilla.org/r/1893/#review1483

> Does this look right to you? It's not a standard <a> tag, it's the action name of a form.

This is how it is done upstream, for better or worse. I'm just going to take it.
(Assignee)

Comment 19

4 years ago
https://reviewboard.mozilla.org/r/1895/#review1485

Yes, this is pretty much all from upstream as-is.
(Assignee)

Comment 20

4 years ago
https://reviewboard.mozilla.org/r/1905/#review1487

> Do we care about changing the behavior of these graph/ links? Or will the rev be pased in the sessionvars?

Doh. Looks like this is a Mozilla-ism. I'll keep it in.
(Assignee)

Comment 21

4 years ago
/r/1893 - hgtemplates: URL escape {url}; r=bkero
/r/1895 - hgtemplates: add missing branches, bookmarks, and help sections; r=bkero
/r/1897 - hgtemplates: use {logourl}
/r/1899 - hgtemplates: link to latest file revision; r=bkero
/r/1901 - hgtemplates: add file comparison; r=bkero
/r/1903 - hgtemplates: refresh graph from upstream; r=bkero
/r/1905 - hgtemplates: clean up navigation headers; r=bkero
/r/1907 - hgtemplates: expand map entries into multiple lines
/r/1909 - hgtemplates: escape some values; r=bkero
/r/1911 - hgtemplates: move graph and pushlog
/r/1913 - hgtemplates: escape some values
/r/1915 - hgtemplates: add bookmarktag
/r/1917 - hgtemplates: add bookmarks to summary / fix branches link
/r/1919 - hgtemplates: display bookmarks in manifest
/r/1925 - hgtemplates: synchronize excanvas.js
/r/1927 - hgtemplates: synchronize some CSS rules
/r/1929 - hgtemplates: synchronize style.css
/r/1931 - hgtemplates: synchronize changeset.tmpl
/r/1933 - hgtemplates: synchronize fileannotate.tmpl
/r/1935 - hgtemplates: synchronize filerevision.tmpl
/r/1937 - hgtemplates: synchronize shortlog.tmpl
/r/1939 - hgtemplates: add comparison link in file links
/r/1941 - hgtemplates: synchronize diff and annotate line entries
/r/1943 - hgtemplates: make template formatting consistent with upstream
/r/1945 - hgtemplates: make filelogrename consistent with upstream
/r/1947 - hgtemplates: render age in tagentry consistent with upstream
/r/1949 - hgtemplates: make branchentry consistent with upstream
/r/1951 - hgtemplates: remove filelogentry_old
/r/1953 - hgtemplates: make navigation header consistent with upstream
/r/1955 - hgtemplates: synchronize hg icon and logo with upstream
/r/1957 - hgtemplates: synchronize gitweb
/r/1959 - hgtemplates: import paper style
/r/1961 - hgtemplates: synchronize raw style
/r/1963 - hgtemplates: remove old .hgtags file
/r/1965 - hgtemplates: URL escape {url} in atom style
/r/1967 - hgtemplates: add missing atom files
/r/1969 - hgtemplates: synchronize atom changelogentry.tmpl with upstream
/r/1971 - hgtemplates: synchronize rss style with upstream
/r/1973 - hgtemplates: synchronize command line map files
/r/1975 - hgtemplates: remove gitweb_ecma style
/r/2099 - hgtemplates: move "age" class in shortlogentry

Pull down these commits:

hg pull review -r 052a508bd787d61b0e504e5c2bbbbdcccb819962
https://reviewboard.mozilla.org/r/1919/#review1499

lgtm - the deployment is global I assume, but most repos won't have bookmarks so should be a non-issue at bottom of current list.
(Assignee)

Comment 24

4 years ago
/r/1893 - hgtemplates: URL escape {url}; r=bkero
/r/1895 - hgtemplates: add missing branches, bookmarks, and help sections; r=bkero
/r/1897 - hgtemplates: use {logourl}
/r/1899 - hgtemplates: link to latest file revision; r=bkero
/r/1901 - hgtemplates: add file comparison; r=bkero
/r/1903 - hgtemplates: refresh graph from upstream; r=bkero
/r/1905 - hgtemplates: clean up navigation headers; r=bkero
/r/1907 - hgtemplates: expand map entries into multiple lines
/r/1909 - hgtemplates: escape some values; r=bkero
/r/1911 - hgtemplates: move graph and pushlog
/r/1913 - hgtemplates: escape some values
/r/1915 - hgtemplates: add bookmarktag
/r/1917 - hgtemplates: add bookmarks to summary / fix branches link
/r/1919 - hgtemplates: display bookmarks in manifest
/r/1925 - hgtemplates: synchronize excanvas.js
/r/1927 - hgtemplates: synchronize some CSS rules
/r/1929 - hgtemplates: synchronize style.css
/r/1931 - hgtemplates: synchronize changeset.tmpl
/r/1933 - hgtemplates: synchronize fileannotate.tmpl
/r/1935 - hgtemplates: synchronize filerevision.tmpl
/r/1937 - hgtemplates: synchronize shortlog.tmpl
/r/1939 - hgtemplates: add comparison link in file links
/r/1941 - hgtemplates: synchronize diff and annotate line entries
/r/1943 - hgtemplates: make template formatting consistent with upstream
/r/1945 - hgtemplates: make filelogrename consistent with upstream
/r/1947 - hgtemplates: render age in tagentry consistent with upstream
/r/1949 - hgtemplates: make branchentry consistent with upstream
/r/1951 - hgtemplates: remove filelogentry_old
/r/1953 - hgtemplates: make navigation header consistent with upstream
/r/1955 - hgtemplates: synchronize hg icon and logo with upstream
/r/1957 - hgtemplates: synchronize gitweb
/r/1965 - hgtemplates: URL escape {url} in atom style
/r/1967 - hgtemplates: add missing atom files
/r/1969 - hgtemplates: synchronize atom changelogentry.tmpl with upstream
/r/1971 - hgtemplates: synchronize rss style with upstream
/r/1973 - hgtemplates: synchronize command line map files
/r/2099 - hgtemplates: move "age" class in shortlogentry
/r/1975 - hgtemplates: remove gitweb_ecma style

Pull down these commits:

hg pull review -r f58c4e02fd69ec4907535e29e91122a9287f1c53
(Assignee)

Comment 25

4 years ago
/r/1893 - hgtemplates: URL escape {url}; r=bkero
/r/1895 - hgtemplates: add missing branches, bookmarks, and help sections; r=bkero
/r/1897 - hgtemplates: use {logourl}
/r/1899 - hgtemplates: link to latest file revision; r=bkero
/r/1901 - hgtemplates: add file comparison; r=bkero
/r/1903 - hgtemplates: refresh graph from upstream; r=bkero
/r/1905 - hgtemplates: clean up navigation headers; r=bkero
/r/1907 - hgtemplates: expand map entries into multiple lines
/r/1909 - hgtemplates: escape some values; r=bkero
/r/1911 - hgtemplates: move graph and pushlog
/r/1913 - hgtemplates: escape some values
/r/1915 - hgtemplates: add bookmarktag
/r/1917 - hgtemplates: add bookmarks to summary / fix branches link
/r/1919 - hgtemplates: display bookmarks in manifest; r=hwine
/r/1925 - hgtemplates: synchronize excanvas.js
/r/1927 - hgtemplates: synchronize some CSS rules
/r/1929 - hgtemplates: synchronize style.css
/r/1931 - hgtemplates: synchronize changeset.tmpl
/r/1933 - hgtemplates: synchronize fileannotate.tmpl
/r/1935 - hgtemplates: synchronize filerevision.tmpl
/r/1937 - hgtemplates: synchronize shortlog.tmpl
/r/1939 - hgtemplates: add comparison link in file links
/r/1941 - hgtemplates: synchronize diff and annotate line entries
/r/1943 - hgtemplates: make template formatting consistent with upstream
/r/1945 - hgtemplates: make filelogrename consistent with upstream
/r/1947 - hgtemplates: render age in tagentry consistent with upstream
/r/1949 - hgtemplates: make branchentry consistent with upstream
/r/1951 - hgtemplates: remove filelogentry_old
/r/1953 - hgtemplates: make navigation header consistent with upstream
/r/1955 - hgtemplates: synchronize hg icon and logo with upstream
/r/1957 - hgtemplates: synchronize gitweb
/r/1965 - hgtemplates: URL escape {url} in atom style
/r/1967 - hgtemplates: add missing atom files
/r/1969 - hgtemplates: synchronize atom changelogentry.tmpl with upstream
/r/1971 - hgtemplates: synchronize rss style with upstream
/r/1973 - hgtemplates: synchronize command line map files
/r/2099 - hgtemplates: move "age" class in shortlogentry
/r/1975 - hgtemplates: remove gitweb_ecma style

Pull down these commits:

hg pull review -r 021a3036a31275512fe8ee659e1f8d8ccadb3d69
https://reviewboard.mozilla.org/r/1949/#review1863

::: hgtemplates/gitweb_mozilla/map
(Diff revision 6)
> -    <td><a class="list" href="{url|urlescape}rev/{node|short}{sessionvars%urlparameter}"><b>{node|short}</b></a></td>
> +    <td><a class="list" href="{url|urlescape}shortlog/{node|short}{sessionvars%urlparameter}"><b>{node|short}</b></a></td>

This is s/rev/shortlog/. Is that expected? Aren't those two pages different?
https://reviewboard.mozilla.org/r/1957/#review1877

Didn't look too closely at this since this code isn't used in production (unless a user requests a specific style). As far as I'm concerned we can just nuke the dir and check out a fresh gitweb/ copy from upstream.
https://reviewboard.mozilla.org/r/1967/#review1883

::: hgtemplates/atom/map
(Diff revision 6)
> +filedifflink = '{file|escape}<br />'

This and the following 3 lines should probably be their own commit, but shouldn't really matter.
https://reviewboard.mozilla.org/r/1967/#review1891

> This and the following 3 lines should probably be their own commit, but shouldn't really matter.

Misinterpreted the issue, the map is in /atom/. Disregard.
https://reviewboard.mozilla.org/r/1973/#review1895

This looks fine to me, but I'm not sure if any of our automated testing is going to use these, then choke on the different output. I suppose those clients will have their own templates, so these should never be used.
(Assignee)

Comment 60

4 years ago
https://reviewboard.mozilla.org/r/1949/#review1915

> This is s/rev/shortlog/. Is that expected? Aren't those two pages different?

AFAICT, this URL has been present since gitweb_mozilla was checked in and wasn't changed by Mozilla. It must have changed upstream.
(Assignee)

Comment 62

4 years ago
I pushed all these changesets. Will deploy shortly.
(Assignee)

Comment 63

4 years ago
This is deployed \o/
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Depends on: 1123973
Depends on: 1123992
(Assignee)

Updated

4 years ago
Blocks: 1124183
No longer blocks: 1124183
reopening until regressions resolved (current open blockers)
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 65

4 years ago
No need to reopen. Other bugs are on file. They are regressions. They are related but don't need to keep this bug open while they are outstanding.
Status: REOPENED → RESOLVED
Last Resolved: 4 years ago4 years ago
No longer depends on: 1123973, 1123992, 1124253
Resolution: --- → FIXED

Updated

4 years ago
Attachment #8543203 - Flags: review?(bkero) → review+
(Assignee)

Comment 66

4 years ago
Attachment #8543203 - Attachment is obsolete: true
Attachment #8619002 - Flags: review+
Attachment #8619003 - Flags: review+
Attachment #8619004 - Flags: review+
Attachment #8619005 - Flags: review+
Attachment #8619006 - Flags: review+
Attachment #8619007 - Flags: review+
Attachment #8619008 - Flags: review+
Attachment #8619009 - Flags: review+
Attachment #8619010 - Flags: review+
Attachment #8619011 - Flags: review+
Attachment #8619012 - Flags: review+
Attachment #8619013 - Flags: review+
Attachment #8619014 - Flags: review+
Attachment #8619015 - Flags: review+
Attachment #8619016 - Flags: review+
Attachment #8619017 - Flags: review+
Attachment #8619018 - Flags: review+
Attachment #8619019 - Flags: review+
Attachment #8619020 - Flags: review+
Attachment #8619021 - Flags: review+
Attachment #8619022 - Flags: review+
Attachment #8619023 - Flags: review+
Attachment #8619024 - Flags: review+
Attachment #8619025 - Flags: review+
Attachment #8619026 - Flags: review+
Attachment #8619027 - Flags: review+
Attachment #8619028 - Flags: review+
Attachment #8619029 - Flags: review+
Attachment #8619030 - Flags: review+
Attachment #8619031 - Flags: review+
Attachment #8619032 - Flags: review+
Attachment #8619033 - Flags: review+
Attachment #8619034 - Flags: review+
Attachment #8619035 - Flags: review+
Attachment #8619036 - Flags: review+
Attachment #8619037 - Flags: review+
Attachment #8619038 - Flags: review+
Attachment #8619039 - Flags: review+
Attachment #8619040 - Flags: review+
Attachment #8619041 - Flags: review+
Attachment #8619042 - Flags: review+
You need to log in before you can comment on or make changes to this bug.