Bug 253586

Summary: [PATCH] databases/oracle8-client: unbreak for 13+
Product: Ports & Packages Reporter: Konstantin Stroykovskiy <bonkor>
Component: Individual Port(s)Assignee: Eugene Grosbein <eugen>
Status: Closed FIXED    
Severity: Affects Some People CC: eugen
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Patch needed to build the port
none
fix build for 13+ eugen: maintainer-approval?

Description Konstantin Stroykovskiy 2021-02-17 14:08:03 UTC
Created attachment 222527 [details]
Patch needed to build the port

Base system objcopy does not have the necessary functionality.
Comment 1 Eugene Grosbein freebsd_committer freebsd_triage 2021-02-17 16:24:59 UTC
Base system "objcopy" is the "elfcopy" utility from elftoolchain for supported versions of FreeBSD (11+). It really has working --redefine-syms flag despite its manual pages does not document it.

Have you really tested your patch and with which FreeBSD version?

The change of NO_WERROR to MK_WERROR breaks build for both FreeBSD 11.4 and FreeBSD 12.2.
Comment 2 Eugene Grosbein freebsd_committer freebsd_triage 2021-02-18 07:05:14 UTC
Created attachment 222547 [details]
fix build for 13+
Comment 3 Eugene Grosbein freebsd_committer freebsd_triage 2021-02-18 07:06:44 UTC
Dear maintainer, please test and approve alternative patch using system objcopy and not breaking build for other stable branches.
Comment 4 Konstantin Stroykovskiy 2021-02-18 12:31:53 UTC
Comment on attachment 222547 [details]
fix build for 13+

Port with this patch is working. Tested on 14.0 and 12.2.
Comment 5 commit-hook freebsd_committer freebsd_triage 2021-02-19 07:25:49 UTC
A commit references this bug:

Author: eugen
Date: Fri Feb 19 07:25:33 UTC 2021
New revision: 566032
URL: https://svnweb.freebsd.org/changeset/ports/566032

Log:
  databases/oracle8-client: unbreak build for FreeBSD 13+

  PR:		253586
  Approved by:	Konstantin Stroykovskiy (maintainer)

Changes:
  head/databases/oracle8-client/files/patch-freebsd-Makefile
  head/databases/oracle8-client/files/patch-freebsd-redefine-syms.sh
Comment 6 Eugene Grosbein freebsd_committer freebsd_triage 2021-02-19 07:28:29 UTC
Fixed, thank you for the report!