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

Remove apps.vpn #1008

Merged
merged 1 commit into from Oct 7, 2016
Merged

Remove apps.vpn #1008

merged 1 commit into from Oct 7, 2016

Conversation

alexandergall
Copy link
Contributor

Remove dead code, obsoleted by the l2vpn program on the l2vpn branch.

Remove dead code, obsoleted by the l2vpn program on the l2vpn branch.
@eugeneia eugeneia self-assigned this Sep 2, 2016
@eugeneia
Copy link
Member

eugeneia commented Sep 2, 2016

I would prefer to merge this once the replacement lands upstream. Might not make a difference, but I don't know who is using apps.vpn?

@alexandergall
Copy link
Contributor Author

The new VPN code will remain on the l2vpn topic branch (in that sense, the replacement is already there and has been so for a long time). You can phase out the old vpws code whenever you like.

eugeneia added a commit to eugeneia/snabb that referenced this pull request Sep 5, 2016
# Conflicts:
#	src/apps/vpn/vpws.lua
@eugeneia eugeneia added the merged label Sep 5, 2016
eugeneia added a commit to eugeneia/snabb that referenced this pull request Sep 9, 2016
@eugeneia eugeneia merged commit fb15c66 into snabbco:master Oct 7, 2016
dpino pushed a commit to dpino/snabb that referenced this pull request Dec 20, 2017
@alexandergall alexandergall deleted the remove-vpws branch February 14, 2018 11:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants