net/turnserver: Disable all database options except SQLITE by default
AbandonedPublic
Actions

Authored by yuri on Jan 9 2018, 8:07 AM.

Details

Reviewers

tcberner
adamw
bhughes

Summary

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=209560

The long-forgotten bug report.
In a very simple app like this, there is no need to have heavy-weight DB backends permanently attached as dependencies. SQLite should more than suffice for 99.9% of users.

Diff Detail

Repository

rP FreeBSD ports repository

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 14213
Build 14376: arc lint + arc unit

Event Timeline

yuri created this revision.Jan 9 2018, 8:07 AM

Herald added a subscriber: mat. · View Herald TranscriptJan 9 2018, 8:07 AM

Harbormaster completed remote builds in B14213: Diff 37672.Jan 9 2018, 8:07 AM

yuri edited the summary of this revision. (Show Details)Jan 9 2018, 8:10 AM

yuri edited the summary of this revision. (Show Details)Jan 9 2018, 9:52 AM

tcberner accepted this revision.Jan 9 2018, 6:30 PM

This revision is now accepted and ready to land.Jan 9 2018, 6:30 PM

I'm the maintainer of net/turnserver, I took over after this PR got positive feedback (but before any change was done). At the very least, I want/need to leave Redis enabled, as this is necessary for receiving runtime notifications from the turnserver. Personally, I don't see the problem in pulling in all client databases, for maximum flexibility.

This revision now requires changes to proceed.Jan 9 2018, 7:20 PM

yuri abandoned this revision.Jan 10 2018, 8:27 PM

Revision Contents
Changeset List

Path

Size

net/

turnserver/

Makefile

3 lines

Diff 37672

View Options

net/turnserver: Disable all database options except SQLITE by defaultAbandonedPublicActions