From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mp10.migadu.com ([2001:41d0:2:bcc0::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by ms9.migadu.com with LMTPS id 0K6NEfqeTGQnRgAASxT56A (envelope-from ) for ; Sat, 29 Apr 2023 06:37:14 +0200 Received: from aspmx1.migadu.com ([2001:41d0:2:bcc0::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by mp10.migadu.com with LMTPS id YIPAEPqeTGRKZwAAG6o9tA (envelope-from ) for ; Sat, 29 Apr 2023 06:37:14 +0200 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by aspmx1.migadu.com (Postfix) with ESMTPS id E4CFEEEAA for ; Sat, 29 Apr 2023 06:37:13 +0200 (CEST) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1pscKW-0008QW-4P; Sat, 29 Apr 2023 00:37:04 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1pscKU-0008Pu-Bk for guix-patches@gnu.org; Sat, 29 Apr 2023 00:37:02 -0400 Received: from debbugs.gnu.org ([209.51.188.43]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1pscKU-0005gD-1f for guix-patches@gnu.org; Sat, 29 Apr 2023 00:37:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1pscKT-0005nR-NR for guix-patches@gnu.org; Sat, 29 Apr 2023 00:37:01 -0400 X-Loop: help-debbugs@gnu.org Subject: [bug#63117] [PATCH] gnu: yt-dlp: Change input. Resent-From: Jack Hill Original-Sender: "Debbugs-submit" Resent-CC: guix-patches@gnu.org Resent-Date: Sat, 29 Apr 2023 04:37:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 63117 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: patch To: Dominik Delgado Steuter Cc: 63117@debbugs.gnu.org Received: via spool by 63117-submit@debbugs.gnu.org id=B63117.168274297022212 (code B ref 63117); Sat, 29 Apr 2023 04:37:01 +0000 Received: (at 63117) by debbugs.gnu.org; 29 Apr 2023 04:36:10 +0000 Received: from localhost ([127.0.0.1]:34856 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1pscJe-0005mB-97 for submit@debbugs.gnu.org; Sat, 29 Apr 2023 00:36:10 -0400 Received: from minsky.hcoop.net ([104.248.1.95]:49518) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1pscJb-0005ly-Fq for 63117@debbugs.gnu.org; Sat, 29 Apr 2023 00:36:09 -0400 Received: from marsh.hcoop.net ([45.55.52.66]) by minsky.hcoop.net with esmtpsa (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1pscJV-0005g0-B9; Sat, 29 Apr 2023 00:36:01 -0400 Date: Sat, 29 Apr 2023 00:36:01 -0400 (EDT) From: Jack Hill X-X-Sender: jackhill@marsh.hcoop.net In-Reply-To: <20230427143200.13000-1-d@delgado.nrw> Message-ID: References: <20230427143200.13000-1-d@delgado.nrw> User-Agent: Alpine 2.21 (DEB 202 2017-01-01) MIME-Version: 1.0 Content-Type: text/plain; format=flowed; charset=US-ASCII X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: guix-patches@gnu.org List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+larch=yhetil.org@gnu.org Sender: guix-patches-bounces+larch=yhetil.org@gnu.org X-Migadu-Flow: FLOW_IN X-Migadu-Country: US ARC-Seal: i=1; s=key1; d=yhetil.org; t=1682743034; a=rsa-sha256; cv=none; b=aRZY8p9JaEb6vZVaWm5jDR2wj4xYfBUNb0wIR/N8oFMvUXvrdKBjrnd+VM/WNVpU4cGVm8 ur0hllV7mRnHjUMs7TuKy9FLMja1oR+rvp3z5hCrhTDcBPyswPk0ebNNGE9ouyn/Q8jnkB wd8GefyFLf2w5ULu2Wl0Mmh5ZoaVAS53kG3vak/PmetkC+3QGAFFj5ohSykaRyIL81PY8o 9Do/qNrW939SolQjfIblT4/VBUmwjmVfxoVnWCSbKujXkngWtiRaSEh1rzQi1n6Sndj47M uLDjER+a7naTL1hcuvENDcuCmEsw3tV/E79dlR6Vw7HUauOxYlswi7M8H8A3WA== ARC-Authentication-Results: i=1; aspmx1.migadu.com; dkim=none; dmarc=none; spf=pass (aspmx1.migadu.com: domain of "guix-patches-bounces+larch=yhetil.org@gnu.org" designates 209.51.188.17 as permitted sender) smtp.mailfrom="guix-patches-bounces+larch=yhetil.org@gnu.org" ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=yhetil.org; s=key1; t=1682743034; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:resent-cc:resent-from:resent-sender: resent-message-id:in-reply-to:in-reply-to:references:references: list-id:list-help:list-unsubscribe:list-subscribe:list-post; bh=3kiEief3MLvhk5CuJdY0u7jviFQtU5SiaI+UlQNNsx0=; b=ocrmQK+dQvwzhqqzGIfZQEvVsxKmw7FepdCgMgazOMB4ybXs8ViwbR0V5WD0mUd07bHWs2 SNqpRPfTZ0G79gZYwaQfaC00lX1F/6ZJFEd7Gc+uT0SnRkKlujCJT45IkS7qng4VJ4vC9l tVONL8XsBGyE3DdHEqSjlJQl2G/Y/K66q3HbQ0pjCChKWG5oztoMIYFTAa7fvph1rtinE4 jBjijE6HgSi/ZFYs+iuQZEYqvxiAtqoKIZGIHC0R5uP9wptXsBVKZXo5966sQRX6vOi0E9 k3jFA1Sr6S81gIsRM2k69UxFV0v7OdiKLVr12d4/u19hKAMewkYJ+ZLUVCz/Ag== X-Migadu-Scanner: scn1.migadu.com Authentication-Results: aspmx1.migadu.com; dkim=none; dmarc=none; spf=pass (aspmx1.migadu.com: domain of "guix-patches-bounces+larch=yhetil.org@gnu.org" designates 209.51.188.17 as permitted sender) smtp.mailfrom="guix-patches-bounces+larch=yhetil.org@gnu.org" X-Migadu-Spam-Score: -3.20 X-Spam-Score: -3.20 X-Migadu-Queue-Id: E4CFEEEAA X-TUID: kSmGyg+gVT9G On Thu, 27 Apr 2023, Dominik Delgado Steuter via Guix-patches via wrote: > yt-dlp complained when the "--add-metadata" flag was used. > youtube-dl does not need ffmpeg as a propagated-input, though. Dominik, Thanks for working on improving our yt-dlp package. Can you explain a little more about why yt-dlp needs ffmpeg to be propagated? I haven't looked too closely, but I suspect yt-dlp just want to be able to call ffmpeg at runtime. If that's true, then I think it would be better to use substitute* in a phase so that yt-dlp can use the full path to ffmpeg, and not clutter folks' profiles. If I'm correct, can you provide an updated patch that does that? I'm happy to provide more guidance if you need it. If I'm wrong and ffmpeg needs to be propagated for some other reason, can you add a comment in the code explaining the need? Best, Jack