Skip to content
This repository has been archived by the owner on Jul 19, 2021. It is now read-only.

Cannot find ./upload folder generated during manual deployment #249

Closed
bmcminn opened this issue Aug 26, 2017 · 7 comments · Fixed by #318
Closed

Cannot find ./upload folder generated during manual deployment #249

bmcminn opened this issue Aug 26, 2017 · 7 comments · Fixed by #318
Labels
Milestone

Comments

@bmcminn
Copy link

bmcminn commented Aug 26, 2017

Problem

Following the instructions on the Shopify Slate page: https://shopify.github.io/slate/

Under "Connect your project with your Shopify store" in task 2, I run slate deploy -m, which results in the following error where the upload/ folder cannot be found, even though it very well exists.

...
...
[02:29:39] all files 158.29 kB
[02:29:42] Finished 'build:svg' after 7.25 s
[02:29:42] Finished 'build:assets' after 7.71 s
[02:29:42] Finished 'build:zip' after 8.56 s
[02:29:42] Starting 'compress'...
[02:29:42] mk-studio-theme.zip 133.15 kB
[02:29:42] Finished 'compress' after 291 ms
[02:29:42] Starting 'output:errors'...
[02:29:42] Finished 'output:errors' after 134 μs
[02:29:42] Finished 'zip' after 8.86 s
[02:29:42] Starting 'open:admin'...
[02:29:42] Finished 'open:admin' after 127 ms
[02:29:42] Starting 'open:zip'...
[02:29:59] 'open:zip' errored after 17 s
[02:29:59] Error: Command failed: start "" "./upload/"
The system cannot find the file ./upload/.

    at ChildProcess.exithandler (child_process.js:270:12)
    at emitTwo (events.js:125:13)
    at ChildProcess.emit (events.js:213:7)
    at maybeClose (internal/child_process.js:897:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:208:5)
[02:29:59] 'deploy:manual' errored after 26 s
[02:29:59] Error in plugin 'run-sequence(open:zip)'
Message:
    Command failed: start "" "./upload/"
The system cannot find the file ./upload/.

Details:
    cause: Error: Command failed: start "" "./upload/"
The system cannot find the file ./upload/.

    isOperational: true
    killed: false
    code: 1
    signal: null
    cmd: start "" "./upload/"
Stack:
Error: Command failed: start "" "./upload/"
The system cannot find the file ./upload/.

    at ChildProcess.exithandler (child_process.js:270:12)
    at emitTwo (events.js:125:13)
    at ChildProcess.emit (events.js:213:7)
    at maybeClose (internal/child_process.js:897:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:208:5)

Replication steps

  1. Install latest Slate module (v0.12.1): npm i -g @shopify/slate
  2. Follow the Shopify Slate guide at: https://shopify.github.io/slate/

More Information

OS: Windows 7 Pro x64
Node: v8.1.4
NPM: 5.0.3


Theme folder organization, post deploy compilation step:

- my-theme-folder
    - dist/
    - src/
    - upload/    <-- it's there, but not where deploy is expecting it I guess
    - .gitignore
    - config.yml
    - package-lock.json
    - package.json
@luisGithubPeru
Copy link

Same error here, with these specifications:

OS: Windows 7 Home Premium SP1
Slate: 0.12.1
Node: v6.11.2
NPM: 3.10.10

@t-kelly t-kelly added the bug label Sep 13, 2017
@t-kelly t-kelly added this to the 0.13.0 milestone Sep 13, 2017
@Aditya94A
Copy link

Any updates? Or workarounds?

@t-kelly
Copy link
Contributor

t-kelly commented Sep 18, 2017

We should be exploring this soon for v0.13.0. Will keep you posted.

@targetcreature
Copy link

Slate appears to have still created the zip file in an "upload" folder.

Is it safe to continue to step 3 (upload zip file) despite the error?

@jameshan425
Copy link

I installed node.js and slate today and got the same error.
But I found the upload directory.
Can I use it though?

OS: Windows 10 Pro 10.0.15063 Build 15063
Slate: 0.12.1
Node: v6.11.4
NPM: 3.10.10

@maximevaillancourt
Copy link
Contributor

maximevaillancourt commented Oct 10, 2017

Thanks everyone for reporting this. We were able to replicate the issue — a fix is on the way (see #318). In the meantime, you should be able to use the zip file generated by Slate in the upload folder without any issue.

@lock
Copy link

lock bot commented Oct 27, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Oct 27, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants