Merge pull request #220 from sasjs/path-fix

fix: replace all backslashes with forward slash
sasjs · Dec 20, 2022 · 1334b2d · 1334b2d
2 parents 10a29cb + ce39dd6
commit 1334b2d
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/src/fs/sync.ts b/src/fs/sync.ts
@@ -7,6 +7,11 @@ import {
 import { HashedFolder } from '../types'
 import { generatePathForSas } from '../utils'
 
+// we need to inject this line to generated sas program
+// ls means line size and ps means page size
+// bigger page = less chance of hitting the page number
+const OPTIONS = 'options ls=max ps=max;'
+
 export const generateProgramToGetRemoteHash = async (remotePath: string) => {
   const compiledMacrosCode = await getCompiledMacrosCode([
     'mp_hashdirectory.sas',
@@ -21,7 +26,7 @@ export const generateProgramToGetRemoteHash = async (remotePath: string) => {
 
   const codeForHashCreation = getCodeForHashCreation()
 
-  const code = compiledMacrosCode + codeForHashCreation
+  const code = OPTIONS + compiledMacrosCode + codeForHashCreation
 
   return setTargetAtStart(code, remotePath)
 }
@@ -56,11 +61,12 @@ export const generateProgramToSyncHashDiff = async (
   const codeForHashCreation = getCodeForHashCreation()
 
   const code =
-    'options ls=max;' +
+    OPTIONS +
     compiledMacrosCode +
     initialProgramContent +
     folderCreationCode +
     codeForHashCreation
+
   return setTargetAtStart(code, remotePath)
 }
 

diff --git a/src/utils/utils.ts b/src/utils/utils.ts
@@ -36,4 +36,4 @@ export const getMacrosPath = () => path.join(__dirname, '..', 'macros')
  * So, this method replaces OS specific path separator with forward slash
  */
 export const generatePathForSas = (resourcePath: string) =>
-  resourcePath.replace(path.sep, '/')
+  resourcePath.replace(/\\/g, '/')