Bug 217559

Summary: [patch] multimedia/smtube: add Qt5 option, provide a choice between supported video players
Product: Ports & Packages Reporter: Andriy Voskoboinyk <avos>
Component: Individual Port(s)Assignee: Pawel Pekala <pawel>
Status: Closed FIXED    
Severity: Affects Only Me CC: avos
Priority: --- Keywords: patch
Version: LatestFlags: pawel: maintainer-feedback+
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Makefile.diff
none
Poudriere logs (10.3-RELEASE i386, 11.0-RELEASE amd64) none

Description Andriy Voskoboinyk freebsd_committer freebsd_triage 2017-03-05 10:07:58 UTC
Created attachment 180529 [details]
Makefile.diff

- Add Qt5 / Qt4 radio-choice.
- Allow to add MPlayer / mpv / VLC / SMPlayer as a runtime dependency for video playback.
- Bump PORTREVISION.
Comment 1 Andriy Voskoboinyk freebsd_committer freebsd_triage 2017-03-05 10:10:55 UTC
(multimedia/gnome-mplayer does not play videos from smtube out-of-the-box, so it is omitted from the list for now)
Comment 2 Andriy Voskoboinyk freebsd_committer freebsd_triage 2017-03-05 10:13:15 UTC
Created attachment 180530 [details]
Poudriere logs (10.3-RELEASE i386, 11.0-RELEASE amd64)

(warnings about libGL are wrong and should go away when compiled with LDFLAGS=-Wl,--as-needed + non-broken compiler (clang ignores this option))
Comment 3 commit-hook freebsd_committer freebsd_triage 2017-03-06 19:28:56 UTC
A commit references this bug:

Author: pawel
Date: Mon Mar  6 19:27:57 UTC 2017
New revision: 435565
URL: https://svnweb.freebsd.org/changeset/ports/435565

Log:
  - Add Qt5 / Qt4 radio-choice
  - Allow to add MPlayer / mpv / VLC / SMPlayer as a runtime
    dependency for video playback

  PR:		217559
  Submitted by:	avos

Changes:
  head/multimedia/smtube/Makefile