Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 30 Jul 2019 12:43:04 +0000 (UTC)
From:      Piotr Kubaj <pkubaj@FreeBSD.org>
To:        ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   svn commit: r507639 - in head/devel/py-llfuse: . files files/files
Message-ID:  <201907301243.x6UCh4gl068221@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: pkubaj
Date: Tue Jul 30 12:43:04 2019
New Revision: 507639
URL: https://svnweb.freebsd.org/changeset/ports/507639

Log:
  devel/py-llfuse: Fix build (warning=error), remove -Werror
  
  -Werror is added to flags by default, breaking build for example with GCC.
  
  Follow upstream and remove it. Also disable DEVELOPER_MODE since it's enabled too widely.
  
  PR:		238887
  Approved by:	portmgr (blanket(s): build fix, ports compliance), linimon (mentor)
  Differential Revision:	https://reviews.freebsd.org/D21067

Added:
  head/devel/py-llfuse/files/files/
  head/devel/py-llfuse/files/files/patch-setup.py   (contents, props changed)
  head/devel/py-llfuse/files/patch-setup.py   (contents, props changed)
Modified:
  head/devel/py-llfuse/Makefile

Modified: head/devel/py-llfuse/Makefile
==============================================================================
--- head/devel/py-llfuse/Makefile	Tue Jul 30 12:36:19 2019	(r507638)
+++ head/devel/py-llfuse/Makefile	Tue Jul 30 12:43:04 2019	(r507639)
@@ -12,8 +12,6 @@ COMMENT=	Python bindings for low-level FUSE API
 
 LICENSE=	LGPL20+
 
-BROKEN_powerpc64=	Does not build: error: conversion to mode_t from int may alter its value
-
 USES=		compiler:c++11-lang fuse pkgconfig python tar:bz2
 USE_PYTHON=	autoplist distutils
 

Added: head/devel/py-llfuse/files/files/patch-setup.py
==============================================================================
--- /dev/null	00:00:00 1970	(empty, because file is newly added)
+++ head/devel/py-llfuse/files/files/patch-setup.py	Tue Jul 30 12:43:04 2019	(r507639)
@@ -0,0 +1,36 @@
+--- setup.py.orig	2017-01-22 22:24:10 UTC
++++ setup.py
+@@ -80,24 +80,24 @@ def main():
+     # We may have unused functions if we compile for older FUSE versions
+     compile_args.append('-Wno-unused-function')
+ 
+-    # Value-changing conversions should always be explicit.
+-    compile_args.append('-Werror=conversion')
+-
+-    # Note that (i > -1) is false if i is unsigned (-1 will be converted to
+-    # a large positive value). We certainly don't want to do this by
+-    # accident.
+-    compile_args.append('-Werror=sign-compare')
+-
+     # Enable all fatal warnings only when compiling from Mercurial tip.
+     # (otherwise we break forward compatibility because compilation with newer
+     # compiler may fail if additional warnings are added)
+-    if DEVELOPER_MODE:
++    if 0:
+         compile_args.append('-Werror')
+         compile_args.append('-Wfatal-errors')
+ 
+         # Unreachable code is expected because we need to support multiple
+         # platforms and architectures.
+         compile_args.append('-Wno-error=unreachable-code')
++
++        # Value-changing conversions should always be explicit.
++        compile_args.append('-Werror=conversion')
++
++        # Note that (i > -1) is false if i is unsigned (-1 will be converted to
++        # a large positive value). We certainly don't want to do this by
++        # accident.
++        compile_args.append('-Werror=sign-compare')
+ 
+     # http://bugs.python.org/issue7576
+     if sys.version_info[0] == 3 and sys.version_info[1] < 2:

Added: head/devel/py-llfuse/files/patch-setup.py
==============================================================================
--- /dev/null	00:00:00 1970	(empty, because file is newly added)
+++ head/devel/py-llfuse/files/patch-setup.py	Tue Jul 30 12:43:04 2019	(r507639)
@@ -0,0 +1,36 @@
+--- setup.py.orig	2017-01-22 22:24:10 UTC
++++ setup.py
+@@ -80,24 +80,24 @@ def main():
+     # We may have unused functions if we compile for older FUSE versions
+     compile_args.append('-Wno-unused-function')
+ 
+-    # Value-changing conversions should always be explicit.
+-    compile_args.append('-Werror=conversion')
+-
+-    # Note that (i > -1) is false if i is unsigned (-1 will be converted to
+-    # a large positive value). We certainly don't want to do this by
+-    # accident.
+-    compile_args.append('-Werror=sign-compare')
+-
+     # Enable all fatal warnings only when compiling from Mercurial tip.
+     # (otherwise we break forward compatibility because compilation with newer
+     # compiler may fail if additional warnings are added)
+-    if DEVELOPER_MODE:
++    if 0:
+         compile_args.append('-Werror')
+         compile_args.append('-Wfatal-errors')
+ 
+         # Unreachable code is expected because we need to support multiple
+         # platforms and architectures.
+         compile_args.append('-Wno-error=unreachable-code')
++
++        # Value-changing conversions should always be explicit.
++        compile_args.append('-Werror=conversion')
++
++        # Note that (i > -1) is false if i is unsigned (-1 will be converted to
++        # a large positive value). We certainly don't want to do this by
++        # accident.
++        compile_args.append('-Werror=sign-compare')
+ 
+     # http://bugs.python.org/issue7576
+     if sys.version_info[0] == 3 and sys.version_info[1] < 2:



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201907301243.x6UCh4gl068221>