Page MenuHomeFreeBSD

Add OPTIONS for a better Plasma 5 desktop and Falkon experience
ClosedPublic

Authored by adridg on Dec 29 2018, 3:00 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Mar 22, 3:46 PM
Unknown Object (File)
Fri, Mar 22, 3:46 PM
Unknown Object (File)
Fri, Mar 22, 3:46 PM
Unknown Object (File)
Fri, Mar 22, 3:46 PM
Unknown Object (File)
Fri, Mar 22, 3:46 PM
Unknown Object (File)
Fri, Mar 22, 10:09 AM
Unknown Object (File)
Fri, Mar 22, 10:09 AM
Unknown Object (File)
Sun, Mar 10, 4:06 AM
Subscribers

Details

Summary

There are various RUN_DEPENDS that you're told to install via
pkg-message, but we may as well install them by default when
building packages. GStreamer is installed already through other
dependencies, so the additional packages are small, and this
means that sound Just Works with both the browser and the desktop.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 21774
Build 21048: arc lint + arc unit

Event Timeline

tcberner added inline comments.
x11/plasma5-plasma/Makefile
23

^ I think this is the wrong place for these.

x11/plasma5-plasma-workspace or x11/plasma5-plasma-desktop seem the btter choices, as this one is just the metaport. So people with minimal installations will not profit from it.

26

^_DESC?

30

^ this could be USE_QT=phonon-gst -- but I thing phonon should leave qt.mk ... so ...

This revision now requires changes to proceed.Dec 29 2018, 3:11 PM

Improve the RUN_DEPENDS with better syntax

Persist in having the option in the wrong place, but add plasma-pa option (OFF)

Move PINENTRY option to where it is most needed

Forgot PORTREVISION, also make OPTIONS_DEFAULT explicit

I would drop the ASLA and PULSEAUDIO _DESCs and use the ones provided by bsd.options.desc.mk. For phonon I would keep the one you defined.

www/falkon/Makefile
32

^ ALSA_DESC is already provided by bsd.options.desc.mk

x11/plasma5-plasma/Makefile
27

^ this is also available in bsd.options.desc.mk

This revision is now accepted and ready to land.Dec 29 2018, 10:15 PM
This revision was automatically updated to reflect the committed changes.