ThoughtWorksInc / lein-s3-static-deploy

Lein task to deploy static website to s3 bucket.
Eclipse Public License 1.0
17 stars 4 forks source link

Fails because of Windows path separator #2

Open jensontech opened 9 years ago

jensontech commented 9 years ago

When I run this on Windows I get - "Deploying ./resources/public/compiled_prod/ to .... java.util.regex.PatternSyntaxException: Illegal/unsupported escape sequence near...." The path displayed has backslashes. Can you change it so it It converts Windows paths to forward slashes?

uday-rayala commented 9 years ago

I think the deploy location is something you give in the project.clj. We dont build any paths in the library.

Please make sure the :local-root parameter has paths compliant to windows. :aws {:access-key ~(System/getenv "AWS_ACCESS_KEY")
:secret-key ~(System/getenv "AWS_SECRET_KEY")
:s3-static-deploy {:bucket "THE BUCKET YOU WANT TO DEPLOY TO"
:local-root "LOCAL DIRECTORY YOU WANT TO DEPLOY FROM"}}

If you are still facing the issue, please share your project.clj.

arosequist commented 9 years ago

I think this might be caused by an issue in s3-sync. Upgrading to 0.4.0 might resolve this.

petergillardmoss commented 9 years ago

I've upgraded s3-sync and published as 0.1.1-SNAPSHOT: https://clojars.org/lein-s3-static-deploy/versions/0.1.1-SNAPSHOT

Unfortunately I don't have access to a Windows box so I am unable to verify. Would you do the honours and then I shall publish 0.1.1.

Thanks

arosequist commented 9 years ago

0.1.1-SNAPSHOT is working for me on Win 7.

jensontech commented 9 years ago
Thanks Anthony.

  I am a bit snowed under at the moment, but this is getting closer
  to the top of the list. I'm sure you know how it gets. Users'
  demands first.

  On 25/02/2015 4:24 PM, Anthony Rosequist wrote:

  0.1.1-SNAPSHOT is working for me on Win 7.
  —
    Reply to this email directly or view
      it on GitHub.
jensontech commented 9 years ago
Hi Peter.

  I have finally had time to retest this. It works fine in Windows
  now.

  It doesn't deploy our static files (which we deploy to the same
  location). When I get time I will do a pull request.

  Thanks,
  jensontech

  On 24/02/2015 2:27 AM, Peter Gillard-Moss wrote:

  I've upgraded s3-sync and published as 0.1.1-SNAPSHOT: https://clojars.org/lein-s3-static-deploy/versions/0.1.1-SNAPSHOT
  Unfortunately I don't have access to a Windows box so I am
    unable to verify. Would you do the honours and then I shall
    publish 0.1.1.
  Thanks
  —
    Reply to this email directly or view
      it on GitHub.