Robin Rosenberg <robin.rosenberg@xxxxxxxxxx> wrote: > This adds an optional page for importing Eclipse style projects > as part of the clone operation. > > This patch I think, greatly simplifies getting an Eclipse based project > into the workbench. Yes, yes it does! > I'm mostly interested in comments on pure bugs. E.g. know it's > blocking when the user selects to import the projects as part of the > clone process, I also know I'm reusing internal messages and not supplying > NLS style constansts for all textx. I noticed you mispelled "showImportWizard" as "showImportWizarad", but otherwise it looks reasonable to me. I'd like to see the NLS stuff extracted out before we apply the patch though. The blocking part is annoying, but there isn't really a better way to do it. What would be worse is going into the background, then popping open a modal dialog when it completes, as it would really disrupts the user. So just don't clone massive projects and expect to use your workbench at the same time. :-) -- Shawn. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
