From mboxrd@z Thu Jan  1 00:00:00 1970
From: Mark H Weaver <mhw@netris.org>
Subject: Re: [Patch] gst-plugins-base
Date: Thu, 31 Mar 2016 13:17:42 -0400
Message-ID: <877fgi1s95.fsf@netris.org>
References: <20160330183809.29bd3b17@debian-netbook>
	<87shz71vlh.fsf@netris.org> <87lh4z1vej.fsf@netris.org>
	<20160331072216.GE7964@debian-netbook>
Mime-Version: 1.0
Content-Type: text/plain
Return-path: <guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org>
Received: from eggs.gnu.org ([2001:4830:134:3::10]:41620)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <mhw@netris.org>) id 1algEa-0000Ws-F0
	for guix-devel@gnu.org; Thu, 31 Mar 2016 13:18:17 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <mhw@netris.org>) id 1algEX-0005Ie-7y
	for guix-devel@gnu.org; Thu, 31 Mar 2016 13:18:16 -0400
Received: from world.peace.net ([50.252.239.5]:42641)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <mhw@netris.org>) id 1algEX-0005HG-3p
	for guix-devel@gnu.org; Thu, 31 Mar 2016 13:18:13 -0400
In-Reply-To: <20160331072216.GE7964@debian-netbook> (Efraim Flashner's message
	of "Thu, 31 Mar 2016 10:22:16 +0300")
List-Id: "Development of GNU Guix and the GNU System distribution."
	<guix-devel.gnu.org>
List-Unsubscribe: <https://lists.gnu.org/mailman/options/guix-devel>,
	<mailto:guix-devel-request@gnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/archive/html/guix-devel>
List-Post: <mailto:guix-devel@gnu.org>
List-Help: <mailto:guix-devel-request@gnu.org?subject=help>
List-Subscribe: <https://lists.gnu.org/mailman/listinfo/guix-devel>,
	<mailto:guix-devel-request@gnu.org?subject=subscribe>
Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org
Sender: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org
To: Efraim Flashner <efraim@flashner.co.il>
Cc: guix-devel@gnu.org

Efraim Flashner <efraim@flashner.co.il> writes:

> On Wed, Mar 30, 2016 at 05:57:24PM -0400, Mark H Weaver wrote:
>> Mark H Weaver <mhw@netris.org> writes:
>> 
>> > Here's one way to do it:
>> >
>> >     (inputs
>> >      `(("cdparanoia" ,cdparanoia)
>> >        ,@(if (any (cute string-prefix? <> (or (%current-target-system)
>> >                                               (%current-system)))
>> >                   '("arm" "mips"))
>
> 64-bit Arm is Aarch64 and not arm64, so this probably won't capture
> 64-bit Arm if/when we start to support it.

We shouldn't assume that the same problem will occur on Aarch64.
I'd prefer to wait and see.

> It turns out I didn't search gstreamer.scm well enough, so if this patch
> looks good then I'll copy the syntax over to the other packages in
> gstreamer and make patches for them too.
>
> efraim@debian-netbook:~/workspace/guix$ grep \(\"orc\"\ \,orc\)
> gnu/packages/*scm
> gnu/packages/gstreamer.scm:           `(("orc" ,orc)))
> gnu/packages/gstreamer.scm:       ("orc" ,orc)
> gnu/packages/gstreamer.scm:       ("orc" ,orc)
> gnu/packages/gstreamer.scm:       ("orc" ,orc)))
> gnu/packages/gstreamer.scm:       ("orc" ,orc)

To be honest, I'm having second thoughts about this entire approach.

My concern is that without orc, the codec implementations (especially
the video codecs) might be so slow as to be practically unusable.  I'm
reluctant to apply this workaround too broadly until we know whether it
results in anything good.  Really, we should be trying to investigate
and fix the underlying problems instead of sweeping them under the rug.

     Thoughts?
       Mark