Bug 238704

Summary: ftp/R-cran-curl: fix build on big-endian architectures
Product: Ports & Packages Reporter: Piotr Kubaj <pkubaj>
Component: Individual Port(s)Assignee: TAKATSU Tomonari <tota>
Status: Closed FIXED    
Severity: Affects Only Me CC: linimon, tota
Priority: --- Flags: bugzilla: maintainer-feedback? (tota)
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
patch pkubaj: maintainer-approval? (tota)

Description Piotr Kubaj freebsd_committer freebsd_triage 2019-06-19 11:39:34 UTC
Created attachment 205218 [details]
patch

BSWAP_32 is undefined on FreeBSD, use bswap32.

It also needs sys/endian.h include.

Needed because of:
  /wrkdirs/usr/ports/ftp/R-cran-curl/work/stage/usr/local/lib/R/library/00LOCK-curl/00new/curl/libs/curl.so: Undefined symbol "BSWAP_32"
Comment 1 TAKATSU Tomonari freebsd_committer freebsd_triage 2019-06-23 11:30:11 UTC
(In reply to Piotr Kubaj from comment #0)

LGTM
Comment 2 commit-hook freebsd_committer freebsd_triage 2019-06-24 07:24:31 UTC
A commit references this bug:

Author: pkubaj
Date: Mon Jun 24 07:24:28 UTC 2019
New revision: 505010
URL: https://svnweb.freebsd.org/changeset/ports/505010

Log:
  ftp/R-cran-curl: fix build on big-endian architectures

  BSWAP_32 is undefined on FreeBSD, use bswap32.

  It also needs sys/endian.h include.

  Needed because of:

    /wrkdirs/usr/ports/ftp/R-cran-curl/work/stage/usr/local/lib/R/library/00LOCK-curl/00new/curl/libs/curl.so: Undefined symbol "BSWAP_32"

  PR:		238704
  Approved by:	tota (maintainer), tcberner (mentor)
  Differential Revision:	https://reviews.freebsd.org/D20726

Changes:
  head/ftp/R-cran-curl/files/
  head/ftp/R-cran-curl/files/patch-src_curl.c