Status

Developer Services
Mercurial: hg.mozilla.org
RESOLVED FIXED
3 years ago
2 years ago

People

(Reporter: gps, Assigned: gps)

Tracking

(Blocks: 2 bugs)

Details

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

MozReview Requests

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(41 attachments, 1 obsolete attachment)

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

Description

3 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

3 years ago
Created attachment 8543203 [details]
MozReview Request: bz://1117021/gps
Attachment #8543203 - Flags: review?(bkero)
(Assignee)

Comment 2

3 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

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

Comment 3

3 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

3 years ago
Blocks: 459727
(Assignee)

Comment 4

3 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

3 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

3 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

3 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

3 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.

Comment 10

3 years ago
https://reviewboard.mozilla.org/r/1895/#review1355

lgtm, lots of replicated code. Are most of these the changes taken from upstream?

Comment 11

3 years ago
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?

Comment 12

3 years ago
https://reviewboard.mozilla.org/r/1899/#review1359

lgtm

Comment 13

3 years ago
https://reviewboard.mozilla.org/r/1901/#review1361

lgtm
(Assignee)

Comment 14

3 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

Comment 15

3 years ago
https://reviewboard.mozilla.org/r/1903/#review1365

lgtm

Comment 16

3 years ago
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.

Comment 17

3 years ago
https://reviewboard.mozilla.org/r/1909/#review1369

lgtm
(Assignee)

Comment 18

3 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

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

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

Comment 20

3 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

3 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/1961/#review1495

Ship It!
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

3 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

3 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

Comment 26

3 years ago
https://reviewboard.mozilla.org/r/1907/#review1821

lgtm

Comment 27

3 years ago
https://reviewboard.mozilla.org/r/1909/#review1823

Ship It!

Comment 28

3 years ago
https://reviewboard.mozilla.org/r/1911/#review1825

Ship It!

Comment 29

3 years ago
https://reviewboard.mozilla.org/r/1913/#review1827

Ship It!

Comment 30

3 years ago
https://reviewboard.mozilla.org/r/1915/#review1829

Ship It!

Comment 31

3 years ago
https://reviewboard.mozilla.org/r/1917/#review1831

Ship It!

Comment 32

3 years ago
https://reviewboard.mozilla.org/r/1919/#review1833

Ship It!

Comment 33

3 years ago
https://reviewboard.mozilla.org/r/1925/#review1835

Seems to work, okay.

Comment 34

3 years ago
https://reviewboard.mozilla.org/r/1927/#review1837

Ship It!

Comment 35

3 years ago
https://reviewboard.mozilla.org/r/1929/#review1839

Ship It!

Comment 36

3 years ago
https://reviewboard.mozilla.org/r/1931/#review1843

Ship It!

Comment 37

3 years ago
https://reviewboard.mozilla.org/r/1933/#review1845

Ship It!

Comment 38

3 years ago
https://reviewboard.mozilla.org/r/1935/#review1847

Ship It!

Comment 39

3 years ago
https://reviewboard.mozilla.org/r/1937/#review1849

w00t infinite scrolling

Comment 40

3 years ago
https://reviewboard.mozilla.org/r/1939/#review1851

Ship It!

Comment 41

3 years ago
https://reviewboard.mozilla.org/r/1955/#review1853

Ship It!

Comment 42

3 years ago
https://reviewboard.mozilla.org/r/1947/#review1855

Ship It!

Comment 43

3 years ago
https://reviewboard.mozilla.org/r/1945/#review1857

Ship It!

Comment 44

3 years ago
https://reviewboard.mozilla.org/r/1941/#review1859

Ship It!

Comment 45

3 years ago
https://reviewboard.mozilla.org/r/1943/#review1861

Ship It!

Comment 46

3 years ago
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?

Comment 47

3 years ago
https://reviewboard.mozilla.org/r/1951/#review1867

Ship It!

Comment 48

3 years ago
https://reviewboard.mozilla.org/r/1953/#review1875

Ship It!

