Bug 216834

Summary: audio/optimfrog not PREFIX-clean
Product: Ports & Packages Reporter: Mikhail Teterin <mi>
Component: Individual Port(s)Assignee: Matthew Rezny <rezny>
Status: Closed FIXED    
Severity: Affects Only Me CC: lifanov, riggs
Priority: --- Keywords: easy, patch, patch-ready
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Use -p option to pass prefix none

Description Mikhail Teterin freebsd_committer freebsd_triage 2017-02-05 20:01:54 UTC
Created attachment 179654 [details]
Use -p option to pass prefix

The the author's install-script allows to overwrite the desired prefix on command-line, the port's do-install does not make use of the option.

The paths should also be quoted -- just in case a directory has a space in its name.
Comment 1 Bugzilla Automation freebsd_committer freebsd_triage 2017-02-05 20:01:54 UTC
Maintainer informed via mail
Comment 2 Nikolai Lifanov freebsd_committer freebsd_triage 2017-02-16 19:15:18 UTC
take
Comment 3 Nikolai Lifanov freebsd_committer freebsd_triage 2017-02-16 19:41:40 UTC
Reset assignee: rezny has a commit bit now :)
Comment 4 Matthew Rezny freebsd_committer freebsd_triage 2017-02-25 13:27:46 UTC
Mikhail, thanks for pointing out the oversight. I had not considered the need to set prefix when creating the port since it defaults to /usr/local, but the user can well specify an alternate PREFIX for ports.

Nikolai, thanks for assigning to me so I notice the PR. I just checked my mailbox to verify that I got no email about this PR on the 5th although I did get one about an hour later regarding a different PR, so the message from Bugzilla Automation is false; the maintainer was not informed prior to today.
Comment 5 commit-hook freebsd_committer freebsd_triage 2017-02-25 14:50:53 UTC
A commit references this bug:

Author: rezny
Date: Sat Feb 25 14:50:29 UTC 2017
New revision: 434816
URL: https://svnweb.freebsd.org/changeset/ports/434816

Log:
  Pass PREFIX to the install script in case it's not /usr/local

  PR:		216834
  Submitted by:	mi
  Approved by:	swills (mentor)
  Differential Revision:	https://reviews.freebsd.org/D9800

Changes:
  head/audio/optimfrog/Makefile