-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
dpp: update to 10.0.30. #51144
dpp: update to 10.0.30. #51144
Conversation
ea7ef25
to
8fbb96f
Compare
7f84a44
to
eee5b94
Compare
What's the argument for adding a build option? Options present a terrible user experience and should generally be avoided. Usually, we provide them to:
As your addition neither depends on the arch nor adds dependencies, I wonder why it isn't just added unconditionally. |
There were some issue compiling it with coroutine support enabled with gcc 12 which Void was using, and it was fine when compiled against gcc 13. My concern is that some people probably don't want to upgrade their compiler from 12 to 13 just to use this library.
|
Void's GCC is at version 13, so the package will be built with GCC 13. Supporting builds on GCC 12 is not an issue and there is no need to complicate the template. If coroutine support is useful, just enable it; otherwise, just leave it out. |
eee5b94
to
b0c6742
Compare
Testing the changes
Local build testing