atom / chocolatey

chocolatey installer for Atom
MIT License
12 stars 17 forks source link

Potential AppVeyor CI issue? #36

Closed jrsconfitto closed 9 years ago

jrsconfitto commented 9 years ago

i'm getting AppVeyor build failures in a PR for my language grammar: jugglingnutcase/language-powershell#33. You can see the full build log for one of the failing commits in this link, but here's a relevant snippet:

cd %APPVEYOR_BUILD_FOLDER%
apm install
'apm' is not recognized as an internal or external command,
operable program or batch file.
Command exited with code 1

i have experienced a number of these failures and have yet to find one that has failed within the past few days for any other reason. i would expect that apm would be found after a chocolately install, especially because the appveyor file is up-to-date with what atom/ci suggests.

It was working until recently, and the only other package in the atom org that i could find atom/fuzzaldrin using an Appveyor.yml file doesn't have any recent builds.

i'm not sure if this is the right place to report this, i checked atom/ci, but i think that this issue may be more relevant here.

If i can be more helpful, please let me know. i've made a couple attempts to track down why this is happening, but they've been fruitless.

kevinsawicki commented 9 years ago

Would you mind trying something for me?

Can you try using this appveyor.yml and see if it works?

I was testing out a fix on that package but for some reason my build is just showing as infinitely queued, https://ci.appveyor.com/project/kevinsawicki/wrap-guide/build/3

jrsconfitto commented 9 years ago

Yeah, I don't mind doing that. I'll let you know how it goes.

I noticed you had infinite builds too, queued up for months IIRC! Weird.

On Jan 27, 2015, at 5:27 PM, Kevin Sawicki notifications@github.com wrote:

Would you mind trying something for me?

Can you try using this appveyor.yml and see if it works?

I was testing out a fix on that package but for some reason my build is just showing as infinitely queued, https://ci.appveyor.com/project/kevinsawicki/wrap-guide/build/3

— Reply to this email directly or view it on GitHub.

jrsconfitto commented 9 years ago

Looks much better, here's a successful build with your suggested appveyor.yml file.

kevinsawicki commented 9 years ago

Okay, 0.177 came out today with some apm test fixes and so if you use the new template from https://github.com/atom/ci/blob/master/appveyor.yml things should properly build/test now.

jrsconfitto commented 9 years ago

Thanks! :+1: