https://www.redmine.org/https://www.redmine.org/favicon.ico?16793021292010-08-12T05:30:40ZRedmineRedmine - Feature #6092: Truncate Git revision labels in Activity page/feed and allow configurable lengthhttps://www.redmine.org/issues/6092?journal_id=192732010-08-12T05:30:40ZAdam Soltys
<ul><li><strong>File</strong> <a href="/attachments/4294">patch.diff</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/4294/patch.diff">patch.diff</a> added</li><li><strong>Status</strong> changed from <i>New</i> to <i>Resolved</i></li><li><strong>% Done</strong> changed from <i>0</i> to <i>100</i></li></ul><p>Attached patch applies the fix. Also available at <a class="external" href="http://github.com/asoltys/redmine">http://github.com/asoltys/redmine</a></p> Redmine - Feature #6092: Truncate Git revision labels in Activity page/feed and allow configurable lengthhttps://www.redmine.org/issues/6092?journal_id=192742010-08-12T05:32:53ZAdam Soltys
<ul></ul><p>Oh, I should mention I didn't implement this as a configurable setting as requested. Maybe later...</p> Redmine - Feature #6092: Truncate Git revision labels in Activity page/feed and allow configurable lengthhttps://www.redmine.org/issues/6092?journal_id=192782010-08-12T07:35:57ZFelix Schäfer
<ul><li><strong>Category</strong> set to <i>UI</i></li><li><strong>Status</strong> changed from <i>Resolved</i> to <i>New</i></li></ul><p>Please leave the status changes to the devs, thank you. (resolved is for patches in trunk but not in stable yet).</p>
<p>The patch looks good to me, I'm not sure we'd need it configurable either. If we take the git etiquette of 72 chars per line, add the "Revision " and ": " around the revision hash (11 chars total, might be somewhat different in other locales) and 8 chars for the revision hash, we end up with 91 chars, which leaves some space for the other locales to have longer strings.</p>
<p><code>JB: I like the idea as it is, but I'd rather make a @short_revision</code> attribute each adapter can override at will rather than a hard default like that. I especially think people using SCMs with sequential numbering would be quite surprised to see commit <code>10000000</code> followed by commit <code>10000000</code> :-)</p> Redmine - Feature #6092: Truncate Git revision labels in Activity page/feed and allow configurable lengthhttps://www.redmine.org/issues/6092?journal_id=192872010-08-12T15:02:36ZAdam Soltys
<ul></ul><p>Agreed Felix. I'll work on moving it at least into a variable if not into a setting. Also, sorry about changing the ticket status, I'll know now not to do that anymore.</p> Redmine - Feature #6092: Truncate Git revision labels in Activity page/feed and allow configurable lengthhttps://www.redmine.org/issues/6092?journal_id=193732010-08-14T22:06:01ZAdam Soltys
<ul><li><strong>File</strong> <a href="/attachments/4309">patch.diff</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/4309/patch.diff">patch.diff</a> added</li></ul><p>Ok, I've implemented this as a setting now. I figure this way it's not necessary to have separate values in each adapter. I've defaulted the value to 8. If someone with an svn repository with over 10 million revisions needs to increase this, they can!</p> Redmine - Feature #6092: Truncate Git revision labels in Activity page/feed and allow configurable lengthhttps://www.redmine.org/issues/6092?journal_id=193742010-08-15T01:22:21ZToshi MARUYAMA
<ul></ul><p>Adam Soltys wrote:</p>
<blockquote>
<p>Ok, I've implemented this as a setting now. I figure this way it's not necessary to have separate values in each adapter. I've defaulted the value to 8. If someone with an svn repository with over 10 million revisions needs to increase this, they can!</p>
</blockquote>
<p>Your patch is great!<br />Can you implement setting of "7.days" at <a class="source" href="https://www.redmine.org/projects/redmine/repository/svn/entry/tags/1.0.0/app/models/repository/git.rb#L50">source:tags/1.0.0/app/models/repository/git.rb#L50</a> and <a class="external" href="http://www.redmine.org/attachments/3272/git-fast-browse.patch">http://www.redmine.org/attachments/3272/git-fast-browse.patch</a> in <a class="issue tracker-1 status-5 priority-4 priority-default closed" title="Defect: Redmine+Git+PostgresSQL 8.4 fails with linux kernel tree (encoding) (Closed)" href="https://www.redmine.org/issues/4773">#4773</a> ?</p> Redmine - Feature #6092: Truncate Git revision labels in Activity page/feed and allow configurable lengthhttps://www.redmine.org/issues/6092?journal_id=193752010-08-15T04:10:31ZAdam Soltys
<ul><li><strong>File</strong> <a href="/attachments/4310">patch.diff</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/4310/patch.diff">patch.diff</a> added</li></ul><p>Toshi, ok, I did these two things as settings for you, but I don't actually think they should be included in trunk and presented to the average user. They're specific to git, for one thing, and they deal with some pretty obscure internal performance mechanisms.</p>
<p>A better solution in my opinion would be to revisit the git adapter and see if we can get some more performance out of it by default. If github can do it, so can we!</p> Redmine - Feature #6092: Truncate Git revision labels in Activity page/feed and allow configurable lengthhttps://www.redmine.org/issues/6092?journal_id=193762010-08-15T04:32:44ZAdam Soltys
<ul></ul><p>I just realized I was incorrectly using the log_display_limit setting instead of the truncation setting in a couple places of code. Fixed that. Here's the patch again with just the truncation setting addressing the original issue, this time done properly.</p>
<p>I've also placed this change in a "for-eric" branch on my github fork and sent him a pull request. <a class="external" href="http://github.com/asoltys/redmine/tree/for-eric">http://github.com/asoltys/redmine/tree/for-eric</a></p> Redmine - Feature #6092: Truncate Git revision labels in Activity page/feed and allow configurable lengthhttps://www.redmine.org/issues/6092?journal_id=193772010-08-15T04:34:03ZAdam Soltys
<ul><li><strong>File</strong> <a href="/attachments/4311">revision_truncation_length_as_setting.diff</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/4311/revision_truncation_length_as_setting.diff">revision_truncation_length_as_setting.diff</a> added</li></ul> Redmine - Feature #6092: Truncate Git revision labels in Activity page/feed and allow configurable lengthhttps://www.redmine.org/issues/6092?journal_id=193802010-08-16T00:04:45ZToshi MARUYAMA
<ul></ul><p>Adam Soltys wrote:</p>
<blockquote>
<p>Toshi, ok, I did these two things as settings for you, but I don't actually think they should be included in trunk and presented to the average user. They're specific to git, for one thing, and they deal with some pretty obscure internal performance mechanisms.</p>
</blockquote>
<p>7.days problem is reported at <a class="issue tracker-1 status-5 priority-3 priority-lowest closed" title="Defect: git tab,browsing, very slow -- even after first time (Closed)" href="https://www.redmine.org/issues/6013">#6013</a> and "git log -n 1 FILE_or_DIR" problem is reported at <a class="issue tracker-1 status-5 priority-4 priority-default closed" title="Defect: Redmine hangs up while browsing Git repository (Closed)" href="https://www.redmine.org/issues/5096">#5096</a>.</p> Redmine - Feature #6092: Truncate Git revision labels in Activity page/feed and allow configurable lengthhttps://www.redmine.org/issues/6092?journal_id=194392010-08-17T03:14:54ZAdam Soltys
<ul></ul><p>Thanks for pointing to those issues Toshi. I hadn't been following redmine development for the last year or so but I'm glad to see that other people have taken up the charge on these matters. I was exposed first-hand to some of the general performance problems that redmine has with distributed scm's back when I worked on <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Feature: Browse through Git branches (Closed)" href="https://www.redmine.org/issues/1406">#1406</a>, so I've gained an appreciation and desire to fix these things once and for all.</p>
<p>I still don't agree with exposing these two settings at the moment because I still see them as temporary hacks to solve performance problems that I think we could do better on. In particular, I have some ideas about possibly pre-fetching and caching some repository history or implementing some more sophisticated on-demand lookups and partial history scans that might give us adequate performance without requiring users to sacrifice the "last revision" field or manually specify how far back to look for dirty changesets.</p>
<p>I need to do a bit more thinking and design-work before I tackle a full-on reimplementation of the git and mercurial adapters though. So in the meantime I'm just trying to get my feet wet again with contributing to redmine. I do have plans to become a regular contributor through the coming fall and winter though and repository management and performance will be a central area of interest for me.</p> Redmine - Feature #6092: Truncate Git revision labels in Activity page/feed and allow configurable lengthhttps://www.redmine.org/issues/6092?journal_id=197292010-08-25T20:40:56ZJean-Baptiste Barth
<ul></ul><p>Didn't have the time to read everything but thank you guys for your work on these topics.</p>
About the original issue, I agree we should truncate git's sha1 revision names but I have 2 worries about it :
<ul>
<li>I don't like the idea of introducing a Setting for that. There are already bunches of useless settings in redmine, and consulting administration panel always give me the feeling that we have too many things here which leads to confusion and bloat software ; I'd rather turn it into a constant in the class in a first approach. I think nobody cares Github truncates revision names to 7 or 8 chars because nobody reads these IDs, they're just here to point to the commit. We should have the same approach, decide a number (6? 7? 8? 10?) and begin with this.</li>
<li>after some thoughts about it, I think this modification must be applied at adapter level, only to SCM with non sequential revision numbers.</li>
</ul>
<p>Adam: can you confirm the last patch is the only one to keep ? We can delete other patches so that it's clearer which one to work on.</p> Redmine - Feature #6092: Truncate Git revision labels in Activity page/feed and allow configurable lengthhttps://www.redmine.org/issues/6092?journal_id=197622010-08-26T00:04:46ZToshi MARUYAMA
<ul></ul><p>Yuya's implementation of Mercurial overhaul (<a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Feature: Mercurial overhaul (Closed)" href="https://www.redmine.org/issues/4455">#4455</a>) shows activity with mercurial style such as <a href="http://dev.openttdcoop.org/projects/ttrs/repository/revisions/ff70262d476d" class="external">12:ff70262d476d</a> .<br />You can see at <a class="external" href="http://dev.openttdcoop.org/projects/ttrs/activity">http://dev.openttdcoop.org/projects/ttrs/activity</a> .</p>
<p>I pushed yuya's <a href="http://bitbucket.org/yuja/redmine-mq-issue4455" class="external">mq</a> to my github repository <a href="http://github.com/marutosi/redmine/tree/issue-4455-yuya-mq-20100825" class="external">branch</a> .</p> Redmine - Feature #6092: Truncate Git revision labels in Activity page/feed and allow configurable lengthhttps://www.redmine.org/issues/6092?journal_id=197692010-08-26T07:42:50ZFelix Schäfer
<ul></ul><p>Jean-Baptiste Barth wrote:</p>
<blockquote>
About the original issue, I agree we should truncate git's sha1 revision names but I have 2 worries about it :
<ul>
<li>I don't like the idea of introducing a Setting for that. There are already bunches of useless settings in redmine, and consulting administration panel always give me the feeling that we have too many things here which leads to confusion and bloat software ; I'd rather turn it into a constant in the class in a first approach. I think nobody cares Github truncates revision names to 7 or 8 chars because nobody reads these IDs, they're just here to point to the commit. We should have the same approach, decide a number (6? 7? 8? 10?) and begin with this.</li>
<li>after some thoughts about it, I think this modification must be applied at adapter level, only to SCM with non sequential revision numbers.</li>
</ul>
</blockquote>
<p>Agreed, I'd go with 8 chars and an extra <code>short_revision</code> virtual attribute in the adapter.</p> Redmine - Feature #6092: Truncate Git revision labels in Activity page/feed and allow configurable lengthhttps://www.redmine.org/issues/6092?journal_id=197912010-08-27T01:09:44ZToshi MARUYAMA
<ul></ul><p>Adam Soltys wrote:</p>
<blockquote>
<p>In particular, I have some ideas about possibly pre-fetching and caching some repository history or implementing some more sophisticated on-demand lookups and partial history scans that might give us adequate performance without requiring users to sacrifice the "last revision" field or manually specify how far back to look for dirty changesets.</p>
</blockquote>
<p>7.days is only git adapter problem.<br />There are some ideas of git at after <a href="http://www.redmine.org/issues/4773#note-13" class="external">note-13</a> of <a class="issue tracker-1 status-5 priority-4 priority-default closed" title="Defect: Redmine+Git+PostgresSQL 8.4 fails with linux kernel tree (encoding) (Closed)" href="https://www.redmine.org/issues/4773">#4773</a> .</p>
<p>Because mercurial has revision number, Redmine can know mercurial "last committed revision"(tip) easily.<br />But mercurial tip <strong>is not latest revision</strong> (see <a href="http://www.redmine.org/issues/4455#note-23" class="external">note-23</a> of <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Feature: Mercurial overhaul (Closed)" href="https://www.redmine.org/issues/4455">#4455</a> ).<br />This is the reason of Redmine 1.0.x mercurial adapter fetching performance problem.<br />Yuya fixed this problem at <a href="http://www.redmine.org/issues/4455#note-144" class="external">note-144</a> of <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Feature: Mercurial overhaul (Closed)" href="https://www.redmine.org/issues/4455">#4455</a> and I pushed to my github <a href="http://github.com/marutosi/redmine/tree/hg-patches-svntrunk" class="external">hg-patches-svntrunk branch</a> .</p> Redmine - Feature #6092: Truncate Git revision labels in Activity page/feed and allow configurable lengthhttps://www.redmine.org/issues/6092?journal_id=197932010-08-27T01:27:27ZAdam Soltys
<ul></ul><p>Thanks for looking at this Jean-Baptiste. I agree with you about the importance of avoiding software bloat and I don't like exposing obscure technical settings to the users either.</p>
<p>I'll have to look at the code though to see if it's possible to move this setting into the adapters. I think in some places where the revisions are being shortened we don't have access to the adapter objects, which is why I went with a global setting, but maybe I can figure something out. So yes, the latest patch supersedes the other ones, but hold off on looking at it because I'll probably be coming up with another one soon in light of this latest discussion.</p>
<p>Toshi, thanks again for the info. I looked at Yuya's branch quickly and it seems like he's done a lot of good work that I need to take a closer look at.</p> Redmine - Feature #6092: Truncate Git revision labels in Activity page/feed and allow configurable lengthhttps://www.redmine.org/issues/6092?journal_id=198302010-08-29T17:00:17ZYuya Nishihara
<ul></ul><p>Adam Soltys wrote:</p>
<blockquote>
<p>Toshi, thanks again for the info. I looked at Yuya's branch quickly and it seems like he's done a lot of good work that I need to take a closer look at.</p>
</blockquote>
Hi, in short, my patches try to switch revision formats by repository classes (Mercurial or not).<br />Implemented as follows:
<ul>
<li>implement Changeset#format_identifier and Revision#format_identifier</li>
<li>pass changeset or revision object to format_revision and link_to_revision helpers,<br /> so that it can utilize #format_identifier</li>
</ul> Redmine - Feature #6092: Truncate Git revision labels in Activity page/feed and allow configurable lengthhttps://www.redmine.org/issues/6092?journal_id=198712010-08-31T13:03:23ZToshi MARUYAMA
<ul></ul><p>Adam Soltys wrote:</p>
<blockquote>
<p>Toshi, ok, I did these two things as settings for you, but I don't actually think they should be included in trunk and presented to the average user. They're specific to git, for one thing, and they deal with some pretty obscure internal performance mechanisms.</p>
<p>A better solution in my opinion would be to revisit the git adapter and see if we can get some more performance out of it by default. If github can do it, so can we!</p>
</blockquote>
<p>Github seems to use Ajax for calling "git log -n 1 FILE_or_DIR".</p>
<p>Redmine calls "git log -n 1 FILE_or_DIR" not only in browsing directory tree but also in cat/diff/annotate.<br />Because entry() of abstract_adapter.rb calls entries() at <a class="source" href="https://www.redmine.org/projects/redmine/repository/svn/entry/tags/1.0.1/lib/redmine/scm/adapters/abstract_adapter.rb#L84">source:tags/1.0.1/lib/redmine/scm/adapters/abstract_adapter.rb#L84</a> .</p> Redmine - Feature #6092: Truncate Git revision labels in Activity page/feed and allow configurable lengthhttps://www.redmine.org/issues/6092?journal_id=204722010-09-19T14:17:37ZToshi MARUYAMA
<ul><li><strong>File</strong> <a href="/attachments/4502">yuya-issue-4455-mq-ab997af9e-exclude-hg.diff</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/4502/yuya-issue-4455-mq-ab997af9e-exclude-hg.diff">yuya-issue-4455-mq-ab997af9e-exclude-hg.diff</a> added</li><li><strong>File</strong> <a href="/attachments/4503">git-act-before.png</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/4503/git-act-before.png">git-act-before.png</a> added</li><li><strong>File</strong> <a href="/attachments/4504">git-act-after.png</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/4504/git-act-after.png">git-act-after.png</a> added</li></ul><p>I imported Yuya's <a href="http://bitbucket.org/yuja/redmine-mq-issue4455/changeset/ab997af9efe7" class="external">MQ</a> (Mercurial Queues) exclude Mercurial adapter to <a href="http://github.com/marutosi/redmine/commit/9f1b56ec3947b4c8e295c796907bfef4bb968360" class="external">github</a> and done <a href="http://github.com/marutosi/redmine/commit/9225af5c2a4f5410f53cb8c2ec4f869e2dc11d9a" class="external">minor change</a>.<br />Now my github branch is <a href="http://github.com/marutosi/redmine/commits/issue-4455-yuya-mq-20100825-exclude-hg" class="external">issue-4455-yuya-mq-20100825-exclude-hg</a></p>
<p>And I attach the patch and git activity page images of <a class="source" href="https://www.redmine.org/projects/redmine/repository/svn/entry/tags/1.0.1/test/fixtures/repositories/git_repository.tar.gz">source:tags/1.0.1/test/fixtures/repositories/git_repository.tar.gz</a>.</p> Redmine - Feature #6092: Truncate Git revision labels in Activity page/feed and allow configurable lengthhttps://www.redmine.org/issues/6092?journal_id=208082010-09-28T14:17:16ZToshi MARUYAMA
<ul></ul><p>Adam Soltys wrote:</p>
<blockquote>
<p>I need to do a bit more thinking and design-work before I tackle a full-on reimplementation of the git and mercurial adapters though. So in the meantime I'm just trying to get my feet wet again with contributing to redmine. I do have plans to become a regular contributor through the coming fall and winter though and repository management and performance will be a central area of interest for me.</p>
</blockquote>
<p>Adam, please consider <a class="issue tracker-1 status-1 priority-4 priority-default" title="Defect: Git: SCM revisions ordered by date/time (should be reverse commit order) (New)" href="https://www.redmine.org/issues/5357">#5357</a>.<br />Mercurial has same problem (<a class="issue tracker-1 status-5 priority-4 priority-default closed" title="Defect: Sorting for changesets might go wrong on Mercurial repos (Closed)" href="https://www.redmine.org/issues/3567">#3567</a>).<br />Yuya fixed this problem at <a href="http://www.redmine.org/issues/4455#note-144" class="external">note-144</a> of <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Feature: Mercurial overhaul (Closed)" href="https://www.redmine.org/issues/4455">#4455</a>.</p> Redmine - Feature #6092: Truncate Git revision labels in Activity page/feed and allow configurable lengthhttps://www.redmine.org/issues/6092?journal_id=208552010-09-29T22:53:00ZToshi MARUYAMA
<ul></ul><p>FYI:<br /><a href="http://www.redmine.org/boards/2/topics/17587" class="external">Bazaar merged revisions don't show up</a></p> Redmine - Feature #6092: Truncate Git revision labels in Activity page/feed and allow configurable lengthhttps://www.redmine.org/issues/6092?journal_id=214312010-10-15T07:13:43ZToshi MARUYAMA
<ul></ul><p>Toshi MARUYAMA wrote:</p>
<blockquote>
<p>"git log -n 1 FILE_or_DIR" problem is reported at <a class="issue tracker-1 status-5 priority-4 priority-default closed" title="Defect: Redmine hangs up while browsing Git repository (Closed)" href="https://www.redmine.org/issues/5096">#5096</a>.</p>
</blockquote>
<p>I found JPL posted a patch for this problem at <a href="http://www.redmine.org/issues/1435#note-13" class="external">note-13</a> of <a class="issue tracker-1 status-5 priority-4 priority-default closed" title="Defect: slow display of 'repository' tab for huge repository (pagination problem?) (Closed)" href="https://www.redmine.org/issues/1435">#1435</a> .<br />But current redmine support git branch, this patch is incompatible.</p> Redmine - Feature #6092: Truncate Git revision labels in Activity page/feed and allow configurable lengthhttps://www.redmine.org/issues/6092?journal_id=231582010-12-13T14:52:55ZToshi MARUYAMA
<ul><li><strong>File</strong> <a href="/attachments/5004">revision-truncate-20101213.diff</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/5004/revision-truncate-20101213.diff">revision-truncate-20101213.diff</a> added</li><li><strong>File</strong> <a href="/attachments/5005">git-12-digits-activity.png</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/5005/git-12-digits-activity.png">git-12-digits-activity.png</a> added</li><li><strong>File</strong> <a href="/attachments/5006">git-12-digits-annotate.png</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/5006/git-12-digits-annotate.png">git-12-digits-annotate.png</a> added</li><li><strong>File</strong> <a href="/attachments/5007">git-12-digits-browse.png</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/5007/git-12-digits-browse.png">git-12-digits-browse.png</a> added</li></ul><p>I update the patch to set truncating character number at adapter level.<br />This patch contains unit tests.</p>
<p>And I pushed to my github.</p>
<ul>
<li><a class="external" href="https://github.com/marutosi/redmine/commits/git-activity">https://github.com/marutosi/redmine/commits/git-activity</a></li>
<li><a class="external" href="https://github.com/marutosi/redmine/commit/710d3f355de44d87ccb3ec0caf49ec77917ca16b">https://github.com/marutosi/redmine/commit/710d3f355de44d87ccb3ec0caf49ec77917ca16b</a></li>
</ul>
<p>This patch truncates 8 characters.</p>
<p>You can set truncating number.</p>
<ul>
<li>blame/annotate<br /> <a class="external" href="https://github.com/marutosi/redmine/blob/710d3f355de44d87/lib/redmine/scm/adapters/git_adapter.rb#L271">https://github.com/marutosi/redmine/blob/710d3f355de44d87/lib/redmine/scm/adapters/git_adapter.rb#L271</a></li>
<li>Others<br /> <a class="external" href="https://github.com/marutosi/redmine/blob/710d3f355de44d87/app/models/repository/git.rb#L39">https://github.com/marutosi/redmine/blob/710d3f355de44d87/app/models/repository/git.rb#L39</a></li>
</ul>
<p>I attach truncating 12 characters images.</p> Redmine - Feature #6092: Truncate Git revision labels in Activity page/feed and allow configurable lengthhttps://www.redmine.org/issues/6092?journal_id=231902010-12-15T04:10:58ZToshi MARUYAMA
<ul></ul><p><a class="issue tracker-2 status-5 priority-3 priority-lowest closed" title="Feature: Shorten git commit ids in Atom feeds and on activity page (Closed)" href="https://www.redmine.org/issues/3244">#3244</a>, <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Feature: Shorten revision ID for git events (Closed)" href="https://www.redmine.org/issues/3945">#3945</a> and <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Feature: Truncate Git revision labels in Activity page/feed and allow configurable length (Closed)" href="https://www.redmine.org/issues/6092">#6092</a> are duplicate.</p> Redmine - Feature #6092: Truncate Git revision labels in Activity page/feed and allow configurable lengthhttps://www.redmine.org/issues/6092?journal_id=233242010-12-20T11:06:04ZToshi MARUYAMA
<ul><li><strong>File</strong> <a href="/attachments/5034">revision-truncate-20101213-fix-fixtures.diff</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/5034/revision-truncate-20101213-fix-fixtures.diff">revision-truncate-20101213-fix-fixtures.diff</a> added</li></ul><p><a class="attachment" href="https://www.redmine.org/attachments/5004">revision-truncate-20101213.diff</a> has a lack of fixtures at unit tests.<br />I fixed it.<br />revision-truncate-20101213-fix-fixtures.diff is additional patch for <a class="attachment" href="https://www.redmine.org/attachments/5004">revision-truncate-20101213.diff</a> .</p>
<p>And I pushed to my github.</p>
<ul>
<li><a class="external" href="https://github.com/marutosi/redmine/commits/git-activity">https://github.com/marutosi/redmine/commits/git-activity</a></li>
<li><a class="external" href="https://github.com/marutosi/redmine/commit/325ee1e73bafeae5567d3e3a7ba09d1d2181841a">https://github.com/marutosi/redmine/commit/325ee1e73bafeae5567d3e3a7ba09d1d2181841a</a></li>
</ul> Redmine - Feature #6092: Truncate Git revision labels in Activity page/feed and allow configurable lengthhttps://www.redmine.org/issues/6092?journal_id=236172011-01-02T06:46:39ZToshi MARUYAMA
<ul><li><strong>Category</strong> changed from <i>UI</i> to <i>SCM</i></li><li><strong>Assignee</strong> set to <i>Toshi MARUYAMA</i></li></ul> Redmine - Feature #6092: Truncate Git revision labels in Activity page/feed and allow configurable lengthhttps://www.redmine.org/issues/6092?journal_id=236202011-01-02T09:55:44ZToshi MARUYAMA
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>Closed</i></li><li><strong>Target version</strong> set to <i>1.1.0</i></li><li><strong>Resolution</strong> set to <i>Fixed</i></li></ul><p>Implemented in svn trunk by <a class="changeset" title="Changing revision label and identifier at SCM adapter level (#3724, #6092) Contributed by Yuya N..." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/4613">r4613</a> and 1.1 by <a class="changeset" title="Merged r4613 from trunk." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/4614">r4614</a>.</p>