Skip to content
This repository has been archived by the owner on Apr 24, 2020. It is now read-only.

Fix Appveyor builds #562

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix Appveyor builds #562

wants to merge 2 commits into from

Conversation

lgeiger
Copy link
Contributor

@lgeiger lgeiger commented Sep 27, 2016

With the default npmversion, the builds only succeed on node 4.

This issue was fixed in npm: nodejs/node-gyp#972

@lgeiger
Copy link
Contributor Author

lgeiger commented Sep 27, 2016

Everything green. You just need to activate appveyor again:
bildschirmfoto 2016-09-27 um 20 35 23

@ronkorving
Copy link
Collaborator

Activate?

@lgeiger
Copy link
Contributor Author

lgeiger commented Sep 28, 2016

Activate?

Pull requests and Pushes aren't build on appveyor right now. The last build, according to the readme batch, happend 10 months ago: https://ci.appveyor.com/project/reqshark/zeromq-node

@ronkorving
Copy link
Collaborator

You're right. Problem is, I don't have access to this repos settings, @JustinTulloss does. I'll ask again (it's been a while), @JustinTulloss, can I have administrative access on this repo so I can setup hooks for services like Appveyor? Thanks in advance.

@lgeiger
Copy link
Contributor Author

lgeiger commented Sep 28, 2016

I don't know if you need admin access for this or not. Based on the URL @reqshark setup appveypr last time.

@reqshark
Copy link
Contributor

oh ya @lgeiger, that's right, let me know if you guys want something done from my side, or even better, anyone can make a new one and use that appveyor to reflect the latest build status ;)

@lgeiger
Copy link
Contributor Author

lgeiger commented Sep 28, 2016

I can't make a new one, because it looks like you need push access to the repo.
@reqshark If you login to your appveyor account and add/reactivate this repo everything should work again.

@reqshark
Copy link
Contributor

check your email, i added you as collaborator admin, let me know if you need to be user admin

@reqshark
Copy link
Contributor

On appveyor

@lgeiger
Copy link
Contributor Author

lgeiger commented Sep 28, 2016

I changed the tracking repository. The build on master works. I'll close and reopen to trigger a new PR build.

@lgeiger lgeiger closed this Sep 28, 2016
@lgeiger lgeiger reopened this Sep 28, 2016
@lgeiger
Copy link
Contributor Author

lgeiger commented Sep 28, 2016

@reqshark Thanks for your help. The webhook still doesn't work. Could it be that you don't have push access to this repository anymore?

@reqshark
Copy link
Contributor

Yes

@lgeiger
Copy link
Contributor Author

lgeiger commented Sep 28, 2016

I see.

@ronkorving, @JustinTulloss To set it up you just have to login to https://ci.appveyor.com/ with your github account and add this repo as a project.

@ronkorving
Copy link
Collaborator

Added

@lgeiger
Copy link
Contributor Author

lgeiger commented Sep 29, 2016

I'm sorry to bug you again but the webhooks are still not working 😕

@lgeiger lgeiger mentioned this pull request Sep 29, 2016
@ronkorving
Copy link
Collaborator

So, I went to that URL, added the project. But indeed, it's under my name. Not under the name of the owner of this repo, which is @JustinTulloss.

@lgeiger
Copy link
Contributor Author

lgeiger commented Sep 29, 2016

I was sure that you'll find JustinTulloss/zeromq.node in that project list. But I'm not very familiar with the appveyor permissions so I might be wrong.

@ronkorving
Copy link
Collaborator

I find it in the list indeed. I did that before. The URL reflected my username, not @JustinTulloss though (but it seems that's normal). I manually started a build, and the latest commit message does seem to reflect the latest on @JustinTulloss (my own fork is out-of-date at this point). But has it set up the hooks? I doubt it.

@rgbkrk
Copy link

rgbkrk commented Oct 31, 2016

We ended up bringing our fork of zeromq.node (which was originally in github.com/nteract) over to the zeromq org at https://github.com/zeromq/zeromq.js so we can move forward with making builds easier. Would you like to join the group @ronkorving?

@ronkorving
Copy link
Collaborator

ronkorving commented Nov 7, 2016

Hey, that sounds great. Let's bring these together. Also, please have a look at my #516 branch and let me know (in that PR) what you think.

How far has your fork drifted from this repo?

@lgeiger
Copy link
Contributor Author

lgeiger commented Nov 7, 2016

@ronkorving Great to have you on board! 🎉

The fork of the actual lib hasn't drifted at all. We tried to keep the delta at a minimum.
The changes are mostly regarding installation and deployment of prebuilt binaries.

#516 looks very promising but is well out of my expertise. I'm the guy that only solves build issues 😉 😂

@ronkorving
Copy link
Collaborator

Understood :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants