From owner-svn-ports-head@freebsd.org Sun Feb 18 17:21:49 2018 Return-Path: Delivered-To: svn-ports-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 12151F19E2A; Sun, 18 Feb 2018 17:21:49 +0000 (UTC) (envelope-from yuri@freebsd.org) Received: from shell1.rawbw.com (shell1.rawbw.com [198.144.192.42]) by mx1.freebsd.org (Postfix) with ESMTP id 9297980B9F; Sun, 18 Feb 2018 17:21:48 +0000 (UTC) (envelope-from yuri@freebsd.org) Received: from yv.noip.me (c-24-4-131-132.hsd1.ca.comcast.net [24.4.131.132]) (authenticated bits=0) by shell1.rawbw.com (8.15.1/8.15.1) with ESMTPSA id w1IHLc3a003798 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=NO); Sun, 18 Feb 2018 09:21:39 -0800 (PST) (envelope-from yuri@freebsd.org) X-Authentication-Warning: shell1.rawbw.com: Host c-24-4-131-132.hsd1.ca.comcast.net [24.4.131.132] claimed to be yv.noip.me Reply-To: yuri@freebsd.org Subject: Re: svn commit: r462195 - in head/multimedia: ffmpeg kodi kodi-devel libbluray libxine mplayer mpv vdr-plugin-xineliboutput vlc To: Tijl Coosemans Cc: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org References: <201802172145.w1HLjURm079504@repo.freebsd.org> <20180218145421.1c7dfa2c@kalimero.tijl.coosemans.org> From: Yuri Message-ID: <0ecf6c26-300c-0e69-951f-2d7e2a2e6b87@freebsd.org> Date: Sun, 18 Feb 2018 09:21:37 -0800 User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 MIME-Version: 1.0 In-Reply-To: <20180218145421.1c7dfa2c@kalimero.tijl.coosemans.org> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Content-Language: en-US X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.25 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, 18 Feb 2018 17:21:49 -0000 On 02/18/18 05:54, Tijl Coosemans wrote: > Can you put this in post-patch? extract should give a clean copy of > upstream source. Ok, thanks, will correct this. > I would also use plain "@${LN} -s linux ${WRKSRC}/..." > here instead of RLN, but maybe that's a personal thing. I was using plain "@${LN}" before, but then somebody suggested to use "${RLN}" instead, and I actually find it more readable now. But you are right, this is a matter of personal preference. Yuri