From owner-freebsd-ports-bugs@freebsd.org Wed Nov 4 15:31:59 2015 Return-Path: Delivered-To: freebsd-ports-bugs@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 6D325A2698B for ; Wed, 4 Nov 2015 15:31:59 +0000 (UTC) (envelope-from bugzilla-noreply@freebsd.org) Received: from kenobi.freebsd.org (kenobi.freebsd.org [IPv6:2001:1900:2254:206a::16:76]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 4CF111BB7 for ; Wed, 4 Nov 2015 15:31:59 +0000 (UTC) (envelope-from bugzilla-noreply@freebsd.org) Received: from bugs.freebsd.org ([127.0.1.118]) by kenobi.freebsd.org (8.15.2/8.15.2) with ESMTP id tA4FVx4Y051475 for ; Wed, 4 Nov 2015 15:31:59 GMT (envelope-from bugzilla-noreply@freebsd.org) From: bugzilla-noreply@freebsd.org To: freebsd-ports-bugs@FreeBSD.org Subject: [Bug 204282] [NEW PORT]: devel/gecode3: gecode version 3.7.3 is needed for compatibility for dep-selector-libgecode rubygem. Date: Wed, 04 Nov 2015 15:31:59 +0000 X-Bugzilla-Reason: AssignedTo CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: Ports & Packages X-Bugzilla-Component: Individual Port(s) X-Bugzilla-Version: Latest X-Bugzilla-Keywords: X-Bugzilla-Severity: Affects Some People X-Bugzilla-Who: danfe@FreeBSD.org X-Bugzilla-Status: In Progress X-Bugzilla-Priority: --- X-Bugzilla-Assigned-To: danfe@FreeBSD.org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: bug_status cc assigned_to Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit X-Bugzilla-URL: https://bugs.freebsd.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 X-BeenThere: freebsd-ports-bugs@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: Ports bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 04 Nov 2015 15:31:59 -0000 https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=204282 Alexey Dokuchaev changed: What |Removed |Added ---------------------------------------------------------------------------- Status|New |In Progress CC| |danfe@FreeBSD.org Assignee|freebsd-ports-bugs@FreeBSD. |danfe@FreeBSD.org |org | --- Comment #3 from Alexey Dokuchaev --- > # Created by: mokhi64@gmail.com While not required, it's better (more consistent with other ports) to provide fully-qualified email address and $FreeBSD$ id tag: > # Created by: John Doe > $FreeBSD$ Portlint should catch this error by the way, but you already know this. ;-) > COMMENT= Generic Constraint Development Environment (Needed for dep-selector-libgecode and berkshelf) COMMENT line is too long (again, portlint(1) should've caught that) and the the part in parenthesis is useless, as are excessive capitalizations. Correct COMMENT should be something like: > COMMENT= Generic constraint development environment Here, BUILD_DEPENDS typically come before LIB_DEPENDS: > LIB_DEPENDS= libmpfr.so:${PORTSDIR}/math/mpfr > BUILD_DEPENDS= bash:${PORTSDIR}/shells/bash Also, usually build-depends on bash(1) can be easily fixed as it often due to linuxisms, not the actual bash(1) features that to do map easily to BSD sh(1) syntax. > CONFIGURE_ARGS= [long list of options] Are they really all needed? Typically you'd only want to change the defaults, not list them all. ;-) > MAKE_JOBS_UNSAFE=yes Did you try to fix it so it becomes jobs-safe? > SHLIB= 37 > ... > PLIST_SUB= SHLIB="${SHLIB}" SHLIB variable is used only once (quoted above). Consider doing this instead: > PLIST_SUB= SHLIB=PORTVERSION:R In this place in Makefile, knobs are also not well sorted (style bug). > GIST_CONFIGURE_ON= --enable-gist --enable-qt > GIST_CONFIGURE_OFF= --disable-gist --disable-qt Consider using GIST_CONFIGURE_ENABLE knob(s) instead. > .include > > .if ${ARCH} == "amd64" > CFLAGS+= -D__USE_ISOC99 > .endif Setting CFLAGS_amd64 would DTRT and you'd also won't have to include here. > post-install: > @${STRIP_CMD} ${STAGEDIR}${PREFIX}/bin/fz > # @${STRIP_CMD} ${STAGEDIR}${PREFIX}/lib/libgecode*.so.[0-9]*.[0-9]* Why the last line is commented? Also, consider using INSTALL_TARGET=install-strip if configure script supports it. In pkg-plist: @dirrmtry lines are not required anymore. Now, about the patch: > --- configure.orig > +++ configure > @@ -1,4 +1,4 @@ > -#! /bin/sh > +#! /usr/bin/env bash > # From configure.ac Id: configure.ac.in 12614 2012-03-22 01:42:32Z tack . > # Guess values for system-dependent variables and create Makefiles. > # Generated by GNU Autoconf 2.68 for GECODE 3.7.3. What's wrong with /bin/sh? Why this port needs bash(1)? -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug.