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

systemv.js change to allow to work with amazon AWS servers #33

Closed
wants to merge 2 commits into from
Closed

systemv.js change to allow to work with amazon AWS servers #33

wants to merge 2 commits into from

Conversation

JohnNelsonAPPX
Copy link

No description provided.

This updated file allows node-linux to work with Amazon AWS servers.
@tacooper
Copy link

Note that this is only required until the underlying issue #31 is fixed.

@coreybutler
Copy link
Owner

There are already checks for amzn. @tacooper is right though... #31 needs to be resolved. If someone submits a PR against #31, I would merge it. I just don't have time to address it personally at the moment.

@JohnNelsonAPPX
Copy link
Author

My apologies. I actually fixed this 6 months ago, but am new to git. Was revisiting it when I saw I didn't submit the pull request to you like I thought I did, but instead of checking the new code first, I just went ahead and submitted the pull request.

@coreybutler
Copy link
Owner

No worries... I'm happy people are committing!

@tacooper
Copy link

tacooper commented Feb 3, 2017

@coreybutler I submitted a PR (#38), let me know when you get a chance to review it :)

@tacooper
Copy link

tacooper commented Jul 7, 2017

@coreybutler any idea of when you'll have time to review/merge PR #38? thanks

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

Successfully merging this pull request may close these issues.

3 participants