From owner-freebsd-ports-bugs@FreeBSD.ORG Sat Mar 11 02:20:24 2006 Return-Path: X-Original-To: freebsd-ports-bugs@hub.freebsd.org Delivered-To: freebsd-ports-bugs@hub.freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 7ECCB16A42C for ; Sat, 11 Mar 2006 02:20:24 +0000 (GMT) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [216.136.204.21]) by mx1.FreeBSD.org (Postfix) with ESMTP id A7D8F43D55 for ; Sat, 11 Mar 2006 02:20:23 +0000 (GMT) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (gnats@localhost [127.0.0.1]) by freefall.freebsd.org (8.13.4/8.13.4) with ESMTP id k2B2KNp1060249 for ; Sat, 11 Mar 2006 02:20:23 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.13.4/8.13.4/Submit) id k2B2KNxB060248; Sat, 11 Mar 2006 02:20:23 GMT (envelope-from gnats) Date: Sat, 11 Mar 2006 02:20:23 GMT Message-Id: <200603110220.k2B2KNxB060248@freefall.freebsd.org> To: freebsd-ports-bugs@FreeBSD.org From: Ion-Mihai Tetcu Cc: Subject: Re: ports/94300: [NEW PORT]: net-im/sim-im-devel: Plugin-based instant messenger X-BeenThere: freebsd-ports-bugs@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: Ion-Mihai Tetcu List-Id: Ports bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 11 Mar 2006 02:20:24 -0000 The following reply was made to PR ports/94300; it has been noted by GNATS. From: Ion-Mihai Tetcu To: "Alexey V. Panfilov" Cc: FreeBSD-gnats-submit@FreeBSD.org Subject: Re: ports/94300: [NEW PORT]: net-im/sim-im-devel: Plugin-based instant messenger Date: Sat, 11 Mar 2006 04:12:31 +0200 On Fri, 10 Mar 2006 17:15:10 +0300 (MSK) "Alexey V. Panfilov" wrote: > > >Number: 94300 > >Category: ports > >Synopsis: [NEW PORT]: net-im/sim-im-devel: Plugin-based instant messenger [ ... ] Thanks for your work. > sed 's/^X//' >sim-im-devel/Makefile << 'END-of-sim-im-devel/Makefile' > X# New Ports Collection Makefile for: sim-im-devel > X# Date created: 3 Mar 2006 > X# Whom: Alexey V. Panfilov > X# > X# $FreeBSD$ > X > XPORTNAME= sim-im-devel > XPORTVERSION= 0.9.5.377 > XPORTREVISION= 1 Why do you set PORTREVISION for a new port ? > XCATEGORIES= net-im kde You're defaulting KDE to off, so kde shouldn't be in CATEGORIES. > XMASTER_SITES= http://sim-im.lehis.ru/dist/ > X > XMAINTAINER= sim-im@lehis.ru > XCOMMENT= Plugin-based instant messenger > X > XLIB_DEPENDS= xslt.2:${PORTSDIR}/textproc/libxslt > X > XUSE_BZIP2= yes > XUSE_GMAKE= yes > XUSE_QT_VER= 3 > XUSE_PERL5_BUILD= yes > XGNU_CONFIGURE= yes > XREINPLACE_CMD= yes This is wrong. You probably thought about USE_REINPLEACE= yes, which is not needed anymore; more, you don't use this anywhere. [ ... ] > XPORT_DBDIR?= /var/db/ports > XLATEST_LINK= ${PORTNAME}${PKGNAMESUFFIX} > XOPTIONSFILE?= ${PORT_DBDIR}/${LATEST_LINK}/options > X > X.if exists(${OPTIONSFILE}) > X.include "${OPTIONSFILE}" > X.endif Do you really need this block ? Is there any reason you can't use ..include after OPTIONS= ... and before checking WITH* and .include at the end of the file instead of .include ? > X.if defined(WITH_DEB) There is a standard knob for debug called DEBUG (please see /usr/ports/KNOBS). > XCONFIGURE_ARGS+=--enable-debug > X.endif > X.if defined(WITH_SSL) > XUSE_OPENSSL= yes > XPLIST_SUB+= SSL="" > X.else > XCONFIGURE_ARGS+=--disable-openssl > XPLIST_SUB+= SSL="@comment " > X.endif > X.if !defined(WITH_KDE) > XCONFIGURE_ARGS+=--disable-kde --enable-mt > XLIB_DEPENDS+= aspell.16:${PORTSDIR}/textproc/aspell > XPLIST_SUB+= KDE="@comment " \ > X SPELL="" > X.else > XUSE_KDELIBS_VER=3 > XPLIST_SUB+= KDE="" \ > X SPELL="@comment " > X.endif > X > Xpre-everything:: > X.if defined(WITH_SSL) > X @${ECHO_MSG} "You can disable SSL support by defining WITHOUT_SSL." > X.endif > X.if defined(WITH_KDE) > X @${ECHO_MSG} "You can disable KDE support by defining WITHOUT_KDE." > X.endif These two echo are redundant with the OPTIONS screen, no ? Please run portlint(1) before submitting a new port or an update. Thanks again for your work on porting this. -- IOnut - Unregistered ;) FreeBSD "user" "Intellectual Property" is nowhere near as valuable as "Intellect" There is a multi-legged creature crawling on your shoulder. -- Spock, "A Taste of Armageddon", stardate 3193.9