-
-
Notifications
You must be signed in to change notification settings - Fork 13.7k
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
prosody: remove deprecated libevent #256738
Conversation
@@ -73,6 +73,7 @@ stdenv.mkDerivation rec { | |||
''; | |||
|
|||
passthru = { | |||
withLibevent = withLibevent; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we need this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think so
withLibevent = withLibevent; |
${ optionalString cfg.package.withLibevent '' | ||
-- libevent was opted into (default is "epoll") | ||
network_backend = "event" | ||
'' } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
${ optionalString cfg.package.withLibevent '' | |
-- libevent was opted into (default is "epoll") | |
network_backend = "event" | |
'' } |
we might as well remove this
@@ -779,8 +779,10 @@ in | |||
|
|||
admins = ${toLua cfg.admins} | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -73,6 +73,7 @@ stdenv.mkDerivation rec { | |||
''; | |||
|
|||
passthru = { | |||
withLibevent = withLibevent; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think so
withLibevent = withLibevent; |
@@ -2,7 +2,7 @@ | |||
, icu | |||
, lua | |||
, nixosTests | |||
, withLibevent ? true | |||
, withLibevent ? false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can even go as far and just drop that option entirely
`use_libevent` config option has been deprecated as there are 3 networking back-ends (see: https://prosody.im/doc/network_backend). It is probably unwise to stray from the defaults (epoll). Review instructed to flat-out remove it versus previously making it opt-in.
d223a47
to
ccb0f87
Compare
@SuperSandro2000 made adjustments. Could help until the rewrite is approved. |
Yes I agree we should go forward with this PR. We're to close to next release to merge the other PR at this point. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Is this good? |
use_libevent
config option has been deprecated as there are 3 networking back-ends (see: https://prosody.im/doc/network_backend). It is probably unwise to stray from the defaults (epoll).Description of changes
Info for this change provided by @Zash & @K900
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)