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

Ask for integration #4

Closed
flashultra opened this issue Jul 6, 2022 · 8 comments
Closed

Ask for integration #4

flashultra opened this issue Jul 6, 2022 · 8 comments

Comments

@flashultra
Copy link

Hi Chuck!
I saw your library hxBitcoin when I was looking for BigInteger type for Haxe and I did some test against other BigInteger implementation.
Your library is the best one from the performance stand point. It's faster either from native Java implementation too!

Incredible work!

I want to add your BigInt implementation in Haxe standart library, but Haxe use MIT license . Your library is licensed under the custom license .

Would it be possible for me to suggest your BigInt code under the Haxe MIT license ( standart one use for all Haxe code) ?

If you agree, I can push a commit from my account or If you want, I can prepeare commit and send to you , so you can suggest a commit under your account (so your name will be as committer).

As I say, your implementation is supperior and I think BigInteger class will boost our small Haxe ecosystem.

Thank you.

@cbatson
Copy link
Owner

cbatson commented Jul 6, 2022 via email

@flashultra
Copy link
Author

Thank you very much Chuck !
Really appreciate your approve to relicense BigInt integration in Haxe Std.
I will update you for the status after creating a pull request.

Thank you again.

@flashultra
Copy link
Author

Hi Chuck!
I want to inform you that I created a pull request HaxeFoundation/haxe#10750 with your implementation.
Thank you so much again !
I saw you have some very good implementation of many crypto functions . Would be possible to use and relicense those functions in Haxe crypto lib via a Haxe MIT license ?

@flashultra
Copy link
Author

@cbatson
I finished the PR. It was really very easy to work with your code !
In summary, I did some fixes for Python and add new methods pow, modPow, isPrime . Unfortunately my modPow implementation is very slow, but it works.

I want to thank you again for your contribution. You have done a tremendous amount of work and I am sure many people will benefit from it after the PR merge

I'm thinking of adding some of yours crypto/elliptic methods to the Haxe crypto lib ( https://github.com/haxefoundation/crypto ).

Could you confirm that I have your permission to do so and relicense it under the MIT licence ?

@cbatson
Copy link
Owner

cbatson commented Jul 31, 2022

Hello @flashultra. Thank you for all your work on HaxeFoundation/haxe#10750 to get the tests passing on other platforms. I have not been paying too close attention to that PR, but please make sure any new methods have sufficient test coverage. I also made a comment on that PR requesting credit as the author if possible.

Regarding the crypto/elliptic work, will it be possible to be credited in the source as the original author? If so then I agree to the inclusion and license change. Thank you.

@flashultra
Copy link
Author

Hi @cbatson .
I added tests for all the new methods, which gave me the opportunity to find and fix some edge cases for other Haxe targets.

About the credit as the author, I can't speak for the main Haxe repository as I'm not a main contributor there and I don't have merge rights , but I do have a such rights for a crypto repository .

I think it would be no problem to add you as a author it in the crypto repository , but since it is under the Haxe wing I should ask before do it. So before I start with any work of such kind of implementation I will let you know

Thank you again for allowing me to contribute you work to Haxe foundation!

@flashultra
Copy link
Author

Hi @cbatson !
I just added credited in the source to you as the original author . Thank you again !
About crypto , could I use same credit in the source to you as the original author , when implement yours crypto/elliptic work and relicense it under the Haxe MIT license ?

@cbatson
Copy link
Owner

cbatson commented Aug 1, 2022

Yes that is perfect and fine with me. Thank you!

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

No branches or pull requests

2 participants