Merge pull request #22 from JFox/fix-docker-requirements

Fix requirements paths in container when using docker. Closes issue #21

Author: ubaniabalogun

index.js

@@ -32,6 +32,7 @@ class PkgPyFuncs {
     this.useDocker = config.useDocker || false
     this.dockerImage = config.dockerImage || `lambci/lambda:build-${this.serverless.service.provider.runtime}`
     this.containerName = config.containerName || 'serverless-package-python-functions'
+    this.dockerServicePath = '/var/task'
   }
 
   clean(){
@@ -75,13 +76,15 @@ class PkgPyFuncs {
     }
 
     let cmd = 'pip'
-    let args = ['install','--upgrade','-t', upath.normalize(buildPath), '-r', upath.normalize(requirementsPath)]
+    let args = ['install','--upgrade','-t', upath.normalize(buildPath), '-r']
     if ( this.useDocker === true ){
       cmd = 'docker'
-      args = ['exec',this.containerName, 'pip', ...args]
+      args = ['exec', this.containerName, 'pip', ...args]
+      requirementsPath = `${this.dockerServicePath}/${requirementsPath}`
     }
 
-    return this.runProcess(cmd,args)
+    args = [...args, upath.normalize(requirementsPath)]
+    return this.runProcess(cmd, args)
   }
 
   checkDocker(){
@@ -112,7 +115,7 @@ class PkgPyFuncs {
     } else {
       this.runProcess(
         'docker',
-        ['run', '--rm', '-dt', '-v', `${process.cwd()}:/var/task`,
+        ['run', '--rm', '-dt', '-v', `${process.cwd()}:${this.dockerServicePath}`,
           '--name',this.containerName, this.dockerImage, 'bash']
         )
       this.log('Container created')