Bug 228543

Summary: Remove kld from CATEGORIES if USES=kmod is defined
Product: Ports & Packages Reporter: Mateusz Piotrowski <0mp>
Component: Individual Port(s)Assignee: Mateusz Piotrowski <0mp>
Status: Closed FIXED    
Severity: Affects Only Me CC: swills
Priority: --- Keywords: easy, patch-ready
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Patch removing explicit kld from CATEGORIES
none
Patch removing explicit kld from CATEGORIES (revision: 1) none

Description Mateusz Piotrowski freebsd_committer 2018-05-27 12:18:21 UTC
Created attachment 193761 [details]
Patch removing explicit kld from CATEGORIES

This patch removes "kld" from CATEGORIES if USES is set to "kmod" because according to Porter's Handbook:

> (5.3.3. Choosing the Right Category)
> Ports installing loadable kernel modules also have the virtual category 
> kld in their CATEGORIES line. This is one of the things handled 
> automatically by adding USES=kmod.
Comment 1 Mateusz Piotrowski freebsd_committer 2018-05-27 12:20:36 UTC
Created attachment 193762 [details]
Patch removing explicit kld from CATEGORIES (revision: 1)

Change to the previous patch: patch 3 more ports.
Comment 2 Mateusz Piotrowski freebsd_committer 2018-07-20 01:03:28 UTC
Differential revision: https://reviews.freebsd.org/D16359
Comment 3 commit-hook freebsd_committer 2018-07-20 09:31:00 UTC
A commit references this bug:

Author: 0mp
Date: Fri Jul 20 09:30:01 UTC 2018
New revision: 474998
URL: https://svnweb.freebsd.org/changeset/ports/474998

Log:
  Remove redundant kld from CATEGORIES if a port uses kmod.

  "USES=kmod" already adds kld to CATEGORIES so it is unnecessary to set
  CATEGORIES to kld explicitly.

  PR:		228543
  Reviewed by:	krion (mentor)
  Approved by:	krion (mentor)
  Differential Revision:	https://reviews.freebsd.org/D16359

Changes:
  head/graphics/drm-next-kmod/Makefile
  head/graphics/drm-stable-kmod/Makefile
  head/graphics/gpu-firmware-kmod/Makefile
  head/multimedia/pwcbsd/Makefile
  head/net/userfw/Makefile
  head/sysutils/scprotect/Makefile