Mercurial > hg > thermostat-tools-eclipse

changeset 139:9a72cb1423af

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Update TODOs and FIXMEs in export wizard
author Omair Majid <omajid@redhat.com>
date Thu, 05 Jun 2014 13:16:42 -0400
parents a9c281500b55
children e605201414d0
files com.redhat.thermostat.tools.eclipse.plugin/src/com/redhat/thermostat/tools/eclipse/plugin/wizards/ExportPluginWizard.java
diffstat 1 files changed, 5 insertions(+), 3 deletions(-) [+]
line wrap: on
line diff
--- a/com.redhat.thermostat.tools.eclipse.plugin/src/com/redhat/thermostat/tools/eclipse/plugin/wizards/ExportPluginWizard.java	Thu Jun 05 13:01:00 2014 -0400
+++ b/com.redhat.thermostat.tools.eclipse.plugin/src/com/redhat/thermostat/tools/eclipse/plugin/wizards/ExportPluginWizard.java	Thu Jun 05 13:16:42 2014 -0400
@@ -79,12 +79,14 @@
 
             monitor.worked(1);
 
+            // FIXME don't hardcode distribution project name
             final String distributionModuleName = project.getName() + "-distribution";
-            // TODO unzip and extract to thermostatLocation
             File projectDir = project.getLocation().toFile();
-            // FIXME don't hardcode distribution project name
             File distributionDir = new File(projectDir, distributionModuleName);
             File distributionTargetDir = new File(distributionDir, "target");
+
+            // FIXME find the zip matching the exact pom version
+
             String[] files = distributionTargetDir.list(new FilenameFilter() {
                 @Override
                 public boolean accept(File dir, String name) {
@@ -127,7 +129,7 @@
                 processBuilder.redirectOutput(ProcessBuilder.Redirect.PIPE);
                 Process process = processBuilder.start();
 
-                // FIXME make this non-blocking. Add tons of feedback.
+                // FIXME expose process result to user for feedback.
 
                 boolean result = process.waitFor(5, TimeUnit.MINUTES);
                 if (!result) {