Bug 235482

Summary: net/libmaxminddb: use GCC on i386 since using clang generates an incorrect maxminddb_config.h
Product: Ports & Packages Reporter: Ralf van der Enden <tremere>
Component: Individual Port(s)Assignee: Po-Chuan Hsieh <sunpoet>
Status: Closed FIXED    
Severity: Affects Some People CC: tremere
Priority: --- Flags: bugzilla: maintainer-feedback? (sunpoet)
Version: Latest   
Hardware: Any   
OS: Any   
Bug Depends on:    
Bug Blocks: 235154    
Attachments:
Description Flags
Change to GCC on i386
none
Add sunpoet's patch to fix maxminddb_config.h on i386
tremere: maintainer-approval?
Poudriere Log none

Description Ralf van der Enden 2019-02-04 08:52:36 UTC
Created attachment 201710 [details]
Change to GCC on i386

On i386 building with Clang gives you an incorrect include/maxminddb_config.h.
configure tries to detect support for unsigned __int128 in two ways. The first one fails for both GCC and Clang, but the second is incorrectly detected as true by Clang. This can cause issues in packages depending on libmaxminddb (.ie security/modsecurity3)

Bump portrevision because a new maxminddb_config.h needs to be generated on i386. Also petted portlint so it doesn't generate any errors anymore.
Comment 1 Po-Chuan Hsieh freebsd_committer freebsd_triage 2019-02-05 12:55:43 UTC
Try https://people.FreeBSD.org/~sunpoet/patch/net-libmaxminddb.txt instead of using GCC on i386.
Comment 2 Ralf van der Enden 2019-02-05 13:52:12 UTC
Created attachment 201763 [details]
Add sunpoet's patch to fix maxminddb_config.h on i386


Your patch works as expected, so redid my initial patch.
Comment 3 commit-hook freebsd_committer freebsd_triage 2019-02-05 14:27:32 UTC
A commit references this bug:

Author: sunpoet
Date: Tue Feb  5 14:26:58 UTC 2019
New revision: 492238
URL: https://svnweb.freebsd.org/changeset/ports/492238

Log:
  Fix __int128 detection on i386

  PR:		235482
  Reported by:	Ralf van der Enden <tremere@cainites.net>

Changes:
  head/net/libmaxminddb/Makefile
  head/net/libmaxminddb/files/
  head/net/libmaxminddb/files/patch-configure
Comment 4 Po-Chuan Hsieh freebsd_committer freebsd_triage 2019-02-05 14:28:12 UTC
Committed. Thanks!
Comment 5 Helmut Ritter 2022-07-26 11:44:49 UTC
Created attachment 235483 [details]
Poudriere Log