Merge pull request #749 from JunoLab/sp/fixrelativepaths

fix relative julia path

Author: aviatesk

lib/misc/paths.js

@@ -24,13 +24,18 @@ export function expandHome (p) {
 
 export function fullPath (p) {
   return new Promise((resolve, reject) => {
-    if (fs.existsSync(p)) return resolve(p)
+    if (fs.existsSync(p)) {
+      return resolve(fs.realpathSync(p))
+    }
     const current_dir = process.cwd()
     const exepath = path.dirname(process.execPath)
+
     try {
       process.chdir(exepath)
       const realpath = fs.realpathSync(p)
-      fs.existsSync(realpath) && resolve(realpath)
+      if (fs.existsSync(realpath)) {
+        resolve(realpath)
+      }
     } catch (err) {
       console.log(err)
     } finally {
@@ -55,12 +60,16 @@ export function fullPath (p) {
 
 export function getVersion (path = jlpath()) {
   return new Promise((resolve, reject) => {
-    child_process.exec(`"${path}" --version`, (err, stdout, stderr) => {
-      if (err) return reject(stderr)
-      const res = stdout.match(/(\d+)\.(\d+)\.(\d+)/)
-      if (!res) return reject('Couldn\'t resolve version.')
-      const [_, major, minor, patch] = res
-      return resolve({ major, minor, patch })
+    fullPath(path).then(path => {
+      child_process.exec(`"${path}" --version`, (err, stdout, stderr) => {
+        if (err) return reject(stderr)
+        const res = stdout.match(/(\d+)\.(\d+)\.(\d+)/)
+        if (!res) return reject('Couldn\'t resolve version.')
+        const [_, major, minor, patch] = res
+        return resolve({ major, minor, patch })
+      })
+    }).catch(e => {
+      reject('Couldn\'t resolve version.')
     })
   })
 }