From owner-freebsd-ports@FreeBSD.ORG Sun Jan 16 21:02:31 2011 Return-Path: Delivered-To: ports@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id B0D20106564A for ; Sun, 16 Jan 2011 21:02:31 +0000 (UTC) (envelope-from jhelfman@experts-exchange.com) Received: from mail.experts-exchange.com (mail.experts-exchange.com [72.29.183.251]) by mx1.freebsd.org (Postfix) with ESMTP id 8C8A68FC0C for ; Sun, 16 Jan 2011 21:02:31 +0000 (UTC) Received: from mail.experts-exchange.com (localhost [127.0.0.1]) by mail.experts-exchange.com (Postfix) with ESMTP id 8B7881266255; Sun, 16 Jan 2011 12:47:27 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=e-e.com; h= user-agent:in-reply-to:content-transfer-encoding :content-disposition:content-type:content-type:mime-version :references:message-id:subject:subject:from:from:date:date :received:received:received; s=ee; t=1295210847; x=1297025247; bh=fBuONV951B6AaB9Yk8r0bj/J2Fbip4jilWN8BmIGVO8=; b=Fu9H60epqB94 9jgmloDVIpuLfRhHnAGt+tUEv8RQxtPPz1zwNiVh6nrGjdFTBd6xM6mkTTJT7nvY wvy9PCMiKQ0E7OfGpgBMpKDx3+HN+QGn8EePewZO4pAZwPdXgQo1FRRjFbWRmWIm 4GDDJESEiiIQnk9d7VWHKvQG6WmIkTg= X-Virus-Scanned: amavisd-new at experts-exchange.com Received: from mail.experts-exchange.com ([127.0.0.1]) by mail.experts-exchange.com (mail.experts-exchange.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id qCnww+lTG5zv; Sun, 16 Jan 2011 12:47:27 -0800 (PST) Received: from experts-exchange.com (unknown [192.168.103.122]) by mail.experts-exchange.com (Postfix) with SMTP id 3B2DC126624E; Sun, 16 Jan 2011 12:47:27 -0800 (PST) Received: (nullmailer pid 1794 invoked by uid 1001); Sun, 16 Jan 2011 20:44:16 -0000 Date: Sun, 16 Jan 2011 12:44:16 -0800 From: Jason Helfman To: Ruslan Mahmatkhanov Message-ID: <20110116204416.GA1654@eggman.experts-exchange.com> References: <11111295127072@web122.yandex.ru> <4D3329FE.7020901@yandex.ru> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <4D3329FE.7020901@yandex.ru> X-Operating-System: FreeBSD 8.1-RELEASE X-Living-The-Dream: I love the SLO Life! User-Agent: Mutt/1.5.21 (2010-09-15) Cc: ports@freebsd.org Subject: Re: New port needs review: net/erlyvideo X-BeenThere: freebsd-ports@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Porting software to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 16 Jan 2011 21:02:31 -0000 On Sun, Jan 16, 2011 at 08:25:18PM +0300, Ruslan Mahmatkhanov thus spake: >16.01.2011 00:31, Ruslan Mahmatkhanov пишет: >> Hello! >> >> I'm finally completed with port of erlyvideo (see erlyvideo.org for details). But before i'll post PR, i want that somebody review it. Because i'm sure there is something that can be made better. I'm attached the diff to make commenting more convenient but you can also download port directory if you prefer: >> http://happy-nation.by.ru/ports/tb/erlyvideo.tgz >> >> Thanks in advance for any suggestions, critics, corrections. > >Just in case anybody else noticed it, there was MASTER_SITES fetch error >in original tarball. It was pointed out by Kurt Jaeger. Now it was fixed >and new tarball reupploaded. There is also updated diff-file: >http://happy-nation.by.ru/ports/erlyvideo.diff > >There is also short usage manual: >- install the port >- cp /usr/local/etc/erlyvideo/*.conf[ig].sample into just *.conf[ig] >- then do `erlyctl start` >- place some mpeg file into /var/erlyvideo/movies >- go to http://localhost:8082/flowplayer/ with flash-aware web-browser >- click play button >- you should see your mpeg movie packed to flv and transferred to you >via rtmp > >More complicated usage scenarios (like RTSP H.264 cameras, FLME etc) is >available on http://erlyvideo.org/docs. > >-- >Regards, >Ruslan Just a couple of notes, so far... The comment reads redundantly, and is too long so it will not display properly when listing with pkg_info: +COMMENT= Erlyvideo is an RTMP flash streaming server, written in erlang Perhaps consider this: RMTP flash streaming server written in erlang It appears that git is required for the program: --- erlyvideo.orig/files/patch-Makefile 1970-01-01 03:00:00.000000000 +0300 +++ erlyvideo/files/patch-Makefile 2011-01-15 15:03:24.000000000 +0300 @@ -0,0 +1,21 @@ +--- Makefile.orig 2010-12-16 00:51:55.000000000 +0300 ++++ Makefile 2011-01-15 15:03:11.000000000 +0300 +@@ -19,7 +19,6 @@ + deps/amf: update_deps + + update_deps: rebar.config +- [ -d wwwroot/player ] || git clone git://github.com/erlyvideo/erlyplayer wwwroot/player + ./rebar get-deps + + However there is no dependency for git, if I am interpretting this correctly. If this updates, or pulls down code during installation, you may wish to package it, or not do this step. I don't believe post-fetch fetching or updating of files to build a port is allowed through pointyhat. Others can speak to that, though. You may wish to consider a for loop on the make install target. Are these directories created during the install process of the package? I didn't see these directories created outside of the Makefile. I noticed many '.gitignore' files in the pkg-plist. Is this necessary? I've never worked with git before, or this application. When submitting this as a port through the PR system, I would suggest using the shar utility: http://www.freebsd.org/doc/en_US.ISO8859-1/books/porters-handbook/porting-submitting.html -jgh -- Jason Helfman System Administrator experts-exchange.com http://www.experts-exchange.com/M_4830110.html E4AD 7CF1 1396 27F6 79DD 4342 5E92 AD66 8C8C FBA5