From owner-svn-ports-head@FreeBSD.ORG Sun Oct 7 09:45:26 2012 Return-Path: Delivered-To: svn-ports-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 44A221065676; Sun, 7 Oct 2012 09:45:26 +0000 (UTC) (envelope-from alexkozlov0@gmail.com) Received: from mail-we0-f182.google.com (mail-we0-f182.google.com [74.125.82.182]) by mx1.freebsd.org (Postfix) with ESMTP id 752FE8FC0A; Sun, 7 Oct 2012 09:45:24 +0000 (UTC) Received: by mail-we0-f182.google.com with SMTP id x43so2485494wey.13 for ; Sun, 07 Oct 2012 02:45:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=sender:date:from:to:cc:subject:message-id:references:mime-version :content-type:content-disposition:in-reply-to; bh=XAynU4B2v40awVJ4OzWff+Y4FzGfwL1oKJ4ODOhUn8s=; b=drCRdjzUQgSnfQKYbQWQEd1WgEHcSp6pcBkefTVl7albGp96Iwb1yNP70JZ5QB3E/f orX2CVc8KIh3PSI8usliSJEjPUkS7mM9V8RPokvKUiuC+8Z986W7N5ynp43UxY6yePiW gg7v6G4EPr9FyuVmguCc9zAiLDdv+Ir0lz/zKgRHHQB7g4ykPaaX2MvmopSA3ugAEeM2 NWeYbfKBW5snLsOyee7WuV9ZxxO639izyzUWbPu2zH7ACRbsfa+IGlhbspZ8Mv4Do9gP 6ewSioFUeKP1O3ox3DrhwBjDXXOG827sX/fmh494uKNmyqcGrVpSXdGzfB7tbVt6eA0K FeXg== Received: by 10.180.95.97 with SMTP id dj1mr13902180wib.3.1349603123438; Sun, 07 Oct 2012 02:45:23 -0700 (PDT) Received: from ravenloft.kiev.ua (ravenloft.kiev.ua. [94.244.131.95]) by mx.google.com with ESMTPS id p4sm14672101wix.0.2012.10.07.02.45.21 (version=TLSv1/SSLv3 cipher=OTHER); Sun, 07 Oct 2012 02:45:23 -0700 (PDT) Sender: Alex Kozlov Date: Sun, 7 Oct 2012 12:45:19 +0300 From: Alex Kozlov To: Boris Samorodov Message-ID: <20121007094519.GA23625@ravenloft.kiev.ua> References: <201210062105.q96L5Dth040796@svn.freebsd.org> <50713F44.5010603@passap.ru> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <50713F44.5010603@passap.ru> Cc: svn-ports-head@freebsd.org, svn-ports-all@freebsd.org, ports-committers@freebsd.org Subject: Re: svn commit: r305403 - head/graphics/eos-movrec X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for the ports tree for head List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 07 Oct 2012 09:45:26 -0000 On Sun, Oct 07, 2012 at 12:37:24PM +0400, Boris Samorodov wrote: > Hello Alex, > > > Log: > > - Convert Makefile headers to new style > > - Add DESKTOP_ENTRIES > > - Pass maintainership to nemysis > As for me this is too simplified commit log. May be added: > > - Use bzip2 instead of zip tarball (greatly reduses the size) > - Remove duplicate PORTREVISION > - Remove the indefinite article from COMMENT > - Remove shlib version from the dependency > - convert and install icon *.png from *.ico Apologies, I forgot to update message file. Convert and install icon included in add DESKTOP_ENTRIES though, desktop entry need to have an icon. > And those changes are definitely demand a PORTREVISION bump. > There are some notes inline. > > > PR: ports/172342 > Since you committed a different patch it's better to point this out. The patch was substantially improved after discussion on irc, but it's still (technically) submitted by pr author. I guess I could have used something like: PR: ports/172342 (based on) > > Submitted by: nemysis > > > > Modified: > > head/graphics/eos-movrec/Makefile (contents, props changed) > > head/graphics/eos-movrec/distinfo (contents, props changed) > > > > Modified: head/graphics/eos-movrec/Makefile > > ============================================================================== > > --- head/graphics/eos-movrec/Makefile Sat Oct 6 20:50:12 2012 (r305402) > > +++ head/graphics/eos-movrec/Makefile Sat Oct 6 21:05:12 2012 (r305403) > > @@ -1,34 +1,42 @@ > > -# New ports collection makefile for: EOS camera movie record > > -# Date created: 15 Aug 2011 > > -# Whom: Alexey Dokuchaev > > -# > > # $FreeBSD$ > > -# > > +# Created by: Alexey Dokuchaev > Imho this line should be the first one. At the moment there are three or four variants of Makefile headers in the tree. I try to be as close as possible to the official version: http://www.freebsd.org/doc/en_US.ISO8859-1/books/porters-handbook/porting-samplem.html > > PORTNAME= eos-movrec > > DISTVERSION= 0.3.1_beta > > PORTREVISION= 1 > > CATEGORIES= graphics > I'm not sure, but may be this port belongs to multimedia? Hm. I'm not sure either, will ask maintainer. > > -MASTER_SITES= SF/${PORTNAME}/${PORTNAME}/0.3_beta > > +MASTER_SITES= SF/${PORTNAME}/${PORTNAME}/0.3_beta/ > > > > -MAINTAINER= ports@FreeBSD.org > > -COMMENT= A program to capture short movies with Canon DSLR camera > > +MAINTAINER= nemysis@gmx.ch > > +COMMENT= Program to capture short movies with Canon DSLR camera > May be this is a matter of taste but I'll say that "Program to" may be > trimmed too. > > LICENSE= GPLv2 > > > > -LIB_DEPENDS= gphoto2.2:${PORTSDIR}/graphics/libgphoto2 > > +LIB_DEPENDS= gphoto2:${PORTSDIR}/graphics/libgphoto2 > > +BUILD_DEPENDS= pnmtopng:${PORTSDIR}/graphics/netpbm > > > > -USE_ZIP= yes > > +USE_BZIP2= yes > > USE_CMAKE= yes > > +CMAKE_VERBOSE= yes > Seems to be a diagnistic only tool. Is it necessary for every build? I don't see any harm in it. But I will ask the maintainer if he agrees to remove it. > > +post-build: > > + @(cd ${WRKSRC} && ${LOCALBASE}/bin/winicontoppm -bestqual eos-movrec2.ico | \ > > + ${LOCALBASE}/bin/pnmtopng > eos-movrec.png) > > This is an interesting trick. Is it documented anywhere? No. You need to read netpbm documentation online. -- Alex