From owner-freebsd-ports@FreeBSD.ORG Sat Feb 11 16:31:41 2012 Return-Path: Delivered-To: freebsd-ports@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id D79EC106564A for ; Sat, 11 Feb 2012 16:31:41 +0000 (UTC) (envelope-from avg@FreeBSD.org) Received: from citadel.icyb.net.ua (citadel.icyb.net.ua [212.40.38.140]) by mx1.freebsd.org (Postfix) with ESMTP id 2B5418FC14 for ; Sat, 11 Feb 2012 16:31:40 +0000 (UTC) Received: from porto.starpoint.kiev.ua (porto-e.starpoint.kiev.ua [212.40.38.100]) by citadel.icyb.net.ua (8.8.8p3/ICyb-2.3exp) with ESMTP id SAA01507; Sat, 11 Feb 2012 18:31:38 +0200 (EET) (envelope-from avg@FreeBSD.org) Received: from localhost ([127.0.0.1]) by porto.starpoint.kiev.ua with esmtp (Exim 4.34 (FreeBSD)) id 1RwFrN-000DLS-OI; Sat, 11 Feb 2012 18:31:37 +0200 Message-ID: <4F3697E7.2050207@FreeBSD.org> Date: Sat, 11 Feb 2012 18:31:35 +0200 From: Andriy Gapon User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:10.0) Gecko/20120202 Thunderbird/10.0 MIME-Version: 1.0 To: Alexander Logvinov References: <4F3101C3.8070209@FreeBSD.org> In-Reply-To: X-Enigmail-Version: 1.3.5 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Cc: freebsd-ports@FreeBSD.org Subject: Re: net/freerdp 1.0.0 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: Sat, 11 Feb 2012 16:31:41 -0000 on 10/02/2012 06:56 Alexander Logvinov said the following: > 2012/2/10 Alexander Logvinov : >> I don't think we have to create a new port. A new version of >> net/remmina coming soon with FreeRDP 1.0 support. Therefore, we should >> wait for it. > Here is 1.0.1 version (not fully tested) with SSE2 option from your > port: http://people.freebsd.org/~avl/misc/freerdp-1.0.1.tar . Thanks! > Great! Thank you very much! Just a few comments: o IMO, it might make sense to swap default setting for PULSEAUDIO and ALSA - ALSA is just a single library of wrapper code for OSS while PULSEAUDIO involves a daemon, some potentially non-trivial configuration, etc. So for packages ALSA looks like a better choice. o In my version of port update I decided to not use -L${LOCALBASE}/lib in LDFLAGS and -I${LOCALBASE}/include in CFLAGS. I also decided to not use various -DXXX_INCLUDE_DIR:PATH flags. The reason is that I believe that the cmake detection code should properly work on FreeBSD and if it doesn't work, then it's a bug. So I decided to use some patches and I intend to report all the problems upstream: http://public.kitware.com/pipermail/cmake-developers/2012-February/003119.html http://sourceforge.net/mailarchive/forum.php?thread_name=4F3013BE.9080707%40icyb.net.ua&forum_name=freerdp-devel o Also, it looks like the following bug related to SSE2 on amd64 has not been fixed yet: http://sourceforge.net/mailarchive/forum.php?thread_name=4F344225.7060202%40icyb.net.ua&forum_name=freerdp-devel -- Andriy Gapon