Bug 206126 - Mk/Uses/uidfix.mk: Add INCSOWN and INCSGRP
Summary: Mk/Uses/uidfix.mk: Add INCSOWN and INCSGRP
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Ports Framework (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Mikhail Teterin
URL:
Keywords: feature, needs-qa, patch
Depends on:
Blocks:
 
Reported: 2016-01-11 05:47 UTC by Mikhail Teterin
Modified: 2016-04-13 12:22 UTC (History)
2 users (show)

See Also:
koobs: maintainer-feedback? (portmgr)


Attachments
Overwrite INCSOWN and INCSGRP as well (279 bytes, patch)
2016-01-11 05:47 UTC, Mikhail Teterin
no flags Details | Diff
Properly add the new knobs to the environment (548 bytes, patch)
2016-01-11 05:49 UTC, Mikhail Teterin
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mikhail Teterin freebsd_committer freebsd_triage 2016-01-11 05:47:29 UTC
Created attachment 165387 [details]
Overwrite INCSOWN and INCSGRP as well

Some times one needs to install header files along with the library(ies). The patch adds INCSOWN and INCSGRP to the collection of *OWN and *GRP already there.
Comment 1 Mikhail Teterin freebsd_committer freebsd_triage 2016-01-11 05:49:58 UTC
Created attachment 165388 [details]
Properly add the new knobs to the environment
Comment 2 Baptiste Daroussin freebsd_committer freebsd_triage 2016-01-11 07:11:05 UTC
Thanks for the patch, looks good to me
Comment 3 Mathieu Arnold freebsd_committer freebsd_triage 2016-04-13 11:35:26 UTC
This was approved, so...
Comment 4 commit-hook freebsd_committer freebsd_triage 2016-04-13 12:21:42 UTC
A commit references this bug:

Author: mi
Date: Wed Apr 13 12:20:40 UTC 2016
New revision: 413204
URL: https://svnweb.freebsd.org/changeset/ports/413204

Log:
  Set the user:group for the include-files as well.

  PR:		206126
  Approved by:	bapt

Changes:
  head/Mk/Uses/uidfix.mk
Comment 5 Mikhail Teterin freebsd_committer freebsd_triage 2016-04-13 12:22:28 UTC
Thanks for the reminder, mat.