Bug 208162 - [patch] emulators/wine-staging: add missing dependency on graphics/s2tc
Summary: [patch] emulators/wine-staging: add missing dependency on graphics/s2tc
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: David Naylor
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2016-03-20 19:42 UTC by David Naylor
Modified: 2016-03-22 06:26 UTC (History)
3 users (show)

See Also:
gerald: maintainer-feedback+


Attachments
Makefile patch for STAGING_LIB_DEPENDS (367 bytes, patch)
2016-03-20 19:42 UTC, David Naylor
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description David Naylor freebsd_committer freebsd_triage 2016-03-20 19:42:36 UTC
Created attachment 168434 [details]
Makefile patch for STAGING_LIB_DEPENDS

The attached patch adds a missing dependency on s2tc, this is required for the wined3d-DXTn patchset (https://github.com/wine-compholio/wine-staging/tree/master/patches/wined3d-DXTn).
Comment 1 mickael.maillot 2016-03-20 19:47:52 UTC
+1
i confirm: with libtxc_dxtn.so wine with staging correctly use DXTn, tested with World of Warcraft for month now.
Comment 2 Gerald Pfeifer freebsd_committer freebsd_triage 2016-03-22 00:37:48 UTC
Approved.

No PORTREVISION bump is necessary for wine-devel, since this does not
change the package for the default build.

I recommend you do bump PORTREVISION for wine-staging as part of the
same commit, since there things will change.
Comment 3 commit-hook freebsd_committer freebsd_triage 2016-03-22 06:26:05 UTC
A commit references this bug:

Author: dbn
Date: Tue Mar 22 06:25:43 UTC 2016
New revision: 411637
URL: https://svnweb.freebsd.org/changeset/ports/411637

Log:
  emulators/wine-staging: add missing dependency.

  Add a missing dependency on libtxc_dxtn.so (provided by graphics/s2tc).

  PORTREVISION is not bumped (despite policy) for the following reasons:
   - Bumping PORTREVISION just for the slave port wine-staging is likely to
     result in human error later on (i.e. it will cause issues if wine-devel
     needs a bump, or an update).
   - Bumping both wine-devel and wine-staging is overkill.
   - The port is newly updated likely to catch most builds with this update
   - The port is updated every two weeks, so everyone gets the improvement
     quickly enough.
   - This change will be in the i386-wine-staging update.

  PR:		208162
  Reported by:	Mickael Maillot <mickael.maillot@gmail.com>
  Approved by:	gerald@

Changes:
  head/emulators/wine-devel/Makefile
Comment 4 David Naylor freebsd_committer freebsd_triage 2016-03-22 06:26:20 UTC
Committed, thanks.