From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mp1 ([2001:41d0:8:6d80::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by ms0.migadu.com with LMTPS id iIx7OBOyTWHjeQEAgWs5BA (envelope-from ) for ; Fri, 24 Sep 2021 13:10:11 +0200 Received: from aspmx1.migadu.com ([2001:41d0:8:6d80::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by mp1 with LMTPS id 4HceNBOyTWFNJwAAbx9fmQ (envelope-from ) for ; Fri, 24 Sep 2021 11:10:11 +0000 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 BD8E1276DD for ; Fri, 24 Sep 2021 13:10:11 +0200 (CEST) Received: from localhost ([::1]:51542 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1mTj5m-0003Ka-4U for larch@yhetil.org; Fri, 24 Sep 2021 07:10:10 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:58840) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1mTj1m-0006yN-J1 for guix-patches@gnu.org; Fri, 24 Sep 2021 07:06:06 -0400 Received: from debbugs.gnu.org ([209.51.188.43]:45228) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1mTj1m-00055J-9Z for guix-patches@gnu.org; Fri, 24 Sep 2021 07:06:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1mTj1m-0002PC-4P for guix-patches@gnu.org; Fri, 24 Sep 2021 07:06:02 -0400 X-Loop: help-debbugs@gnu.org Subject: [bug#50751] [PATCH] gnu: update Trezor support Resent-From: Attila Lendvai Original-Sender: "Debbugs-submit" Resent-CC: guix-patches@gnu.org Resent-Date: Fri, 24 Sep 2021 11:06:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 50751 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: patch To: Sarah Morgensen Cc: 50751@debbugs.gnu.org Received: via spool by 50751-submit@debbugs.gnu.org id=B50751.16324815139149 (code B ref 50751); Fri, 24 Sep 2021 11:06:02 +0000 Received: (at 50751) by debbugs.gnu.org; 24 Sep 2021 11:05:13 +0000 Received: from localhost ([127.0.0.1]:56768 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1mTj0y-0002NU-OT for submit@debbugs.gnu.org; Fri, 24 Sep 2021 07:05:12 -0400 Received: from mail-4323.protonmail.ch ([185.70.43.23]:37582) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1mTj0u-0002N6-BC for 50751@debbugs.gnu.org; Fri, 24 Sep 2021 07:05:10 -0400 Date: Fri, 24 Sep 2021 11:05:00 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=lendvai.name; s=protonmail2; t=1632481501; bh=WHpz5T91QzlmhNfU4iH/jEKEsCZEYOBwp7ci+9MDyj4=; h=Date:To:From:Cc:Reply-To:Subject:In-Reply-To:References:From; b=KQ9QCRsiozKJtMLL+zKxNPni13zxXadSD+w3KiYY2LRj/pF3fp6TUYgtp+DDrtoVs C5vcj2zaLruD3aHEzcCzur5AhjazTH9JEj4l8dsMo6SgbAZhLZuebjlLYB3sQJb9J+ rRRTqgoPp2SJPRAYZTgUvmqs1e89uXbpcmHsLXDEzvSTdZBLMizJnSmdStdWWIIXpU p8cAPaBnHUZprtpbVErnVhFdirVsNuZylsZ03vSUsA+FBSHGX54LLJ4i/jPmKclpyG Y/Wwum+/1dXBGZvXOEYCs7TsqE3FhFV8hAxh0TBd/9D0iXTYzxFqNB8QEOwnDPXF7u YdvsdlztucdiQ== From: Attila Lendvai Message-ID: In-Reply-To: <86mto3kumt.fsf@mgsn.dev> References: <20210923083045.10372-1-attila@lendvai.name> <86mto3kumt.fsf@mgsn.dev> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable 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: , Reply-To: Attila Lendvai Errors-To: guix-patches-bounces+larch=yhetil.org@gnu.org Sender: "Guix-patches" X-Migadu-Flow: FLOW_IN X-Migadu-Spam-Score: -4.00 Authentication-Results: aspmx1.migadu.com; none X-Migadu-Queue-Id: BD8E1276DD X-Spam-Score: -4.00 X-Migadu-Scanner: scn0.migadu.com X-TUID: RF25SmHkeBZK > > Fetch everything from their git repos, instead of pypi. > > Given the resulting version mismatch, and having to pick apart the > sources, is there a compelling reason for this? It looks like the same > files are included. an objective reason: one of these versions were not in pipy at the time i was working on this. a subjective reason: one less organization to rely on. if anything would happen to pipy, e.g. some random API change, then that wouldn't affect us anymore. if github went down or went hostile, then we could just change the git URL and continue with a minor glitch. > > - gnu/packages/finance.scm (python-trezor-agent): Update to 0.14.2. > > > > (python-trezor): Update to 0.12.3. > > (trezor-agent): Update to 0.11.0. > > (trezord): Update to 2.0.31. > > - gnu/packages/patches/trezor-agent-fix-argv0.patch: New file. > > When at all possible, version updates should be separate commits from cha= nges. ok, makes sense, because now that i think about it, it was broken with the prior version, too. i'll resend it in two patches once i have tested th= em. > > ;; overwrite the wrapper with the real thing. > > (install-file "./trezor_agent.py" > > - (string-append out "/bin")) > > - #t)))))) > > - (string-append out "/bin"))) > > - #t))))) > > This wasn't added by your patch, but it's odd that the reason for > overriding the wrapper wasn't documented.... i have added a comment explaining the situation. thank you for the feedback! the updated patches will be coming a bit later. - attila PGP:=C2=A05D5F 45C7 DFCD 0A39