Bug 243444

Summary: [maintainer update] japanese/lynx-current: Fix build error on -current and update to the latest development release
Product: Ports & Packages Reporter: WATANABE Kazuhiro <CQG00620>
Component: Individual Port(s)Assignee: Wen Heping <wen>
Status: Closed FIXED    
Severity: Affects Only Me CC: wen
Priority: --- Keywords: buildisok
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
A svn diff for japanese/lynx-current
none
poudriere testport log on 12.1-RELEASE/amd64
none
poudriere testport log on 13.0-CURRENT/amd64 without the patch
none
poudriere testport log on 13.0-CURRENT/amd64 with the patch none

Description WATANABE Kazuhiro 2020-01-19 09:10:35 UTC
Created attachment 210860 [details]
A svn diff for japanese/lynx-current

(1) Update to the latest development release.

(2) Use ncursesw instead of ncurses. (Makefile)

(3) Add IDN and SSL configuration options. (Makefile)

Recent lynx detects GNU Libidn automatically at build time and links it
if exists.  But japanese/lynx-current doesn't describe the dependency anywhere.
To fix the problem, add an IDN configuration option (on by default).  
Also adds a SSL option (on by default).

These codes are taken from www/{lynx,lynx-current}.  Thanks!

(4) Fix poudriere warnings about symlinks with an absolute pathname.
    (Makefile)

Poudriere testport log says:

| ====> Running Q/A tests (stage-qa)
| Warning: Bad symlink '/usr/local/share/lynx_help/COPYING' pointing to an absolute pathname '/usr/local/share/doc/lynx/COPYING'
| Warning: Bad symlink '/usr/local/share/lynx_help/COPYHEADER' pointing to an absolute pathname '/usr/local/share/doc/lynx/COPYHEADER'

To fix the problem, add a code to the post-install-DOCS-on target to override
the symlinks with a relative pathname.

This code is taken from www/lynx.  Thanks!

(5) Some small fix to Makefile.
    
Add LICENSE_FILE and modify CONFLICTS.  Fix portlint FATAL messages belongs to
the order of variables (PKGNAMESUFFIX and USES).
Comment 1 WATANABE Kazuhiro 2020-01-19 09:13:03 UTC
Created attachment 210861 [details]
poudriere testport log on 12.1-RELEASE/amd64
Comment 2 Automation User 2020-01-19 09:28:51 UTC
Build info is available at https://gitlab.com/swills/freebsd-ports/pipelines/110533257
Comment 3 WATANABE Kazuhiro 2020-02-24 08:54:15 UTC
Due to the system ncurses update, japanese/lynx-current causes build error on
the latest -current:

 [package - head-amd64-default][japanese/lynx-current] Failed for ja-lynx-current-2.8.9.d8 in build
 https://docs.freebsd.org/cgi/getmsg.cgi?fetch=16672808+0+current/freebsd-pkg-fallout

This patch also fix the error, so please commit.
Comment 4 WATANABE Kazuhiro 2020-02-24 08:57:38 UTC
Created attachment 211880 [details]
poudriere testport log on 13.0-CURRENT/amd64 without the patch

Tested on FreeBSD-13.0-CURRENT-amd64-20200220-r358133.
Comment 5 WATANABE Kazuhiro 2020-02-24 08:58:56 UTC
Created attachment 211881 [details]
poudriere testport log on 13.0-CURRENT/amd64 with the patch

Tested on FreeBSD-13.0-CURRENT-amd64-20200220-r358133.
Comment 6 commit-hook freebsd_committer freebsd_triage 2020-02-24 14:41:15 UTC
A commit references this bug:

Author: wen
Date: Mon Feb 24 14:40:59 UTC 2020
New revision: 526972
URL: https://svnweb.freebsd.org/changeset/ports/526972

Log:
  - Update to latest release
  - Use ncursesw instead of ncurses
  - Add IDN and SSL configuration options
  - Add LICENSE_FILE
  - Update CONFLICTS

  PR:		243444
  Submitted by:	CQG00620@nifty.ne.jp(maintainer)

Changes:
  head/japanese/lynx-current/Makefile
  head/japanese/lynx-current/distinfo