Bug 234061

Summary: net-im/prosody: fix build with GCC-based architectures, respect CFLAGS and LDFLAGS
Product: Ports & Packages Reporter: Piotr Kubaj <pkubaj>
Component: Individual Port(s)Assignee: David Thiel <lx>
Status: Closed FIXED    
Severity: Affects Only Me CC: linimon, pkubaj
Priority: --- Flags: bugzilla: maintainer-feedback? (lx)
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
patch
none
Poudriere errors none

Description Piotr Kubaj freebsd_committer freebsd_triage 2018-12-16 15:18:27 UTC
Created attachment 200154 [details]
patch

This port requires C11-compatible compiler. It also doesn't respect CFLAGS and LDFLAGS, so USE_CSTD actually didn't have any effect.

Add patch to respect CFLAGS.

Tested on powerpc64 and amd64.

Hardware sponsored by IntegriCloud.
Comment 1 David Thiel freebsd_committer freebsd_triage 2018-12-18 22:22:42 UTC
Created attachment 200242 [details]
Poudriere errors
Comment 2 David Thiel freebsd_committer freebsd_triage 2018-12-18 22:23:08 UTC
This doesn't build for me under poudriere for 12.0-RELEASE.
Comment 3 Piotr Kubaj freebsd_committer freebsd_triage 2018-12-19 15:58:41 UTC
It looks like my change to patch-configure breaks it, but my change to Makefile is enough to make it build on powerpc64 and doesn't break amd64.

Can you commit that?

To clear the confusion:
Index: Makefile
===================================================================
--- Makefile    (revision 487742)
+++ Makefile    (working copy)
@@ -20,8 +20,7 @@
                ${LUA_MODLIBDIR}/bit.so:devel/lua51-bitop
 LIB_DEPENDS=   libidn.so:dns/libidn

-USES=          cpe gmake lua:51 shebangfix ssl
-USE_CSTD=      c99
+USES=          compiler:c11 cpe gmake lua:51 shebangfix ssl
 SHEBANG_FILES= prosody prosodyctl
 LUA_PREMK=     yes
 HAS_CONFIGURE= yes
Comment 4 commit-hook freebsd_committer freebsd_triage 2018-12-19 18:45:55 UTC
A commit references this bug:

Author: lx
Date: Wed Dec 19 18:45:39 UTC 2018
New revision: 487809
URL: https://svnweb.freebsd.org/changeset/ports/487809

Log:
  Fix build on GCC architectures.

  PR:		234061
  Submitted by:	Piotr Kubaj

Changes:
  head/net-im/prosody/Makefile
Comment 5 Mark Linimon freebsd_committer freebsd_triage 2018-12-20 06:27:57 UTC
Committed Wed Dec 19 18:45:39 UTC 2018.