This looks good to me ( pending the www -> lang change )
On Mon, May 21, 2012 at 11:20 PM, Jeremy Evans <[email protected]> wrote:
> This updates node-fibers to 0.6.7, the latest version. Doing so
> requires some patches to node.
>
> www/node patches:
>
> * Move npm dependencies (including node-gyp) from being private
> dependencies of npm to being globally installed modules, by moving the
> files to ${LOCALBASE}/lib/node_modules and then symlinking them into
> npm's dependencies. If node-gyp is left as a private dependency,
> you can't set node-gyp in MODNODE_DEPENDS. For something that
> requires node-gyp as a dependency (e.g. node-fibers), you have to
> either disable systrace and have npm install node-gyp over the net,
> or use this hack. Unless we want to ship a separate node-gyp package,
> I think we should use this symlink hack, at least for node-gyp. This
> patch does it for all of npm's dependencies, since I don't really see
> the downside.
>
> * Don't try to chmod 0755 a file that already has mode 0755. This fixes
> a bug when building node-fibers, where npm would call chmod as a regular
> user on a file owned by root, and get EPERM.
>
> * Patch node-gyp to recognize openbsd. Without this, node-gyp assumes
> you are on linux, which breaks node-fibers as node-fibers uses ucontext
> on linux (and that doesn't work on OpenBSD).
>
> devel/node-fibers patches:
>
> * Upgrade to 0.6.7, which uses node-gyp. This eliminates the need for
> some local patches.
>
> * Remove the explicit dependency on node-gyp, as it makes the build
> fail for reasons I have yet to determine.
>
> * Change the path of the cached node-gyp.js binary, as it is does not
> work correctly with our use of npm link.
>
> I haven't tested this enough, it is not yet ready for commit. I'm
> sending this early to get feedback from other porters as to whether
> these changes make sense.
>
> Thanks,
> Jeremy
>
