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

Revert using random port by default #12044

Merged
merged 1 commit into from
Feb 21, 2020

Conversation

FranciscoPombal
Copy link
Member

Revert change introduced in #11637 and also revert the associated follow-up #12000.

Reason: arvidn/libtorrent#4335

Associated discussion: #11935 (after the initial closing of the issue).

@FranciscoPombal FranciscoPombal added Network Issues related to network connectivity Hotfix Fix some bug(s) introduced by recently merged commit(s) labels Feb 21, 2020
@FranciscoPombal FranciscoPombal added this to the 4.2.2 milestone Feb 21, 2020
@FranciscoPombal FranciscoPombal self-assigned this Feb 21, 2020
Chocobo1
Chocobo1 previously approved these changes Feb 21, 2020
glassez
glassez previously approved these changes Feb 21, 2020
@ghost
Copy link

ghost commented Feb 21, 2020

Someone once told me to not include periods in commit title.

@Chocobo1
Copy link
Member

Someone once told me to not include periods in commit title.

Yes, the period should be removed.

@FranciscoPombal
Copy link
Member Author

@Chocobo1 fixed commit message.

@glassez glassez changed the title Revert using random port by default. Revert using random port by default Feb 21, 2020
glassez
glassez previously approved these changes Feb 21, 2020
Revert change introduced in qbittorrent#11637 and also revert the associated follow-up qbittorrent#12000.

Reason: arvidn/libtorrent#4335
@FranciscoPombal
Copy link
Member Author

Sorry had to force push to fix the commit author

@Chocobo1 Chocobo1 merged commit 68709c3 into qbittorrent:master Feb 21, 2020
@Chocobo1
Copy link
Member

@FranciscoPombal
Thank you!

@FranciscoPombal FranciscoPombal deleted the fix-port branch February 21, 2020 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Hotfix Fix some bug(s) introduced by recently merged commit(s) Network Issues related to network connectivity
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants