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

Support Debian package build on aarch64 architecture #232

Closed
wants to merge 2 commits into from

Conversation

Leden
Copy link
Contributor

@Leden Leden commented May 12, 2016

This adds the code recognizing aarch64 architecture in debian/rules.

@dnobori
Copy link
Member

dnobori commented May 26, 2016

Your great patch is much appreciated. We are considering to apply your patch into the SoftEther VPN main tree.

SoftEther VPN Patch Acceptance Policy:
http://www.softether.org/5-download/src/9.patch

You have two options which are described on the above policy.
Could you please choose either option 1 or 2, and specify it clearly on the reply?

@Leden
Copy link
Contributor Author

Leden commented May 26, 2016

Sure, I choose 1st option. I allow to apply the same patch into PacketiX VPN.

@lingqingfeng
Copy link

Hi, I am a newcomer.I want to study the Softether VPN. I have read the source code on windows,but It's so complex.Can I apply for a program instruction?Thanks.

@paskal
Copy link
Contributor

paskal commented Nov 27, 2016

@Leden, I'm afraid you have to fix something after latest merges.

@dnobori dnobori mentioned this pull request Nov 27, 2016
@dnobori
Copy link
Member

dnobori commented Nov 27, 2016

Thank you so much for your contribution to enrich the SoftEther VPN source code.

Your patch has been merged on the main source-tree of SoftEther VPN.

As a token of our gratitude, your GitHub username has been added on the AUTHORS.TXT file and on the header of the related source file.
Please see: https://github.com/SoftEtherVPN/SoftEtherVPN/blob/master/AUTHORS.TXT

Thanks again for your contribution.

@Leden
Copy link
Contributor Author

Leden commented Nov 27, 2016

@paskal sure, i'll look into it as soon as i get a free minute.

@paskal
Copy link
Contributor

paskal commented Nov 27, 2016

@dnobori, you've mentioned this pull request as merged but didn't merged it actually.

@Leden
Copy link
Contributor Author

Leden commented Nov 27, 2016

@paskal, it is merged, in fact. The code itself seems to be copied onto the master branch manually. It's fine, this PR is not needed anymore. I'll close it.

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

Successfully merging this pull request may close these issues.

None yet

5 participants