Bug 219260

Summary: sysutils/screen: use and link with ncurses from the port if installed
Product: Ports & Packages Reporter: Akinori MUSHA <knu>
Component: Individual Port(s)Assignee: Cy Schubert <cy>
Status: Closed FIXED    
Severity: Affects Many People Flags: bugzilla: maintainer-feedback? (cy)
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Patch for sysutils/screen none

Description Akinori MUSHA freebsd_committer 2017-05-13 18:45:59 UTC
Created attachment 182572 [details]
Patch for sysutils/screen

The sysutils/screen port declares it uses ncurses, but it actually doesn't use devel/ncurses when it is installed.

The configure script of screen always picks termlib which is linked with ncurses in the base.
Comment 1 commit-hook freebsd_committer 2017-05-14 04:39:45 UTC
A commit references this bug:

Author: cy
Date: Sun May 14 04:39:16 UTC 2017
New revision: 440861
URL: https://svnweb.freebsd.org/changeset/ports/440861

Log:
  Properly detect devel/ncurses if installed and use it. Previously if
  devel/ncurses was installed, even though the dependency was registered,
  base ncurses would still be linked.

  PR:		219260
  Submitted by:	knu@

Changes:
  head/sysutils/screen/Makefile
  head/sysutils/screen/files/patch-configure
Comment 2 Cy Schubert freebsd_committer 2017-05-14 04:45:59 UTC
Committed. Thanks for the patch.