Skip to content
This repository has been archived by the owner. It is now read-only.
Permalink
Browse files

tools: remove "addon.gypi" and "gyp_addon"

They've been merged into `node-gyp` and aren't serving any purpose
in the node repo. Fixes #3760.
  • Loading branch information
TooTallNate committed Aug 12, 2012
1 parent 320ae69 commit 00fa8864b4b8ff0650e9c82a2ee1f5e87f452c4a
Showing with 0 additions and 66 deletions.
  1. +0 −24 tools/addon.gypi
  2. +0 −42 tools/gyp_addon

This file was deleted.

This file was deleted.

6 comments on commit 00fa886

@ry

This comment has been minimized.

Copy link

ry replied Aug 13, 2012

:)

@jbergstroem

This comment has been minimized.

Copy link
Member

jbergstroem replied Aug 13, 2012

Fyi, this breaks the buildbots: http://buildbot.nodejs.org/builders/Gentoo%2032bit/builds/2426/steps/shell_2/logs/stdio. Probably need to update the build receipt?

@TooTallNate

This comment has been minimized.

Copy link
Author

TooTallNate replied Aug 13, 2012

@jbergstroem That was more or less a bug in node-gyp. v0.6.5 fixed it.

@jbergstroem

This comment has been minimized.

Copy link
Member

jbergstroem replied Aug 13, 2012

@TooTallNate Ok, but why is it still happening (that build was against commit 22804a9)?

@TooTallNate

This comment has been minimized.

Copy link
Author

TooTallNate replied Aug 13, 2012

I guess because @isaacs hasn't updated the bundled version of node-gyp inside npm to that version yet.

@jbergstroem

This comment has been minimized.

Copy link
Member

jbergstroem replied Aug 13, 2012

Figures, thanks. I pinged him on irc an hour ago, lets see what happens :-)

Please sign in to comment.
You can’t perform that action at this time.