Comment 49

3 years ago
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.

Comment 50

3 years ago
https://reviewboard.mozilla.org/r/1965/#review1881

Ship It!

Comment 51

3 years ago
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.

Comment 52

3 years ago
https://reviewboard.mozilla.org/r/1967/#review1885

Ship It!

Comment 53

3 years ago
https://reviewboard.mozilla.org/r/1969/#review1889

Ship It!

Comment 54

3 years ago
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.

Comment 55

3 years ago
https://reviewboard.mozilla.org/r/1971/#review1893

Ship It!

Comment 56

3 years ago
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.

Comment 57

3 years ago
https://reviewboard.mozilla.org/r/1975/#review1897

diediedie

Comment 58

3 years ago
https://reviewboard.mozilla.org/r/2099/#review1899

Ship It!

Comment 59

3 years ago
https://reviewboard.mozilla.org/r/1897/#review1913

Ship It!
(Assignee)

Comment 60

3 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.

Comment 61

3 years ago
https://reviewboard.mozilla.org/r/1949/#review1917

Ship It!
(Assignee)

Comment 62

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

Comment 63

3 years ago
This is deployed \o/
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED

Updated

3 years ago
Depends on: 1123973

Updated

3 years ago
Depends on: 1123992
(Assignee)

Updated

3 years ago
Blocks: 1124183

Updated

3 years ago
No longer blocks: 1124183

Updated

3 years ago
Depends on: 1124253
reopening until regressions resolved (current open blockers)
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 65

3 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: 3 years ago3 years ago
No longer depends on: 1123973, 1123992, 1124253
Resolution: --- → FIXED

Updated

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

Comment 66

2 years ago
Comment on attachment 8543203 [details]
MozReview Request: bz://1117021/gps
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+
(Assignee)

Comment 67

2 years ago
Created attachment 8619002 [details]
MozReview Request: hgtemplates: synchronize excanvas.js
(Assignee)

Comment 68

2 years ago
Created attachment 8619003 [details]
MozReview Request: hgtemplates: synchronize some CSS rules
(Assignee)

Comment 69

2 years ago
Created attachment 8619004 [details]
MozReview Request: hgtemplates: synchronize style.css
(Assignee)

Comment 70

2 years ago
Created attachment 8619005 [details]
MozReview Request: hgtemplates: synchronize changeset.tmpl
(Assignee)

Comment 71

2 years ago
Created attachment 8619006 [details]
MozReview Request: hgtemplates: synchronize fileannotate.tmpl
(Assignee)

Comment 72

2 years ago
Created attachment 8619007 [details]
MozReview Request: hgtemplates: synchronize filerevision.tmpl
(Assignee)

Comment 73

2 years ago
Created attachment 8619008 [details]
MozReview Request: hgtemplates: synchronize shortlog.tmpl
(Assignee)

Comment 74

2 years ago
Created attachment 8619009 [details]
MozReview Request: hgtemplates: add comparison link in file links
(Assignee)

Comment 75

2 years ago
Created attachment 8619010 [details]
MozReview Request: hgtemplates: synchronize diff and annotate line entries
(Assignee)

Comment 76

2 years ago
Created attachment 8619011 [details]
MozReview Request: hgtemplates: make template formatting consistent with upstream
(Assignee)

Comment 77

2 years ago
Created attachment 8619012 [details]
MozReview Request: hgtemplates: make filelogrename consistent with upstream
(Assignee)

Comment 78

2 years ago
Created attachment 8619013 [details]
MozReview Request: hgtemplates: render age in tagentry consistent with upstream
(Assignee)

Comment 79

2 years ago
Created attachment 8619014 [details]
MozReview Request: hgtemplates: make branchentry consistent with upstream
(Assignee)

Comment 80

2 years ago
Created attachment 8619015 [details]
MozReview Request: hgtemplates: remove filelogentry_old
(Assignee)

Comment 81

2 years ago
Created attachment 8619016 [details]
MozReview Request: hgtemplates: make navigation header consistent with upstream
(Assignee)

