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.

Sign up for GitHub

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

Jump to bottom

Release v0.4.0 #188

Merged
merged 4 commits into from
Aug 19, 2023
Merged

Release v0.4.0 #188

merged 4 commits into from
Aug 19, 2023

Conversation

pmelange
Copy link
Contributor

Update the changelog
Bump the version to 0.4.0
Update the list of supported distros to include OpenWrt

@pmelange
Copy link
Contributor Author

setup.py will need to be changed again to get it ready for 0.4.1-dev1 (or 0.5.0-dev1). Let me know what is preferred.

@RalfJung @mitar

@mitar
Copy link
Member

mitar commented Aug 17, 2023

Let's go with 0.4.1-dev1 after the release and we can always make a bigger version increment afterwards, I think.

Thanks for making it!

@pmelange
Copy link
Contributor Author

If it's looking good enough for the release, then could someone merge and then also tag? Then I will prepare another PR for 0.4.1-dev1

@RalfJung
Copy link
Member

Please wait for me to take a look at the changelog, I should get to it over the weekend. It's been years since the last release, no need to rush. :)

RalfJung
RalfJung reviewed Aug 18, 2023
View reviewed changes
Copy link
Member

@RalfJung RalfJung left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for going through all the changes! I just have some editorial suggestions.

HISTORY.rst Outdated Show resolved Hide resolved
HISTORY.rst Outdated Show resolved Hide resolved
HISTORY.rst Outdated
* Python 3 support.
Python 2 is no longer supported.
**NOTE:** If you are running the broker in a Python virtualenv you will have to
rebuild the virtualenv using a Python 3 interpreter.
This can be achieved by deleting the old virtualenv folder and recreating it.
For recreating the virtualenv you can refer to the "Installation" section of
the broker `installation chapter in the Tunneldigger documentation`_.
* Added cmake buildsystem to the client.
* Removed NAT implementation.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* Removed NAT implementation.
* Removed NAT-based handling of many client tunnels on the same server port.
We now rely on the kernel properly distinguishing those UDP sockets.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changed it so "we" is not used. Instead "The broker now relies on the kernel properly distinguishing those UDP sockets." instead. Done.

HISTORY.rst Outdated Show resolved Hide resolved
HISTORY.rst Outdated Show resolved Hide resolved
HISTORY.rst Outdated Show resolved Hide resolved
HISTORY.rst Outdated Show resolved Hide resolved
HISTORY.rst Outdated Show resolved Hide resolved
HISTORY.rst Outdated Show resolved Hide resolved
HISTORY.rst Outdated Show resolved Hide resolved
@pmelange
Copy link
Contributor Author

pmelange commented Aug 18, 2023 •
edited
Loading

Thanks for going through all the changes! I just have some editorial suggestions.

Thanks for the suggestions. Most of them I did verbatim. A couple I reworded.

@pmelange
Update changelog for 0.4.0
3d5bcbc 
Signed-off-by: Perry Melange <[email protected]>
@PolynomialDivision
Copy link

ping @RalfJung ;) Would be great to get this merged to finally have a complete working version with openwrt. Thanks a lot for your work.

@RalfJung RalfJung force-pushed the release_d538780 branch 3 times, most recently from 2c3eef8 to e8f18ed Compare August 19, 2023 13:03
@RalfJung RalfJung merged commit 2cece2e into wlanslovenija:master Aug 19, 2023
5 checks passed
@pmelange pmelange deleted the release_d538780 branch August 19, 2023 15:30
@RalfJung
Copy link
Member

Merged the PR and pushed the tag. :)

@pmelange
Copy link
Contributor Author

Merged the PR and pushed the tag. :)
Thank you @RalfJung

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

@RalfJung RalfJung RalfJung left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@pmelange @mitar @RalfJung @PolynomialDivision