fixed the update manifest, now created jar has modifications

Ortus-Solutions · Dec 9, 2021 · 8f7fd9b · 8f7fd9b
1 parent 7ae9220
commit 8f7fd9b
Showing 1 changed file with 24 additions and 9 deletions.
diff --git a/src/cfml/system/util/CompileDSL.cfc b/src/cfml/system/util/CompileDSL.cfc
@@ -99,7 +99,9 @@ component accessors=true {
     }
 
 	function toJar( string jarName='' ) {
-		setJarNameString( jarName );
+		if( jarName.len() ) {
+			setJarNameString( jarName );
+		}
 		setCreateJar( true );
 		return this;
 	}
@@ -119,10 +121,11 @@ component accessors=true {
 		return this;
 	}
 
-	function addToManifest( required string customManifest ) {
+	//no longer needed
+	/* function addToManifest( required string customManifest ) {
 		setCustomManifest( fileSystemutil.resolvePath( customManifest, getProjectRoot() ) );
 		return this;
-	}
+	} */
 
 	function manifest( required struct customParams ) {
 		setCustomManifestParams( customParams );
@@ -288,13 +291,15 @@ component accessors=true {
 
 				}
 				jarName &= word & ".jar";
-				setJarNameString( jarName );
-				//job.addLog( ' jarName= #jarName# ' );
 
             }
 
         }
 
+		setJarNameString( jarName );
+		//job.addLog( ' jarName= #jarName# ' );
+		//job.addLog( ' jarName*= #getJarNameString()# ' );
+
         job.complete();
 
         try{
@@ -308,12 +313,13 @@ component accessors=true {
             //j = 'run "#getJavaBinFolder()#jar" --file #currentLibsDir##jarName# #getJarOptionsString()#';
             //j = 'run "#getJavaBinFolder()#jar" --create --file #currentLibsDir#testX.jar "@#tempSrcFileName#" #getJarOptionsString()#';
 			if( !getCustomManifest().len() ) {
-				j = 'run ""#getJavaBinFolder()#jar" cf "#currentLibsDir#\#jarName#" "@#tempSrcFileName#" #getJarOptionsString()#"';
+				j = 'run ""#getJavaBinFolder()#jar" cf "#currentLibsDir##jarName#" "@#tempSrcFileName#" #getJarOptionsString()#"';
 			} else {
-            	j = 'run ""#getJavaBinFolder()#jar" cfm "#currentLibsDir#\#jarName#" "#variables.customManifest#" "@#tempSrcFileName#" #getJarOptionsString()#"';
+            	j = 'run ""#getJavaBinFolder()#jar" cfm "#currentLibsDir##jarName#" "#variables.customManifest#" "@#tempSrcFileName#" #getJarOptionsString()#"';
 			}
             job.addLog( j );
-            command( j ).run(echo=true);
+            //command( j ).run(echo=true);
+			command( j ).run();
 
         } finally {
 			if ( FileExists( tempSrcFileName ) ) {
@@ -339,7 +345,7 @@ component accessors=true {
 
 			j = 'run ""#getJavaBinFolder()#jar" uf "#currentLibsDir##jarName#" -C #currentResourcePath# . "'
 
-			job.addLog( j );
+			//job.addLog( j );
 			//command( j ).run(echo=true);
 			command( j ).run();
 
@@ -407,6 +413,15 @@ component accessors=true {
 			try {
 				writeUpdateManifestFile( tempUpdateManifestFileName, paramStruct );
 			} finally {
+				/*
+				if the file was created then we have to save the filename and path
+				to use it when creating the jar
+				*/
+				if( FileExists( tempUpdateManifestFileName ) ) {
+					setCustomManifest( tempUpdateManifestFileName );
+				}
+
+				// not needed here any more
 				/* if ( FileExists( tempUpdateManifestFileName ) ) {
 					fileDelete( tempUpdateManifestFileName );
 				} */