Comment 82

2 years ago
Created attachment 8619017 [details]
MozReview Request: hgtemplates: synchronize hg icon and logo with upstream
(Assignee)

Comment 83

2 years ago
Created attachment 8619018 [details]
MozReview Request: hgtemplates: synchronize gitweb
(Assignee)

Comment 84

2 years ago
Created attachment 8619019 [details]
MozReview Request: hgtemplates: import paper style
(Assignee)

Comment 85

2 years ago
Created attachment 8619020 [details]
MozReview Request: hgtemplates: synchronize raw style
(Assignee)

Comment 86

2 years ago
Created attachment 8619021 [details]
MozReview Request: hgtemplates: remove old .hgtags file
(Assignee)

Comment 87

2 years ago
Created attachment 8619022 [details]
MozReview Request: hgtemplates: URL escape {url} in atom style
(Assignee)

Comment 88

2 years ago
Created attachment 8619023 [details]
MozReview Request: hgtemplates: add missing atom files
(Assignee)

Comment 89

2 years ago
Created attachment 8619024 [details]
MozReview Request: hgtemplates: synchronize atom changelogentry.tmpl with upstream
(Assignee)

Comment 90

2 years ago
Created attachment 8619025 [details]
MozReview Request: hgtemplates: move "age" class in shortlogentry
(Assignee)

Comment 91

2 years ago
Created attachment 8619026 [details]
MozReview Request: hgtemplates: synchronize command line map files
(Assignee)

Comment 92

2 years ago
Created attachment 8619027 [details]
MozReview Request: hgtemplates: remove gitweb_ecma style
(Assignee)

Comment 93

2 years ago
Created attachment 8619028 [details]
MozReview Request: hgtemplates: synchronize rss style with upstream
(Assignee)

Comment 94

2 years ago
Created attachment 8619029 [details]
MozReview Request: hgtemplates: URL escape {url}; r=bkero
(Assignee)

Comment 95

2 years ago
Created attachment 8619030 [details]
MozReview Request: hgtemplates: add missing branches, bookmarks, and help sections; r=bkero
(Assignee)

Comment 96

2 years ago
Created attachment 8619031 [details]
MozReview Request: hgtemplates: use {logourl}
(Assignee)

Comment 97

2 years ago
Created attachment 8619032 [details]
MozReview Request: hgtemplates: link to latest file revision; r=bkero
(Assignee)

Comment 98

2 years ago
Created attachment 8619033 [details]
MozReview Request: hgtemplates: add file comparison; r=bkero
(Assignee)

Comment 99

2 years ago
Created attachment 8619034 [details]
MozReview Request: hgtemplates: refresh graph from upstream; r=bkero
(Assignee)

Comment 100

2 years ago
Created attachment 8619035 [details]
MozReview Request: hgtemplates: clean up navigation headers; r=bkero
(Assignee)

Comment 101

2 years ago
Created attachment 8619036 [details]
MozReview Request: hgtemplates: expand map entries into multiple lines
(Assignee)

Comment 102

2 years ago
Created attachment 8619037 [details]
MozReview Request: hgtemplates: escape some values; r=bkero
(Assignee)

Comment 103

2 years ago
Created attachment 8619038 [details]
MozReview Request: hgtemplates: move graph and pushlog
(Assignee)

Comment 104

2 years ago
Created attachment 8619039 [details]
MozReview Request: hgtemplates: escape some values
(Assignee)

Comment 105

2 years ago
Created attachment 8619040 [details]
MozReview Request: hgtemplates: add bookmarktag
(Assignee)

Comment 106

2 years ago
Created attachment 8619041 [details]
MozReview Request: hgtemplates: add bookmarks to summary / fix branches link
(Assignee)

Comment 107

2 years ago
Created attachment 8619042 [details]
MozReview Request: hgtemplates: display bookmarks in manifest; r=hwine
You need to log in before you can comment on or make changes to this bug.