Attention is currently required from: laforge, neels.
1 comment:
Patchset:
hmm, except, do we now need to update the build system to ensure that this option builds successfully?
I am not asking Osmocom core team to test builds with this new option. **I** will be making a lot of builds with --disable-ortp, and if it breaks in the future, I will be the one to raise the alert and fix it.
Might reconsider the effort:benefit ratio...?
What effort? I am not asking your team to add the new build variant to your CI, just merge the patch (providing the option) so I don't have to keep reapplying it to every new release and every sandbox build I do.
With the two +1's from you and @osmith@sysmocom.de I can now merge the patch - but I'll give it another few days for further review.
To view, visit change 36975. To unsubscribe, or for help writing mail filters, visit settings.