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

Pull request for issue #1 #4

Merged
merged 4 commits into from Nov 22, 2017

Conversation

3 participants
@HoochDeveloper
Member

HoochDeveloper commented Oct 8, 2017

Issue: #1

Samples for reading NIS in C++, Java, Objective-C

@HoochDeveloper

This comment has been minimized.

Show comment
Hide comment
@HoochDeveloper

HoochDeveloper Oct 8, 2017

Owner

This code is for MacOSX

Owner

HoochDeveloper commented on 8b5318b Oct 8, 2017

This code is for MacOSX

@alexrj alexrj merged commit ef98202 into italia:master Nov 22, 2017

@alexrj

This comment has been minimized.

Show comment
Hide comment
@alexrj

alexrj Nov 22, 2017

Member

Hello!

I initially merged your pull request because I thought about having a single cie-nis-sdk repository with your code and the Python one (from the other pull request) but I realized this would prevent us from having TravisCI running.
So I'm going to create additional individual repositories for your code: cie-nis-java-sdk, cie-nis-objc-sdk, cie-nis-cpp-sdk.

I have one question before proceeding: why did you relicense the repository from BSD-3 (our preferred license) to Apache?

Member

alexrj commented Nov 22, 2017

Hello!

I initially merged your pull request because I thought about having a single cie-nis-sdk repository with your code and the Python one (from the other pull request) but I realized this would prevent us from having TravisCI running.
So I'm going to create additional individual repositories for your code: cie-nis-java-sdk, cie-nis-objc-sdk, cie-nis-cpp-sdk.

I have one question before proceeding: why did you relicense the repository from BSD-3 (our preferred license) to Apache?

@duncanwilcox

This comment has been minimized.

Show comment
Hide comment
@duncanwilcox

duncanwilcox Nov 22, 2017

The MacOS code is derivative of https://github.com/dirkx/CryptoTokenKit-TrivialExample-OpenSC which is Apache licensed.

duncanwilcox commented Nov 22, 2017

The MacOS code is derivative of https://github.com/dirkx/CryptoTokenKit-TrivialExample-OpenSC which is Apache licensed.

@alexrj

This comment has been minimized.

Show comment
Hide comment
@alexrj

alexrj Nov 23, 2017

Member

I just created three new repositories for your code and pulled it preserving history:
https://github.com/italia/cie-nis-cpp-sdk
https://github.com/italia/cie-nis-java-sdk
https://github.com/italia/cie-nis-objc-sdk

Thank you for your contribution, and welcome to the community of Developers Italia as maintainers of the above repositories, on which you now have full admin rights 😃
(Remember: with great power comes great responsibility.)

I also invited you to become part of the italia organization, so that you can display its logo on your profile (don't forget to change the visibility after you accept the invitation!).

Member

alexrj commented Nov 23, 2017

I just created three new repositories for your code and pulled it preserving history:
https://github.com/italia/cie-nis-cpp-sdk
https://github.com/italia/cie-nis-java-sdk
https://github.com/italia/cie-nis-objc-sdk

Thank you for your contribution, and welcome to the community of Developers Italia as maintainers of the above repositories, on which you now have full admin rights 😃
(Remember: with great power comes great responsibility.)

I also invited you to become part of the italia organization, so that you can display its logo on your profile (don't forget to change the visibility after you accept the invitation!).

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