Bug 228178 - net-im/tox: unnecessary dependency on graphics/opencv-core
Summary: net-im/tox: unnecessary dependency on graphics/opencv-core
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Many People
Assignee: freebsd-ports-bugs (Nobody)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-05-11 22:55 UTC by Vladimir Druzenko
Modified: 2018-05-12 16:18 UTC (History)
1 user (show)

See Also:
thierry: maintainer-feedback+

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Vladimir Druzenko freebsd_committer freebsd_triage 2018-05-11 22:55:02 UTC 
Small log from IRC:
[10.05.2018][03:57:09] <VVD> one more strange dependency: toxcore (1.03MiB) => opencv-core (9.70MiB) => tesseract (10.9MiB) => tesseract-data (1.05GiB!!!)
[10.05.2018][03:57:41] <VVD> I want just tox node on small virtual machine
[10.05.2018][04:40:07] <@nurupo> there is a test program av_test at testing/av_test.c that requires opencv, but there is no reason to package test programs along with the library

Patch is very simply:
--- net-im/tox/Makefile.orig
+++ net-im/tox/Makefile
@@ -19,7 +19,6 @@
        libsndfile.so:audio/libsndfile  \
        libportaudio.so:audio/portaudio    \
        libsodium.so:security/libsodium    \
-       libopencv_core.so:graphics/opencv-core  \
        libopus.so:audio/opus           \
        libconfig.so:devel/libconfig

Tested on 11.1-p10 amd64.
Comment 1 Thierry Thomas freebsd_committer freebsd_triage 2018-05-12 14:27:02 UTC 
Thanks for the report: although opencv is checked by configure, I moved it to TEST_DEPENDS, and everything seems fine.
Comment 2 commit-hook freebsd_committer freebsd_triage 2018-05-12 14:27:29 UTC 
A commit references this bug:

Author: thierry
Date: Sat May 12 14:26:56 UTC 2018
New revision: 469731
URL: https://svnweb.freebsd.org/changeset/ports/469731

Log:
  Opencv is checked by configure, but actually it is only needed for
  tests.

  PR:		228178
  Reported by:	vvd (at) unislabs.com

Changes:
  head/net-im/tox/Makefile
Comment 3 Vladimir Druzenko freebsd_committer freebsd_triage 2018-05-12 16:18:04 UTC 
Thanks!