👍 Looks good. @garyb if I merge this, do you think you'd be able to test it, please?

I'd rather look for bower.cmd only if I'm on windows platform. If I have an executable bower.cmd in my path, I'd be surprised if it was executed.

Sure, I can test it now actually.

Well it didn't fail because bower was missing, but I just uploaded this: http://pursuit.purescript.org/packages/purescript-maybe/0.3.3 (no modules listed).

I think @michaelficarra has a point too, adding a platform check should be easy enough I think?

Yeah, the platform check is just System.Info.os == "mingw32" I think.

So you tried running psc-publish using this branch on Windows, with Bower installed, but when you uploaded it, there were no modules?

Also, @garyb, would you mind running it again but posting the output as a gist or something?

Sure thing:

{"uploader":null,"packageMeta":{"homepage":"https://github.com/purescript/purescript-maybe","repository":{"url":"git://github.com/purescript/purescript-maybe.git","type":"git"},"ignore":["**/.*","bower_components","node_modules","output","test","bower.json","gulpfile.js","package.json"],"dependencies":{"purescript-monoid":"^0.3.0"},"name":"purescript-maybe","keywords":["purescript"],"description":"Optional values"},"modules":[],"resolvedDependencies":{"purescript-control":"0.3.0","purescript-invariant":"0.3.0","purescript-monoid":"0.3.0","purescript-prelude":"0.1.1"},"bookmarks":[],"version":"0.3.3","github":["purescript","purescript-maybe"],"versionTag":"v0.3.3","compilerVersion":"0.7.1.0"}

Is this good to merge now?

No, I still need to check the platform before trying bower.cmd.

No worries, just checking. Thanks.

Ok I think this is good to go now. @garyb, would you mind testing this on Windows please?

👍