internalfx/quickshot

A Shopify theme development tool

Rick Davies

@rickydazla

04:28:14 pm - Error: EEXIST: file already exists, mkdir '/Users/Rick/Sites/myshopify/th
emes/taylorstitch/theme/assets'
=== START OF OUTPUT ===
{ Error: EEXIST: file already exists, mkdir '/Users/Rick/Sites/myshopify/themes/taylors

titch/theme/assets'
  cause:
   { Error: EEXIST: file already exists, mkdir '/Users/Rick/Sites/myshopify/themes/tayl
orstitch/theme/assets'
     errno: -17,
     code: 'EEXIST',
     syscall: 'mkdir',
     path: '/Users/Rick/Sites/myshopify/themes/taylorstitch/theme/assets' },
  isOperational: true,
  errno: -17,
  code: 'EEXIST',
  syscall: 'mkdir',
  path: '/Users/Rick/Sites/myshopify/themes/taylorstitch/theme/assets' }
==== END OF OUTPUT ====

Bryan Morris

@internalfx

hmm....I wonder if that is MAC specific?

I basically tell the OS to make the folder if it doesn't exist....

Rick Davies

@rickydazla

it does already exist

i’m (re)downloading an existing theme

to check if there are updates on the live site that were made by administrators and 3rd party apps

… which is why I wonder if I should be running qs theme watch --sync instead

Bryan Morris

@internalfx

no...download is what you want...

That is why I'm curious if this is something MAC does differently....my machine seems to silently ignore mkdir if the folder exists.

So it produces no error and writes the file.

Rick Davies

@rickydazla

oh oh i see

¯_(ツ)_/¯

Bryan Morris

@internalfx

give 3.0.7 a whirl

I wrapped the mkdir logic in a try lets see if it works now...

It may just be something I need to do to properly support MAC

Rick Davies

@rickydazla

whoah

04:39:34 pm - Downloaded templates/gift_card.liquid
{ Error: EEXIST: file already exists, mkdir '/Users/Rick/Sites/myshopify/themes/taylors
titch/theme/templates/customers'

  cause:
   { Error: EEXIST: file already exists, mkdir '/Users/Rick/Sites/myshopify/themes/tayl
orstitch/theme/templates/customers'
     errno: -17,
     code: 'EEXIST',
     syscall: 'mkdir',
     path: '/Users/Rick/Sites/myshopify/themes/taylorstitch/theme/templates/customers'
},
  isOperational: true,
  errno: -17,
  code: 'EEXIST',
  syscall: 'mkdir',
  path: '/Users/Rick/Sites/myshopify/themes/taylorstitch/theme/templates/customers' } '
the folder is already there'
04:39:34 pm - Downloaded templates/customers/register.liquid
{ Error: EEXIST: file already exists, mkdir '/Users/Rick/Sites/myshopify/themes/taylors
titch/theme/templates/customers'
  cause:
   { Error: EEXIST: file already exists, mkdir '/Users/Rick/Sites/myshopify/themes/tayl
orstitch/theme/templates/customers'
     errno: -17,
     code: 'EEXIST',
     syscall: 'mkdir',
     path: '/Users/Rick/Sites/myshopify/themes/taylorstitch/theme/templates/customers'
},
  isOperational: true,
  errno: -17,
  code: 'EEXIST',
  syscall: 'mkdir',
  path: '/Users/Rick/Sites/myshopify/themes/taylorstitch/theme/templates/customers' } '
the folder is already there'
04:39:34 pm - Downloaded templates/customers/reset_password.liquid

file error file error file error

Bryan Morris

@internalfx

Did it actually get the files?

Rick Davies

@rickydazla

yes

Bryan Morris

@internalfx

nice...I went ahead and kept logging the error so we could see it.

BOOM 3.0.8

I got a good feeling about this one.

it hides EEXIST errors when trying to create folders. We don't care if they already exists.

Rick Davies

@rickydazla

:boom:

thanks

Bryan Morris

@internalfx

Why linux is naturally OK with that and MAC isn't? i dunno

Appreciated the heads up and testing....

Not supporting MAC is like not supporting probably 90% of my userbase :laughing:

Rick Davies

@rickydazla

haha

Re: internalfx/quickshot#57

Bryan Morris

@internalfx

Right, so is your config still messed up?

Rick Davies

@rickydazla

No it’s fine… I figured out what changed and did search and replace. It did reject the old config, but I had like 16 targets and I know I’m not alone in that, so my question was really if there could be an automagic way (or even docs as to what changed)?

Bryan Morris

@internalfx

right...I basically turned 3 settings into one.

To make creating configs much less tedious

Rick Davies

@rickydazla

yeah, i like it

Bryan Morris

@internalfx

Now all you need is a name and the "example URL" given from the private app page.

I realized all the info I needed was in that URL so I parsed the data out of it.

I bet you noticed quickshot3 now pretty formats the quickshot.json :smile:

no super long string of JSON

Rick Davies

@rickydazla

yes, that’s great actually

b/c i usually create new targets by editing that directly but i had to beautify it each time

Bryan Morris

@internalfx

@rickydazla there is now a breaking changes section....

https://github.com/internalfx/quickshot/#migrating-from-2x-breaking-changes

Rick Davies

@rickydazla

now we’re cookin on gas!

Trajce

@trajce

I just checked this thread, so qs v3 is official :D

Ill give it a try on windows 10 and ping you back if i got any specific notes

Bryan Morris

@internalfx

@trajce Please Do! I work and develop on Linux so it's a big help when people let me know problems they find on other platforms

Trajce

@trajce

It seems like its working fine, no issues so far on windows.

Where communities thrive

People

Repo info

Activity