Bug 212406 - audio/mpg123: Add SNDIO option
Summary: audio/mpg123: Add SNDIO option
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Roman Bogorodskiy
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2016-09-06 08:40 UTC by Tobias Kortkamp
Modified: 2016-09-06 16:33 UTC (History)
1 user (show)

See Also:
bugzilla: maintainer-feedback? (novel)

Attachments
mpg123.diff (1.18 KB, patch)
2016-09-06 08:40 UTC, Tobias Kortkamp 		tobik: maintainer-approval? (novel)
 Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Tobias Kortkamp freebsd_committer freebsd_triage 2016-09-06 08:40:33 UTC 
Created attachment 174409 [details]
mpg123.diff

This adds a new option to the port that enables building of the audio/sndio output module.

I've also removed CPPFLAGS/LDFLAGS and replaced it with USES=localbase.  stage-qa would warn about libraries not being stripped, so I also added INSTALL_TARGET=install-strip.

Poudriere testport 9.3/i386 ok, portlint ok
Comment 1 commit-hook freebsd_committer freebsd_triage 2016-09-06 16:29:41 UTC 
A commit references this bug:

Author: novel
Date: Tue Sep  6 16:28:46 UTC 2016
New revision: 421440
URL: https://svnweb.freebsd.org/changeset/ports/421440

Log:
  audio/mpg123: add SNDIO option and cleanups

   - add a SNDIO option that enables building of the audio/sndio output
     module
   - replace CPPFLAGS/LDFLAGS with USES=localbase
   - use INSTALL_TARGET=install-strip to strip libraries
   - bump PORTREVISION due to packaging change

  PR:		212406
  Submitted by:	Tobias Kortkamp

Changes:
  head/audio/mpg123/Makefile
  head/audio/mpg123/pkg-plist
Comment 2 Roman Bogorodskiy freebsd_committer freebsd_triage 2016-09-06 16:33:53 UTC 
Committed, thanks!