From owner-freebsd-multimedia@FreeBSD.ORG Mon Feb 26 16:13:22 2007 Return-Path: X-Original-To: multimedia@freebsd.org Delivered-To: freebsd-multimedia@FreeBSD.ORG Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 0718316A400 for ; Mon, 26 Feb 2007 16:13:22 +0000 (UTC) (envelope-from buhnux@gmail.com) Received: from nf-out-0910.google.com (nf-out-0910.google.com [64.233.182.190]) by mx1.freebsd.org (Postfix) with ESMTP id 8878013C442 for ; Mon, 26 Feb 2007 16:13:21 +0000 (UTC) (envelope-from buhnux@gmail.com) Received: by nf-out-0910.google.com with SMTP id k27so1545607nfc for ; Mon, 26 Feb 2007 08:13:20 -0800 (PST) DKIM-Signature: a=rsa-sha1; c=relaxed/relaxed; d=gmail.com; s=beta; h=domainkey-signature:received:received:message-id:date:from:sender:to:subject:cc:in-reply-to:mime-version:content-type:references:x-google-sender-auth; b=jVfDmWmeiAgx3S+dk5mXqKWoto+jvuM7NjJFqVGSctY6vBuIdW9wPze+HpXeHN0A3S+8OCG4v4gwmAPwUlp2fMYqGTCvzY+Q/2x7DsS3es1CJ+lU2AztLdiWo7nqTIVvOsoTK84cYMWiDNFVnTIrlbIKBXZvkTfgyJeN8Y3FI0o= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=beta; h=received:message-id:date:from:sender:to:subject:cc:in-reply-to:mime-version:content-type:references:x-google-sender-auth; b=pOAlRwIT8/IIaD2b4k5tCYlpDNikfRKjCYUIBOb6bF+I0niLfpS4BezqvL1i+jZq81yTo4Qz4oYywX5Y16DKJbsyErXKJTMCYFL9tby17XNFqZRLvu3NrHN+bJMkrGsKBDuHNBiyQTTcyVjjg4sMzHDwb+KICDiSbnAM0IKZi7I= Received: by 10.82.187.16 with SMTP id k16mr1798279buf.1172506399916; Mon, 26 Feb 2007 08:13:19 -0800 (PST) Received: by 10.82.112.15 with HTTP; Mon, 26 Feb 2007 08:13:19 -0800 (PST) Message-ID: Date: Mon, 26 Feb 2007 11:13:19 -0500 From: "Michael Johnson" Sender: buhnux@gmail.com To: "Mikhail Teterin" In-Reply-To: <200702260942.27062@aldan> MIME-Version: 1.0 References: <200702260942.27062@aldan> X-Google-Sender-Auth: 34621668983b7064 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Content-Disposition: inline X-Content-Filtered-By: Mailman/MimeDel 2.1.5 Cc: multimedia@freebsd.org, mezz@freebsd.org Subject: Re: improving vlc-devel X-BeenThere: freebsd-multimedia@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Multimedia discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 26 Feb 2007 16:13:22 -0000 On 2/26/07, Mikhail Teterin wrote: > > Hello! > > Attached is a patch, that fixes two minor annoyances with the port: > > . removes "BROKEN" on amd64 -- just built it fine on my system; > . removes explicit requirements for shared libraries' major > numbers. > > There are several commits in the Makefile's cvs log, which do nothing > but "chase" these numbers -- that's a foolish excercise. Vlc (and most > other > software) almost never care for a particular version of a dependency and > the > shlib major number should only be specified explicitly, when needed -- > ommited _by default_ (vlc's own configure-script is better at detecting > such > incompatibilities anyway). That's not the reason we use LibName.LibVersion in VLC. VLC and many other programs hard link to LibName.LibVersion, we keep the LibVersion in there so it reduces the amount of bug reports we get. Here is an example if libx264.so.50 changes to libx264.so.99 (I chose x264 since we patch it to control the LibVersion number to reduce the number of rebuilds on VLC and mplayer) root@blueherron /usr/ports/multimedia/x264 > mplayer -1 MPlayer 1.0rc1-3.4.6 (C) 2000-2006 MPlayer Team CPU: Unknown CPU Type (Family: 6, Model: 10, Stepping: 0) CPUflags: MMX: 1 MMX2: 1 3DNow: 1 3DNow2: 1 SSE: 1 SSE2: 0 Compiled with runtime CPU detection. Unknown option on the command line: -1 root@blueherron /usr/ports/multimedia/x264 > ldconfig -r|grep x264.so 426:-lx264.50 => /usr/local/lib/libx264.so.50 <<<>>> root@blueherron /usr/ports/multimedia/x264 > ldconfig -r|grep x264.so 426:-lx264.99 => /usr/local/lib/libx264.so.99 root@blueherron /usr/ports/multimedia/x264 > mplayer /libexec/ld-elf.so.1: Shared object "libx264.so.50" not found, required by "mplayer" even though this is mplayer, vlc does the same thing. I'm open to a patch that makes VLC link to LibName.so instead of LibName.so.LibVersion. Michael Doing otherwise requires too many rebuilds to make sense -- in order to > build > a fresh version of vlc, for example, one has to rebuild gaim too, because > vlc > explicitly requires gnutls.15, whereas one's existing gaim install may > refer > to gnutls.12... > > Please, consider the attached patch. Thank you very much for your time, > > -mi > >