ereviewboard Allow "Create Review Request" on multiple projecs, if they are from the same SVN repository

It would be nice, if the creation of a review request would be possible for changes over multiple projects. I guess the problem would be that different projects can be from different repositories. But as the many subclipse actions, it would be sufficient to allow the creation of reviews requests if all selected projects are from the same SVN repository.

Jan 23 '13 14:01 haraldreingruber

Maybe I should mention that we use Subclipse. Don't know if it works with Subversive.

Jan 24 '13 09:01 haraldreingruber

This is related to #115 . Right now I won't have time to implement this. Would you be willing to try and contribute this feature? I'm here to assist.

Jan 28 '13 17:01 rombert

I will try, but finding time is also for me difficult. Maybe I can work out a patch for a first proposal in the next months. I'll let you know.

Feb 15 '13 12:02 haraldreingruber

We've been looking into how to implement this one. What we came up so far is:

create one diff per selected project
in the diff, add the base directory that RB needs to all file references, for example: Index: src/com/foo/Foo.java would become Index: project/trunk/src/com/foo/Foo.java
concatenate all project diffs into one
use / for the base directory when posting the review

This seems to work, however it is then not possible to apply the patch in Eclipse directly when doing a "Download Diff" from RB. We could then of course adapt the "Apply Diff" functionality provided by the plugin to remove the base directory from the patch again and make it possible.

@rombert: you think this would be a viable solution? Or you have an alternative?

Philippe

Apr 29 '13 12:04 philippecade

Well, if you have a working change it's great! Can you create a pull request?

Apr 30 '13 19:04 rombert