Bug 225376

Summary: [emulators/virtualbox-ose] [patch] extra dependency on sysutils/cdrtools
Product: Ports & Packages Reporter: Eugene Grosbein <eugen>
Component: Individual Port(s)Assignee: Eugene Grosbein <eugen>
Status: Closed FIXED    
Severity: Affects Some People CC: marius, vbox
Priority: --- Keywords: easy, patch
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
remove dependency on cdrtools none

Description Eugene Grosbein freebsd_committer freebsd_triage 2018-01-22 10:15:43 UTC 
Created attachment 189960 [details]
remove dependency on cdrtools

emulators/virtualbox-ose depends on sysutils/cdrtools to install makeisofs.

We could replace this dependency with in-base makefs that is capable to make ISO images but it is not needed too. Its configure script says:

# Check for mkisofs, needed to build the CDROM image containing the additions

In fact, VirtualBox build system does NOT use make use of makeisofs while building for FreeBSD as it does not try to buid

https://www.virtualbox.org/wiki/FreeBSD%20build%20instructions says in its Prerequisites section:

mkisofs (required only for additions which aren't supported yet)
=> /usr/ports/sysutils/cdrtools 

So, we need just drop this unneeded dependency. It was introduced over 6 years ago and was never really used:

https://svnweb.freebsd.org/ports/head/emulators/virtualbox-ose/Makefile?revision=274444&view=markup

The patch fixing this is attached.
Comment 1 commit-hook freebsd_committer freebsd_triage 2018-02-05 21:07:33 UTC 
A commit references this bug:

Author: eugen
Date: Mon Feb  5 21:06:41 UTC 2018
New revision: 461020
URL: https://svnweb.freebsd.org/changeset/ports/461020

Log:
  emulators/virtualbox-ose: remove extra dependency on cdrtools.

  Dependency on cdrtools was introduced 6 years ago for sake of mkisofs:
  https://svnweb.freebsd.org/ports/head/emulators/virtualbox-ose/Makefile?revision=274444&view=markup

  In fact, VirtualBox build system does not use makeisofs
  while building for FreeBSD because it does not even try to buid
  VirtualBox Additions ISO image as its documentation says
  in the Prerequisites section:

  	mkisofs (required only for additions which aren't supported yet)
  	=> /usr/ports/sysutils/cdrtools

  	https://www.virtualbox.org/wiki/FreeBSD%20build%20instructions

  And it never really used mkisofs.
  Drop the dependency and bump PORTREVISION.

  PR:		225376
  Approved by:	vbox (maintainer timeout, 2 weeks)

Changes:
  head/emulators/virtualbox-ose/Makefile
  head/emulators/virtualbox-ose/files/patch-configure