2.1.24

.gitlab-ci.yml

@@ -1,9 +1,10 @@
 image: hosttoday/ht-docker-node:lts
 
 stages:
-- test
 - release
 - trigger
+- test
+
 
 before_script:
   - npm install -g npmci

package.json

@@ -1,6 +1,6 @@
 {
   "name": "npmci",
-  "version": "2.1.16",
+  "version": "2.1.24",
   "description": "",
   "main": "dist/index.js",
   "bin": {
@@ -32,7 +32,7 @@
     "shelljs": "^0.7.0",
     "smartfile": "^3.0.10",
     "smartssh": "^1.0.5",
-    "smartstring": "^2.0.4",
+    "smartstring": "^2.0.6",
     "through2": "^2.0.1",
     "typings-global": "^1.0.3"
   }

ts/npmci.build.docker.ts

@@ -29,31 +29,23 @@ let readDockerfiles = function(){
 
 let sortDockerfiles = function(){
     let done = plugins.q.defer();
-    let redoSort:boolean;
-    let sortFunction = function(){
-        redoSort = false;
-        let notYetBuiltImages:string[] = [];
-        NpmciEnv.dockerFiles.forEach((dockerFileArg)=>{
-            notYetBuiltImages.push(dockerFileArg.cleanTag);
+    NpmciEnv.dockerFiles.sort(function(a,b){
+        let cleanTags = [];
+        NpmciEnv.dockerFiles.forEach(function(dockerfileArg){
+            cleanTags.push(dockerfileArg.cleanTag);
         });
-        NpmciEnv.dockerFiles.sort(function(a,b){
-            plugins.beautylog.log("sort build order for Dockerimages");
-            let aIndex = notYetBuiltImages.indexOf(a.cleanTag);
-            if(aIndex != -1){notYetBuiltImages.splice(aIndex,1)}
-            if(notYetBuiltImages.indexOf(b.cleanTag) != -1){
-                redoSort = true;
-                return -1;
-            } else {
-                return 0
-            }
-        });
-        if(redoSort){
-            sortFunction();
+        let aIndex = cleanTags.indexOf(a.cleanTag);
+        let bIndex = cleanTags.indexOf(b.baseImage);
+        console.log(cleanTags);
+        console.log(a.cleanTag);
+        console.log(b.cleanTag);
+        if(bIndex < aIndex && bIndex != -1){
+            return -1;
         } else {
-            done.resolve();
+            return 0
         }
-    };
-    sortFunction();
+    });
+    done.resolve();
     return done.promise;
 }
 
@@ -108,7 +100,7 @@ let dockerFileVersion = function(dockerfileNameArg:string):string{
     let versionString:string;
     let versionRegex = /Dockerfile_([a-zA-Z0-9\.]*)$/;
     let regexResultArray = versionRegex.exec(dockerfileNameArg);
-    if(regexResultArray.length = 2){
+    if(regexResultArray && regexResultArray.length == 2){
         versionString = regexResultArray[1];        
     } else {
         versionString = "latest";