Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

lookup: add node-gyp #370

Closed
wants to merge 1 commit into from
Closed

Conversation

richardlau
Copy link
Member

Checklist
  • commit message follows commit guidelines

Add node-gyp to lookup.json. I believe it meets all the hard requirements for inclusion.

@richardlau
Copy link
Member Author

-bash-4.2$ bin/citgm.js node-gyp
info:    starting            | node-gyp
warn:    update-available    | v1.11.2 (current: v1.11.1)
warn:                        | npm install -g citgm
info:    lookup              | node-gyp
info:    lookup-found        | node-gyp
info:    node-gyp lookup-replace| https://github.com/nodejs/node-gyp/archive/v3.5.0.tar.gz
info:    node-gyp npm:       | Downloading project: https://github.com/nodejs/node-gyp/archive/v3.5.0.tar.gz
info:    node-gyp npm:       | Project downloaded node-gyp-3.5.0.tgz
info:    node-gyp npm:       | npm install started
info:    node-gyp npm:       | npm install successfully completed
info:    node-gyp npm:       | test suite started
info:    passing module(s)   |
info:    module name:        | node-gyp
info:    version:            | 3.5.0
info:    done                | The smoke test has passed.
info:    duration            | test duration: 23547ms
-bash-4.2$

gdams

This comment was marked as off-topic.

MylesBorins

This comment was marked as off-topic.

@richardlau
Copy link
Member Author

Related: nodejs/testing#15

@richardlau
Copy link
Member Author

Looking into AIX failures:

e.g. https://ci.nodejs.org/view/Node.js-citgm/job/gibfahn-citgm-smoker-more-platforms/118/MACHINE=aix61-ppc64/console

error:                     | > node-gyp@3.5.0 test /home/iojs/build/workspace/gibfahn-citgm-smoker-more-platforms/MACHINE/aix61-ppc64/citgm_tmp/3b837526-ee7e-4adc-b864-e7eb3c47f327/node-gyp                                   
error:                     | > tape test/test-*                                                                                                                                                                                 
error:                     |                                                                                                                                                                                                    
error:                     | TAP version 13                                                                                                                                                                                     
error:                     | # build simple addon                                                                                                                                                                               
error:                     | ok 1 should be equal                                                                                                                                                                               
error:                     | ok 2 should end in ok                                                                                                                                                                              
error:                     | ok 3 should be equal                                                                                                                                                                               
error:                     | # configure PYTHONPATH with no existing env                                                                                                                                                        
error:                     |                                                                                                                                                                                                    
error:                     | gyp ERR! find exports file Could not find exports file                                                                                                                                             
error:                     | /home/iojs/build/workspace/gibfahn-citgm-smoker-more-platforms/MACHINE/aix61-ppc64/citgm_tmp/3b837526-ee7e-4adc-b864-e7eb3c47f327/node-gyp/lib/configure.js:245                                    
error:                     | return callback(new Error(msg))                                                                                                                                                                    
error:                     | ^                                                                                                                                                                                                  
error:                     |                                                                                                                                                                                                    
error:                     | TypeError: callback is not a function                                                                                                                                                              
error:                     | at runGyp (/home/iojs/build/workspace/gibfahn-citgm-smoker-more-platforms/MACHINE/aix61-ppc64/citgm_tmp/3b837526-ee7e-4adc-b864-e7eb3c47f327/node-gyp/lib/configure.js:245:16)                     
error:                     | at findConfigs (/home/iojs/build/workspace/gibfahn-citgm-smoker-more-platforms/MACHINE/aix61-ppc64/citgm_tmp/3b837526-ee7e-4adc-b864-e7eb3c47f327/node-gyp/lib/configure.js:170:23)                
...

@richardlau
Copy link
Member Author

It's a problem with one of the node-gyp tests (test/test-configure-python.js) on AIX. I'll upstream a fix.

@gdams
Copy link
Member

gdams commented Feb 27, 2017

Sgtm

@richardlau
Copy link
Member Author

richardlau commented Feb 27, 2017

Raised nodejs/node-gyp#1131. It's unfortunate this test has never passed on AIX, but hey all the more reason to add node-gyp to CITGM 😆

@gdams
Copy link
Member

gdams commented Apr 5, 2017

@richardlau any progress with this?

@richardlau
Copy link
Member Author

Still waiting for nodejs/node-gyp#1131 to land.

@gdams
Copy link
Member

gdams commented Apr 18, 2017

@nodejs-github-bot run CI

@nodejs-github-bot
Copy link
Collaborator

@richardlau
Copy link
Member Author

nodejs/node-gyp#1131 has landed in node-gyp@3.6.1 so I think this is unblocked now.

I've rebased onto the current master and added a native tag. PTAL.

@richardlau
Copy link
Member Author

@nodejs-github-bot run CI

@nodejs-github-bot
Copy link
Collaborator

@richardlau sadly an error occured when I tried to trigger a build :(

@MylesBorins
Copy link
Member

landed in 92f1d6e

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

Successfully merging this pull request may close these issues.

None yet

4 participants