You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bigtop.apache.org by of...@apache.org on 2016/02/25 20:55:44 UTC

[1/2] bigtop git commit: BIGTOP-2347: Clean up build directory after sucessfull build of package (2nd try)

Repository: bigtop
Updated Branches:
  refs/heads/master 510ac7b5e -> 9fbaed3ba


BIGTOP-2347: Clean up build directory after sucessfull build of package (2nd try)


Project: http://git-wip-us.apache.org/repos/asf/bigtop/repo
Commit: http://git-wip-us.apache.org/repos/asf/bigtop/commit/9fbaed3b
Tree: http://git-wip-us.apache.org/repos/asf/bigtop/tree/9fbaed3b
Diff: http://git-wip-us.apache.org/repos/asf/bigtop/diff/9fbaed3b

Branch: refs/heads/master
Commit: 9fbaed3ba488347f872cb763c924cb3353321b7b
Parents: 0bdf437
Author: Olaf Flebbe <of...@ofleb.be>
Authored: Tue Feb 23 21:00:52 2016 +0100
Committer: Olaf Flebbe <of...@ofleb.be>
Committed: Thu Feb 25 20:53:20 2016 +0100

----------------------------------------------------------------------
 packages.gradle | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/bigtop/blob/9fbaed3b/packages.gradle
----------------------------------------------------------------------
diff --git a/packages.gradle b/packages.gradle
index 9e045d8..44160ef 100644
--- a/packages.gradle
+++ b/packages.gradle
@@ -394,7 +394,7 @@ def genTasks = { target ->
     fileTree (dir: "$DEB_PKG_DIR/..", includes: ['*.dsc', '*.diff.gz', '*.debian.tar.gz', '*.debian.tar.xz', "*_source.changes", "*.orig.tar.gz" ]).copy {
       into PKG_OUTPUT_DIR
     }
-    safeDelete(BUILD_DIR)
+    safeDelete(DEB_BLD_DIR)
     touchTargetFile(config.bigtop.components[target].targetsdeb)
   }
 


[2/2] bigtop git commit: BIGTOP-2346: Do not use gradle delete() for sources and build directories

Posted by of...@apache.org.
BIGTOP-2346: Do not use gradle delete() for sources and build directories


Project: http://git-wip-us.apache.org/repos/asf/bigtop/repo
Commit: http://git-wip-us.apache.org/repos/asf/bigtop/commit/0bdf4373
Tree: http://git-wip-us.apache.org/repos/asf/bigtop/tree/0bdf4373
Diff: http://git-wip-us.apache.org/repos/asf/bigtop/diff/0bdf4373

Branch: refs/heads/master
Commit: 0bdf43735f299428915fe79e0cc526c1909db75f
Parents: 510ac7b
Author: Olaf Flebbe <of...@ofleb.be>
Authored: Tue Feb 23 20:57:54 2016 +0100
Committer: Olaf Flebbe <of...@ofleb.be>
Committed: Thu Feb 25 20:53:20 2016 +0100

----------------------------------------------------------------------
 packages.gradle | 24 +++++++++++++++---------
 1 file changed, 15 insertions(+), 9 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/bigtop/blob/0bdf4373/packages.gradle
----------------------------------------------------------------------
diff --git a/packages.gradle b/packages.gradle
index c29c5c1..9e045d8 100644
--- a/packages.gradle
+++ b/packages.gradle
@@ -71,6 +71,12 @@ def ifExists = { url ->
   URLConnection uCon = new URL(url).openConnection()
   return (uCon as HttpURLConnection).responseCode == 200
 }
+def safeDelete = { fileName ->
+  exec {
+    workingDir '.'
+    commandLine 'rm', '-rf', fileName
+  }
+}
 def getDate() {
   new Date().format('E, dd MMM yyyy HH:mm:ss Z')
 }
@@ -226,7 +232,7 @@ def genTasks = { target ->
     def final DOWNLOAD_DST = config.bigtop.components[target].downloaddst ?: ""
     def final SEED_TAR = config.bigtop.components[target].seedtar
 
-    delete(TAR_DIR); mkdir(TAR_DIR)
+    safeDelete(TAR_DIR); mkdir(TAR_DIR)
 
     if (TARBALL_SRC.isEmpty() || TARBALL_SRC.endsWith('.zip')) {
       if (TARBALL_SRC.isEmpty()) {
@@ -250,7 +256,7 @@ def genTasks = { target ->
             (f != "." && f != "..")}).each { f ->
               new File("$TAR_DIR/$TOP_LEVEL_DIR/$f").renameTo("$TAR_DIR/$f")
           }
-          delete(TOP_LEVEL_DIR)
+          safeDelete(TOP_LEVEL_DIR)
         }
       }
       // create SEED_TAR
@@ -388,7 +394,7 @@ def genTasks = { target ->
     fileTree (dir: "$DEB_PKG_DIR/..", includes: ['*.dsc', '*.diff.gz', '*.debian.tar.gz', '*.debian.tar.xz', "*_source.changes", "*.orig.tar.gz" ]).copy {
       into PKG_OUTPUT_DIR
     }
-    delete (BUILD_DIR)
+    safeDelete(BUILD_DIR)
     touchTargetFile(config.bigtop.components[target].targetsdeb)
   }
 
@@ -450,7 +456,7 @@ def genTasks = { target ->
     def final PKG_VERSION = config.bigtop.components[target].version.pkg
     def final BASE_VERSION = config.bigtop.components[target].version.base
     def final PKG_OUTPUT_DIR = config.bigtop.components[target].outputdir
-    delete ("$PKG_BUILD_DIR/rpm")
+    safeDelete ("$PKG_BUILD_DIR/rpm")
     ['INSTALL','SOURCES','BUILD','SRPMS','RPMS'].each { rpmdir ->
       mkdir("$PKG_BUILD_DIR/rpm/$rpmdir")
     }
@@ -592,8 +598,8 @@ def genTasks = { target ->
   task "$target-clean" (dependsOn: "${target}_vardefines",
       description: "Removing $target component build and output directories",
       group: PACKAGES_GROUP) << {
-    delete(config.bigtop.components[target].builddir)
-    delete(config.bigtop.components[target].outputdir)
+    safeDelete(config.bigtop.components[target].builddir)
+    safeDelete(config.bigtop.components[target].outputdir)
   }
   task "$target-help" (description: "List of available tasks for $target", group: PACKAGES_GROUP) << {
     println (target + "\n\t[" + tasks.findAll { alltask -> alltask.name.startsWith(target)}*.name.join(", ") + "]")
@@ -649,9 +655,9 @@ project.afterEvaluate {
       description: "Removing $BUILD_DIR, $OUTPUT_DIR, and $DIST_DIR.\n\t\t" +
         "Cleaning all components' build and output directories.",
       group: PACKAGES_GROUP) << {
-    delete (BUILD_DIR)
-    delete (OUTPUT_DIR)
-    delete (DIST_DIR)
+    safeDelete(BUILD_DIR)
+    safeDelete(OUTPUT_DIR)
+    safeDelete(DIST_DIR)
   }
   task realclean (dependsOn: allclean,
       description: "Removing $BUILD_DIR, $OUTPUT_DIR, $DIST_DIR, and $DL_DIR",