Bug 217546

Summary: [patch] multimedia/smplayer: add Qt5 option, drop qt*-script from dependencies
Product: Ports & Packages Reporter: Andriy Voskoboinyk <avos>
Component: Individual Port(s)Assignee: Rusmir Dusko <nemysis>
Status: Closed FIXED    
Severity: Affects Only Me CC: pawel
Priority: --- Keywords: patch
Version: LatestFlags: bugzilla: maintainer-feedback? (nemysis)
Hardware: Any   
OS: Any   
Attachments:
Description Flags
The patch
none
Poudriere logs (10.3-RELEASE i386, 11.0-RELEASE amd64) none

Description Andriy Voskoboinyk freebsd_committer freebsd_triage 2017-03-04 21:04:37 UTC
Created attachment 180509 [details]
The patch

- Provide a choice between Qt versions (4/5).
- Add possibility to use MPV as a backend; disable support in code when backend is not selected (mplayer2 support is always disabled).
- Add options for additional skins / themes.
- Always disable YT_USE_SCRIPT; smplayer does not have youtube support and downloaded code seems to be unused (if needed, it can be fetched via multimedia/smtube, Help -> "Update the YouTube code"); qt*-script dependency is not needed without it.
- Bump PORTREVISION.
Comment 1 Andriy Voskoboinyk freebsd_committer freebsd_triage 2017-03-04 21:17:22 UTC
Created attachment 180510 [details]
Poudriere logs (10.3-RELEASE i386, 11.0-RELEASE amd64)

(warnings about libXext, libGL are wrong and should go away when compiled with LDFLAGS=-Wl,--as-needed + non-broken compiler (clang ignores this option))
Comment 2 commit-hook freebsd_committer freebsd_triage 2018-01-17 17:17:47 UTC
A commit references this bug:

Author: pawel
Date: Wed Jan 17 17:17:26 UTC 2018
New revision: 459273
URL: https://svnweb.freebsd.org/changeset/ports/459273

Log:
  - Provide a choice between Qt versions (4/5).
  - Add possibility to use MPV as a backend; disable support in code when backend
    is not selected (obsolete mplayer2 support is always disabled).
  - Add options for additional skins / themes
  - Convert to options helpers

  PR:		217546
  Submitted by:	avos
  Approved by:	maintainer timeout

Changes:
  head/multimedia/smplayer/Makefile
  head/multimedia/smplayer/files/patch-src_smplayer.pro
Comment 3 Pawel Pekala freebsd_committer freebsd_triage 2018-01-17 17:23:14 UTC
Committed without YT_USE_SCRIPT disable part because newer versions of smtube
don't have mentioned option anymore. Thanks.