commits/8517

Change 8517

matt_attaway (Matt Attaway)
Matt Attaway committed this change 11 years ago into //guest/matt_attaway/p4nodeapi
Request Review
Download .zip
Add a node.js language binding for Perforce

This is fairly simple extension for node to make it easier
to run Perforce commands from your node.js application. We've
used this a fair amount internally, but it hasn't seen the same level
of love as the other Perforce APIs.

This API is not supported by Perforce support, but you are welcome to post
on the forums if you have questions.
Add Job
0 edited 17 added 0 deleted
Tip: Use n and p to cycle through the changes.
gabor_maghera (Gabor Maghera)
Gabor Maghera commented 11 years ago

Thank you! 😌
Reply
matt_attaway (Matt Attaway)
Matt Attaway commented 11 years ago

My pleasure! I have a lot more stuff like it just laying around that I need to clean up and publish.
Reply
lester_cheung (Lester Cheung)
Lester Cheung commented 11 years ago

Awesome source!
Reply
lester_cheung (Lester Cheung)
Lester Cheung commented 11 years ago

Awesome sauce! :)
Reply
gabor_maghera (Gabor Maghera)
Gabor Maghera commented 11 years ago

FYI - there is a fairly simple third party implementation in the npm registry with the same name - it's a wrapper around the p4 command. Npm does not distinguish the two, so it sees this third-party implementation as a legit update to yours.

$ npm -q outdated p4
Package Current Wanted Latest Location
p4 0.0.2 0.0.4 0.0.4 p4
Reply
gabor_maghera (Gabor Maghera)
Gabor Maghera commented 11 years ago

Here's a link to that implementation in the npm registry: https://npmjs.org/package/p4
Reply
matt_attaway (Matt Attaway)
Matt Attaway commented 11 years ago

We can just rename ours to p4nodejs. Does that seem legit? (I used to manage one of our node experts, but I'm not one myself) 😄
Reply
gabor_maghera (Gabor Maghera)
Gabor Maghera commented 11 years ago

Sure, that sounds good. Publishing it to the npm registry with the new name might be a good idea. I'd be glad to assist if needed.
Reply