Community
Participate
Working Groups
I am running on localhost, win7 with: - latest code from orion git client and server repos - I20110607-0200 as my target platform None of the diffs in the git status page are working for me. Both the inline compare and the side by side compare and failing with suspicious looking URL's. For example I have a change in globalCommands.js and if I try to look at the diff I get the error: Unable to load /gitapi/diff/Default/file/A/web/bundles/org.eclipse.orion.client.core/web/orion/globalCommands.js?conflict=true?conflict=true&parts=diff status:500 Note the double ?conflict=true?conflict=true in the URL Line 849-850 in git-status-table.js looks very suspicious: var diffURI = (this._model.isConflict(itemModel.type) ? itemModel.diffURI : itemModel.diffURI + "?conflict=true"); this._inlineCompareContainer.resolveDiff(diffURI + "?conflict=true", I hacked the file to get rid of the duplicate "?conflict=true" but the URL is still not right. Libing said it was working for him with the latest, but I'm wondering if he had different git-status-table code running when he said this (because he had diffs in that file). So I think that correct/current code might not be pushed yet and there is failing code in the repo.
For the two-way compare, I'm seeing something that looks like path munging. For example: http://localhost:8080/compare/compare.html#/gitapi/diff/Default/file/k/bundles/bundles/org.eclipse.orion.client.core/web/orion/breadcrumbs.js This fails. If I change the browser bar location so get rid of the extra "/bundles" segment, then it works.
Mark noticed that it works correctly if you open git-status from the root directory of the repo. But if you open git-status from somewhere below the root, you see both of these diff errors. This is very discouraging because I thought git-status is not supposed to be scoped in any way.
(In reply to comment #2) > This is very discouraging because I thought git-status is not supposed to be > scoped in any way. I'm saying this based on discussion from bug 339558 (in particular see bug 339558 comment 9 and bug 339558 comment 10). Whether or not we show the git-status command from subfolders, I would expect the page to behave the same once it opens. It seems like we have a subtle difference somewhere that shouldn't be so.
(In reply to comment #2) > Mark noticed that it works correctly if you open git-status from the root > directory of the repo. > > But if you open git-status from somewhere below the root, you see both of these > diff errors. > > This is very discouraging because I thought git-status is not supposed to be > scoped in any way. I reproduced it in a sub folder , investigating it now.
Hi , Tomasz : Please look at this . When I go to git status page from a sub folder , from the git status REST API I saw that the all the file locations have double "/bundles" CommitLocation "/gitapi/commit/HEAD/file/K/bundles/bundles/org.eclipse.orion.client.core/web-ide.conf" DiffLocation "/gitapi/diff/Default/file/K/bundles/bundles/org.eclipse.orion.client.core/web-ide.conf" IndexLocation "/gitapi/index/file/K/bundles/bundles/org.eclipse.orion.client.core/web-ide.conf" Location "/file/K/bundles/bundles/org.eclipse.orion.client.core/web-ide.conf" Name "bundles/org.eclipse.orion.client.core/web-ide.conf" Path "org.eclipse.orion.client.core/web-ide.conf"
Fixed with http://git.eclipse.org/c/e4/org.eclipse.orion.server.git/commit/?id=63ab0f0344b8c0da958e169bff7aa20c7c0f8fe5.
