From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mp0 ([2001:41d0:2:bcc0::]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) by ms0.migadu.com with LMTPS id +CZ3D/W5a2BwGgEAgWs5BA (envelope-from ) for ; Tue, 06 Apr 2021 03:31:33 +0200 Received: from aspmx1.migadu.com ([2001:41d0:2:bcc0::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by mp0 with LMTPS id 6FxHCvW5a2B+XgAA1q6Kng (envelope-from ) for ; Tue, 06 Apr 2021 01:31:33 +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 818551074E for ; Tue, 6 Apr 2021 03:31:32 +0200 (CEST) Received: from localhost ([::1]:50350 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lTaZ1-00085w-5W for larch@yhetil.org; Mon, 05 Apr 2021 21:31:31 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:46212) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lTaYY-00085b-4s for guix-patches@gnu.org; Mon, 05 Apr 2021 21:31:02 -0400 Received: from debbugs.gnu.org ([209.51.188.43]:55570) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1lTaYX-0007L5-SP for guix-patches@gnu.org; Mon, 05 Apr 2021 21:31:01 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1lTaYX-0005es-Oh for guix-patches@gnu.org; Mon, 05 Apr 2021 21:31:01 -0400 X-Loop: help-debbugs@gnu.org Subject: [bug#42885] [PATCH 23/27] gnu: calibre: Update to 5.13.0. Resent-From: Brendan Tildesley Original-Sender: "Debbugs-submit" Resent-CC: guix-patches@gnu.org Resent-Date: Tue, 06 Apr 2021 01:31:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 42885 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: patch To: Leo Famulari Cc: 42885@debbugs.gnu.org Received: via spool by 42885-submit@debbugs.gnu.org id=B42885.161767262221704 (code B ref 42885); Tue, 06 Apr 2021 01:31:01 +0000 Received: (at 42885) by debbugs.gnu.org; 6 Apr 2021 01:30:22 +0000 Received: from localhost ([127.0.0.1]:38883 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1lTaXu-0005e0-0R for submit@debbugs.gnu.org; Mon, 05 Apr 2021 21:30:22 -0400 Received: from mout-p-102.mailbox.org ([80.241.56.152]:27642) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1lTaXr-0005dk-F1 for 42885@debbugs.gnu.org; Mon, 05 Apr 2021 21:30:20 -0400 Received: from smtp1.mailbox.org (smtp1.mailbox.org [80.241.60.240]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-384) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by mout-p-102.mailbox.org (Postfix) with ESMTPS id 4FDqhj1F7pzQjn4; Tue, 6 Apr 2021 03:30:13 +0200 (CEST) X-Virus-Scanned: amavisd-new at heinlein-support.de DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=brendan.scot; s=MBO0001; t=1617672611; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=H6XYDE+7LFaOqOse7GH2dToj3yqwFXy2xUUIWv7DbWk=; b=p9BtSYkGnEhocwEvAcRt3iCb5ENqnp2hxbpeMyCxdunp4P/jHUVKXuCIbvfuFRb1R5K5s3 +DZnrKniNoKew/yOgjn5npFlfW8En8VE+WEIgglUvjbX1iHPugKN26HBSAy7dhOHIP0/cR 2HASAGm9w37PxRwXiklmOLkPwCejXvoax03KiT7NH/CrzElBXC3B+H4LEJ9pkw6S2e9aUE qhGLCvnR6bhwa1x6QTZW6Xc/1Wd/Qilr6qCliTNiodh1pwq/IYDyI5PhsmyEV6FTc1aADh 7+1VeFLSVG5FN1OUaLfMktN/UjFwxbvZSV4Z91Byu4KhqSS6qTMLvBJpdp2SaA== Received: from smtp1.mailbox.org ([80.241.60.240]) by spamfilter05.heinlein-hosting.de (spamfilter05.heinlein-hosting.de [80.241.56.123]) (amavisd-new, port 10030) with ESMTP id KqoxojzLpiRt; Tue, 6 Apr 2021 03:30:08 +0200 (CEST) Date: Tue, 6 Apr 2021 03:30:08 +0200 (CEST) From: Brendan Tildesley Message-ID: <1191049121.29289.1617672608111@office.mailbox.org> In-Reply-To: References: <20210310090053.21612-1-mail@brendan.scot> <20210310090053.21612-23-mail@brendan.scot> <1277129883.83284.1616552006895@office.mailbox.org> <327102377.27084.1617085423564@office.mailbox.org> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Priority: 3 Importance: Normal X-MBO-SPAM-Probability: X-Rspamd-Score: -5.86 / 15.00 / 15.00 X-Rspamd-Queue-Id: AE7E4387 X-Rspamd-UID: 71ba5e 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" X-Migadu-Flow: FLOW_IN ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=yhetil.org; s=key1; t=1617672692; 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: content-transfer-encoding:content-transfer-encoding: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:dkim-signature; bh=H6XYDE+7LFaOqOse7GH2dToj3yqwFXy2xUUIWv7DbWk=; b=Uc1giq/AvcKUtPOBlkWDDPWGCdIXMVHPYlRjxLNHwqzRA0IEjOQJgZTeFo5reV9yvFFvfF jHWEZi9uzPl+RCyMj+w/EyO/dyiTGcRRYwSg541VQphDO/HFJ+bj7yDMLHKl++fUGs42MV TyqprB2+I7D7MGOlF6RDFh/9avaXdzsAAhyvGHrIHWxv+KFrhPZljmcawV7R4+vZVWZMXt g9iFUrcQhysrNZXJxTphAK3kHVC3M0JdTtN3a8bTyz1SPn68PM/HznnXuyZZ8BpOlXP1LP nCzLawXX3KYZle2XIq0B6ik4r1g9+IcctNvkMBX5PB4yooNzSJEEUZw+WFQ2Kw== ARC-Seal: i=1; s=key1; d=yhetil.org; t=1617672692; a=rsa-sha256; cv=none; b=biN+1DZNerhR5ayP5IJVDG0zHwXUojZPVoCt8IwKxS3pSnT+gmZ0UmoGMxPURdU2/n/UL8 HuQCMapSyGZ/8pjMMsJq3rJMVuf++nRevTrMkZR8ktrKKxyjK9K9sV7y4rYqakEHEuF0IT FNaUMD+sgFNnQGw8z5vmlkQVzxMySIbfesOAqUFNT/3NSuTyeSAAusXbMfLsVunPjq1pCe agBPZUZ7kb4gUdRWAd/T06Muopepicb/VeYmlhl/BNiKGG5a6Iu3aPgxnt1NcqSrzbSe47 XwoAy51kALkhDFETwFbu2tqm/3uL/TmCNB6LO9rI+MIoHwbPfFUU08g9FBNpbg== ARC-Authentication-Results: i=1; aspmx1.migadu.com; dkim=fail ("headers rsa verify failed") header.d=brendan.scot header.s=MBO0001 header.b=p9BtSYkG; dmarc=none; spf=pass (aspmx1.migadu.com: domain of guix-patches-bounces@gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=guix-patches-bounces@gnu.org X-Migadu-Spam-Score: 0.06 Authentication-Results: aspmx1.migadu.com; dkim=fail ("headers rsa verify failed") header.d=brendan.scot header.s=MBO0001 header.b=p9BtSYkG; dmarc=none; spf=pass (aspmx1.migadu.com: domain of guix-patches-bounces@gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=guix-patches-bounces@gnu.org X-Migadu-Queue-Id: 818551074E X-Spam-Score: 0.06 X-Migadu-Scanner: scn0.migadu.com X-TUID: 4BbIMNyqmr1+ > On 04/05/2021 9:58 PM Leo Famulari wrote: [...] > If I understand correctly, the issue that any package that uses > python-pyqt5 also needs to be able to find python-pyqt5-sip. Is that > right? > > If so, it sounds like a case for propagated-inputs [0]. Concretely, I made > python-pyqt5-sip a propagated-input of python-pyqt and removed the > 'pyqt5-sip' phase, and Calibre built successfully. > > Does that seem like the right approach? > Sounds good. I didn't realise propagated-inputs did that, I thought they were just normal inputs that were installed along side the package in a profile, but wouldn't make a difference during build time. Does this mean all packages that depend on python-pyqt5 will have python-pyqt5-sip added to its own list of inputs in their own package definition? If so the manual doesn't mention that. > > The reason I added qtsvg was to try fix the Qt test. If you remove the > > line (setenv "SKIP_QT_BUILD_TEST" "true"), this test fails for > > multiple reasons. One of them was qtsvg missing. Another was the > > get_exe_path bit. But a third reason I that its call to printtopdf in > > pyqtwebegine returns an empty string instaed of b'Skia/PDF'. I had no > > idea how to proceed with fixing that so I left it for now. But at > > least fixed the other errors. I assume some SVG related functionality > > will fail without it... > > That's a good point. However, I checked if the built Calibre refers to > qtsvg, and it doesn't [1]. So, it's unlikely that Calibre will be able to > find and use qtsvg, regardless of whether or not it's an input. So, I'd > prefer to leave it out until we understand what it's for and how to make > sure that Calibre can use it. I see, that's a good trick for checking references.. Leave svg out for now then. > > > All good I think. My descriptions were much worse than I realised. > > No worries. Writing the synopses and descriptions is a completely > different type of work from packaging or programming. I often "finish" > some packages, but need to go back later to write the descriptions. I'm > happy to finish these tasks as part of the code review process. > I find writing them the most stressful part because I sit there not knowing what to write. > > python-cchardet differs from in python-chardet in that its not written > > /in/ python, but links to a fast C library to do it, but your > > description/synopsis changes make it look like its all in > > Python. Maybe make the description: > > > > "cChardet is a character encoding detector, binding to the C > > library uchardet for speed." ? > > Thanks, that helps. I amended the synopses and description based on > this. > > I pushed my revisions of your updated branch, rebased on the current > master branch, to Savannah: > > https://git.savannah.gnu.org/cgit/guix.git/log/?h=wip-update-calibre > > [0] > https://guix.gnu.org/manual/en/html_node/package-Reference.html > > [1] This command be used: > $ guix gc --references $(./pre-inst-env guix build --no-grafts calibre) | grep qtsvg