https://www.redmine.org/https://www.redmine.org/favicon.ico?16793021292009-06-08T20:02:29ZRedmineRedmine - Defect #3421: Mercurial reads files from working dir instead of changesetshttps://www.redmine.org/issues/3421?journal_id=96592009-06-08T20:02:29ZLuke Hoersten
<ul><li><strong>Assignee</strong> set to <i>Jean-Philippe Lang</i></li><li><strong>% Done</strong> changed from <i>0</i> to <i>10</i></li></ul><p>A friend is working on a patch for me. Will submit it when we have it.</p> Redmine - Defect #3421: Mercurial reads files from working dir instead of changesetshttps://www.redmine.org/issues/3421?journal_id=96712009-06-09T13:33:28ZLuke Hoersten
<ul><li><strong>File</strong> <a href="/attachments/2165">redmine-mercurial.patch</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/2165/redmine-mercurial.patch">redmine-mercurial.patch</a> added</li><li><strong>% Done</strong> changed from <i>10</i> to <i>50</i></li></ul><p>That'd be great if we could get this patch reviewed and applied. Currently, Mercurial support is nothing more than reading the contents out of some directory without this change. The change actually reads the repository itself.</p> Redmine - Defect #3421: Mercurial reads files from working dir instead of changesetshttps://www.redmine.org/issues/3421?journal_id=103952009-07-29T06:19:05ZJean-Philippe Langjp_lang@yahoo.fr
<ul></ul><p>I'm not sure to understand the actual defect. Please include unit tests in your patch.</p> Redmine - Defect #3421: Mercurial reads files from working dir instead of changesetshttps://www.redmine.org/issues/3421?journal_id=107202009-08-29T06:03:53ZAnil Gulecha
<ul></ul><p>I see the same issue.. I pointed the repository for a project to a local clone of the mercurial repository.. and clicking on Repository takes a long while (It's been about half an hour, and the repository page still wont open). This is the opensolaris kernel repository. If there's any information I can provide about the setup, let me know.</p> Redmine - Defect #3421: Mercurial reads files from working dir instead of changesetshttps://www.redmine.org/issues/3421?journal_id=110232009-09-17T21:29:27ZLuke Hoersten
<ul></ul><p>Jean-Philippe Lang wrote:</p>
<blockquote>
<p>I'm not sure to understand the actual defect. Please include unit tests in your patch.</p>
</blockquote>
<p>Do you understand what the staging area is in Git and why that is not the same thing as the repository? It's similar to that in Mercurial with the working director. I'm not sure what unit tests you are looking for. It won't show anything without a Mercurial repo attached to it.</p> Redmine - Defect #3421: Mercurial reads files from working dir instead of changesetshttps://www.redmine.org/issues/3421?journal_id=110392009-09-18T06:00:59ZJean-Philippe Langjp_lang@yahoo.fr
<ul></ul><p>Luke Hoersten wrote:</p>
<blockquote>
<p>I'm not sure what unit tests you are looking for.</p>
</blockquote>
<p>A test that would fail with the actual code and pass with your patch.</p>
<blockquote>
<p>It won't show anything without a Mercurial repo attached to it.</p>
</blockquote>
<p>Indeed. That's why there are test repositories (see doc/RUNNING_TESTS)</p> Redmine - Defect #3421: Mercurial reads files from working dir instead of changesetshttps://www.redmine.org/issues/3421?journal_id=130982009-12-19T01:04:57ZPeter Fern
<ul></ul><p>Umm, where are you getting this 'hg size' command from?? It doesn't exist on any version I have access to...</p> Redmine - Defect #3421: Mercurial reads files from working dir instead of changesetshttps://www.redmine.org/issues/3421?journal_id=131012009-12-19T02:52:42ZLuke Hoersten
<ul></ul><p>Peter Fern wrote:</p>
<blockquote>
<p>Umm, where are you getting this 'hg size' command from?? It doesn't exist on any version I have access to...</p>
</blockquote>
<p>Ian (the guy who wrote the patch) wrote a Mercurial plugin to give size. If it's not there it should just show '-' I think.</p> Redmine - Defect #3421: Mercurial reads files from working dir instead of changesetshttps://www.redmine.org/issues/3421?journal_id=131032009-12-19T03:07:31ZPeter Fern
<ul></ul><p>Can you provide the mercurial plugin here?</p> Redmine - Defect #3421: Mercurial reads files from working dir instead of changesetshttps://www.redmine.org/issues/3421?journal_id=131052009-12-19T03:20:22ZLuke Hoersten
<ul><li><strong>File</strong> <a href="/attachments/2925">size.py</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/2925/size.py">size.py</a> added</li></ul> Redmine - Defect #3421: Mercurial reads files from working dir instead of changesetshttps://www.redmine.org/issues/3421?journal_id=131062009-12-19T03:25:26ZLuke Hoersten
<ul></ul><p>Peter Fern wrote:</p>
<blockquote>
<p>I think you'll find that's incorrect, and looking at the patch, it doesn't modify the 'cat' method in any way other than to add a default identifier of 'tip'...</p>
</blockquote>
<p>Add a file to your Mercurial, commit, then delete and commit. If you try to view that file through Redmine, it will say it's not there because it's trying to pull it from your working directory instead of the changeset itself.</p> Redmine - Defect #3421: Mercurial reads files from working dir instead of changesetshttps://www.redmine.org/issues/3421?journal_id=131072009-12-19T03:27:20ZPeter Fern
<ul></ul><p>In the mean time, I've added the size method, and a conditional in the model for my upcoming patchset to try the scm.size method and fall back to accessing the filesystem if that returns nil.</p> Redmine - Defect #3421: Mercurial reads files from working dir instead of changesetshttps://www.redmine.org/issues/3421?journal_id=131082009-12-19T03:38:16ZPeter Fern
<ul></ul><p>Luke Hoersten wrote:</p>
<blockquote>
<p>Add a file to your Mercurial, commit, then delete and commit. If you try to view that file through Redmine, it will say it's not there because it's trying to pull it from your working directory instead of the changeset itself.</p>
</blockquote>
<p>That wasn't happening here, but there were some errors handling revisions, etc - the 'entry does not exist in the repository' error is generic, so it's likely some other error was causing your problem, since the cat code currently in trunk works as expected.</p>
<p>Thanks for uploading the size extension, that's working great, and once I do some more testing, I'll get my patch up and link it to this issue for you to test.</p>
<p>If you're still in touch with Ian, can you ask him why he used <code>"glob:#{path}**"</code> instead of <code>"path:#{path}"</code> in the 'entries' method for the patch attached here?</p> Redmine - Defect #3421: Mercurial reads files from working dir instead of changesetshttps://www.redmine.org/issues/3421?journal_id=131092009-12-19T03:40:35ZPeter Fern
<ul></ul><p>Peter Fern wrote:</p>
<blockquote>
<p>since the cat code currently in trunk works as expected.</p>
</blockquote>
<p>Actually, the problem was likely that mercurial wasn't getting passed a revision when calling cat, which is also fixed in my upcoming patchset</p> Redmine - Defect #3421: Mercurial reads files from working dir instead of changesetshttps://www.redmine.org/issues/3421?journal_id=131102009-12-19T03:50:53ZLuke Hoersten
<ul></ul><p>Peter Fern wrote:</p>
<blockquote>
<p>Peter Fern wrote:</p>
<blockquote>
<p>since the cat code currently in trunk works as expected.</p>
</blockquote>
<p>Actually, the problem was likely that mercurial wasn't getting passed a revision when calling cat, which is also fixed in my upcoming patchset</p>
</blockquote>
<p>Sounds very likely. I'll give it a try when you're ready.<br />Thanks for all the work!</p> Redmine - Defect #3421: Mercurial reads files from working dir instead of changesetshttps://www.redmine.org/issues/3421?journal_id=131552009-12-21T00:31:30ZPeter Fern
<ul></ul><p>I can't set issue relationships, but the patch is at <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 - Defect #3421: Mercurial reads files from working dir instead of changesetshttps://www.redmine.org/issues/3421?journal_id=150462010-03-11T10:30:08ZToshi MARUYAMA
<ul><li><strong>File</strong> <a href="/attachments/3335">branch00.png</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/3335/branch00.png">branch00.png</a> added</li><li><strong>File</strong> <a href="/attachments/3336">branch01.png</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/3336/branch01.png">branch01.png</a> added</li><li><strong>% Done</strong> changed from <i>50</i> to <i>90</i></li></ul><p>I finish unit and functional test of this feature.<br /><a class="external" href="http://github.com/marutosi/redmine/tree/hg-overhaul-0.9">http://github.com/marutosi/redmine/tree/hg-overhaul-0.9</a><br /><a class="external" href="http://github.com/marutosi/redmine/commit/4c8794959c06205ba133ea7cdda59e4984d75915">http://github.com/marutosi/redmine/commit/4c8794959c06205ba133ea7cdda59e4984d75915</a></p> Redmine - Defect #3421: Mercurial reads files from working dir instead of changesetshttps://www.redmine.org/issues/3421?journal_id=236702011-01-04T05:00:45ZToshi MARUYAMA
<ul><li><strong>Category</strong> set to <i>SCM</i></li><li><strong>Assignee</strong> changed from <i>Jean-Philippe Lang</i> to <i>Toshi MARUYAMA</i></li><li><strong>% Done</strong> changed from <i>90</i> to <i>0</i></li></ul><p>Note 17 is obsolete. Please see <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 - Defect #3421: Mercurial reads files from working dir instead of changesetshttps://www.redmine.org/issues/3421?journal_id=252792011-02-18T04:14:50ZToshi 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.2.0</i></li><li><strong>Resolution</strong> set to <i>Fixed</i></li></ul><p>Fixed in <a class="changeset" title="scm: mercurial: unit lib test for entries (#3421)." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/4864">r4864</a>.</p>