Bug 230181

Summary: [patch] sysutils/cpupdate: Include microcode updates in packages
Product: Ports & Packages Reporter: Romain Tartière <romain>
Component: Individual Port(s)Assignee: Eugene Grosbein <eugen>
Status: Closed Not Accepted    
Severity: Affects Only Me Flags: bugzilla: maintainer-feedback? (eugen)
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Fix to package microcodes none

Description Romain Tartière freebsd_committer freebsd_triage 2018-07-30 08:34:41 UTC
Created attachment 195624 [details]
Fix to package microcodes

Dear maintainer,

Maybe I am missing a point, but packaging sysutils/cpupdate never include the CPU microcodes: a manual intervention from the port directory is needed to install them (make install-microcodes), which is not applicable when managing systems through packages.

The proposed patch stages the microcode updates when the corresponding options are set and adds the microcode updates to the packing list.  All informational messages related to the manual steps are removed.

The default package is unchanged (because CPM and INTEL are unset by default).
Comment 1 Eugene Grosbein freebsd_committer freebsd_triage 2018-07-30 09:06:36 UTC
All installed files must be added to pkg-plist and therefore included into the package. However, platomav/CPUMicrocodes is really distinct project and its growth is not syncronized with cpupdate.

More serious problem steams from copyright issues: I'm sure that every owner of Intel processors is allowed to utilize published CPU microcodes but I do not know if FreeBSD project is allowed to package and redistribute Intel's microcode files. And I do not want to limit redistribution of cpupdate package itself due to copyrighted blobs added to the package.
Comment 2 commit-hook freebsd_committer freebsd_triage 2018-08-17 19:47:00 UTC
A commit references this bug:

Author: eugen
Date: Fri Aug 17 19:45:56 UTC 2018
New revision: 477447
URL: https://svnweb.freebsd.org/changeset/ports/477447

Log:
  Minor update for sysutils/cpupdate:

  - sync with Intel's microcode-20180807.tgz;
  - sync with platomav/CPUMicrocodes MCE DB r79;
  - follow audio/lame practice and add some comments to the "LICENSE" section
    of port's Makefile describing possible legal issues preventing microcode
    redistribution due to multiple user requests.

  No package changes, so no PORTREVISION bump.

  PR:		226620, 230181

Changes:
  head/sysutils/cpupdate/Makefile
  head/sysutils/cpupdate/distinfo