From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mp12.migadu.com ([2001:41d0:8:6d80::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by ms5.migadu.com with LMTPS id MAJpCuftv2PLeAEAbAwnHQ (envelope-from ) for ; Thu, 12 Jan 2023 12:24:23 +0100 Received: from aspmx1.migadu.com ([2001:41d0:8:6d80::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by mp12.migadu.com with LMTPS id 8HM8Cuftv2NeCgAAauVa8A (envelope-from ) for ; Thu, 12 Jan 2023 12:24:23 +0100 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 94065BA39 for ; Thu, 12 Jan 2023 12:24:22 +0100 (CET) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1pFvfo-0003PD-0f; Thu, 12 Jan 2023 06:23:08 -0500 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 1pFvfj-0003OO-2n for guix-patches@gnu.org; Thu, 12 Jan 2023 06:23:03 -0500 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 1pFvfh-0003qU-QF for guix-patches@gnu.org; Thu, 12 Jan 2023 06:23:02 -0500 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1pFvfh-0003m0-Lj for guix-patches@gnu.org; Thu, 12 Jan 2023 06:23:01 -0500 X-Loop: help-debbugs@gnu.org Subject: [bug#60640] Gnu: Add gdcm Resent-From: =?UTF-8?Q?Tor-bj=C3=B6rn?= Claesson Original-Sender: "Debbugs-submit" Resent-CC: guix-patches@gnu.org Resent-Date: Thu, 12 Jan 2023 11:23:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 60640 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: patch To: Tobias Geerinckx-Rice Cc: 60640@debbugs.gnu.org X-Debbugs-Original-Cc: 60640@debbugs.gnu.org, guix-patches@gnu.org Received: via spool by submit@debbugs.gnu.org id=B.167352252814435 (code B ref -1); Thu, 12 Jan 2023 11:23:01 +0000 Received: (at submit) by debbugs.gnu.org; 12 Jan 2023 11:22:08 +0000 Received: from localhost ([127.0.0.1]:44859 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1pFvep-0003kl-Dj for submit@debbugs.gnu.org; Thu, 12 Jan 2023 06:22:08 -0500 Received: from lists.gnu.org ([209.51.188.17]:58082) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1pFven-0003kc-Fg for submit@debbugs.gnu.org; Thu, 12 Jan 2023 06:22:06 -0500 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 1pFvea-0002fU-Sc for guix-patches@gnu.org; Thu, 12 Jan 2023 06:21:53 -0500 Received: from mail-ej1-x631.google.com ([2a00:1450:4864:20::631]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1pFveW-0003aj-Tr for guix-patches@gnu.org; Thu, 12 Jan 2023 06:21:51 -0500 Received: by mail-ej1-x631.google.com with SMTP id cf18so37589381ejb.5 for ; Thu, 12 Jan 2023 03:21:47 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=DY7lg7sFHgdNRBIZpobFwvRYagZTwu7DDSo2lPBVmJE=; b=ZigLOcYqbk6uCYG6knvT0po+m/G7fACthORM4ALEiub4H3An35nDUiNWxPArkiKIej UN2p3xIn1U6RUd+viYbV9DS6FhujGgESWE+/2sbIKizhi87Po7LE1VtUBvhijkbPCfev Zaa9+yaINIHZ8HidErSZtxOGBzjACeIMNw4myYyu3iJhhqiwJd/jmrr85H2Zto9U1F02 /vf2ZhaO25azz3lVlkTqHXaKzk3PP0ASDL7SEFAgyQWraeOUFWE4Qlr5RuDFJOceQ8Th DkzF6Yaqu+dMm1AuiQj9dLXpS2BIN+i2Tj1FdX1PAeZYrclHHoM4nvEURM9QAj3YQfsP vp/w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=DY7lg7sFHgdNRBIZpobFwvRYagZTwu7DDSo2lPBVmJE=; b=524lJDMZhfl8+pYbCK2Y3NF7tp7tkHLTptFXW993oyPYsMMGmH9kHdzVrPQWCUMKKS kfFNohgTjIPF/jPfD/9+OULfPYd8uubpiJXVyDpLbSauiPAAHyJBta1Du7YTCiVpaib8 Oe69j7VLE9dRCalqGrG119P6ridU5fVhfYu5RM8j9Op/cxr8hXZH5NdHNUt8ytfMgVZa gaPbgoxrK2wgacBnklFx+9l3T6bbo4OL9GEN4cHMhqrW8f8ycXno5xme/o6b4tfdz1Jo TJTjcaJ9TZUYYNxNQV0WWo8CEGb5ycJx9OYgk0qfADbJHzNCpGqICK2Sl96DWrPurufx 6Ktg== X-Gm-Message-State: AFqh2kqSHyL1IxwyUk+HZjRT4v1e3BsZA7+4RP1mr+6CA5k5lmYkFUqS wrYNr0FuMlHgJl+VVou8kAOfswk76ITxbbWTtBLjCYfD X-Google-Smtp-Source: AMrXdXucpgDogkG1Lanr7LB8v0TQXTTBQOzQJJfSMkqAoKMNyXH+Lrc62KYV0xEeMghpowJLHuP5iHoVxua3oeAzet4= X-Received: by 2002:a17:906:d051:b0:7c0:fd1a:6792 with SMTP id bo17-20020a170906d05100b007c0fd1a6792mr4732971ejb.431.1673522505870; Thu, 12 Jan 2023 03:21:45 -0800 (PST) MIME-Version: 1.0 References: <877cxyp1wf.fsf@gmail.com> <87y1qaxhh7.fsf@nckx> <87o7r5li2j.fsf@gmail.com> In-Reply-To: <87o7r5li2j.fsf@gmail.com> From: =?UTF-8?Q?Tor-bj=C3=B6rn?= Claesson Date: Thu, 12 Jan 2023 13:21:34 +0200 Message-ID: Content-Type: multipart/alternative; boundary="0000000000007c70fa05f20f52fe" Received-SPF: pass client-ip=2a00:1450:4864:20::631; envelope-from=tclaesson@gmail.com; helo=mail-ej1-x631.google.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action 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-Country: US X-Migadu-Flow: FLOW_IN ARC-Authentication-Results: i=1; aspmx1.migadu.com; dkim=fail ("headers rsa verify failed") header.d=gmail.com header.s=20210112 header.b=ZigLOcYq; 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"; dmarc=fail reason="SPF not aligned (relaxed)" header.from=gmail.com (policy=none) ARC-Seal: i=1; s=key1; d=yhetil.org; t=1673522662; a=rsa-sha256; cv=none; b=GaoshEOjVOqXhEyGNe69ksFfGtJsK0lLWBGe6LK+nh91lGGqjEkB/GzR6f0mHDHPCsMBff BkCcTJR8ikQtyXqqrp7VBPdUQ41Eq2vF63zUMNj1mB9BFceeSI5WgcA6jrjW8Vf6kfa/7R wPYPVXnvWjFqA2lbduYqRO4eOWatwnN+XwuZOn25//69q78jeklJ+perlrrXFd9SnIwHOn iCwDMYZ3IYWcPCR1Aqq5uKqcnQSjsidWYjL9ufFgnq00dyxHEZIpJOYVE1S1Z48IA7r5MS 48I1Ng+0kmuE/wtd9oSMJJQcVO3YKwyfzCRJv+4RvFEUVcHED/5dGtrqBi/CJg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=yhetil.org; s=key1; t=1673522662; 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: dkim-signature; bh=DY7lg7sFHgdNRBIZpobFwvRYagZTwu7DDSo2lPBVmJE=; b=B1Acwmuchtk9h+zkR63OaK1OnFSD9rCYn3O5F7StCHHjYEk/nLaskt6KGpoAImX1/NAMwq aJqLrpe65HVbtXQGfj0RRg3eYEwE2+XBT7MjMCH6nxoM6+fFwmSHddXnDapkv38LjsraBJ 5HM4jR5qBt5gESPIKeJm1Xb5IJvFlmbpOMR6CRrwPeQMP7fDvv8QJUpjAdrOS4kBuTTLs2 nPDXPz5HmE8tZTQpDyFauZlPmxxUDayP1dxFbp4rP8t2RR7qoW4n2D/tzLLv6Tl56trzji SpaTrqRAhALuO6USF7SmoZBsWMol9xARKEk715YezEprdt5YOSAcJyw7EJayKg== X-Migadu-Queue-Id: 94065BA39 X-Migadu-Scanner: scn0.migadu.com Authentication-Results: aspmx1.migadu.com; dkim=fail ("headers rsa verify failed") header.d=gmail.com header.s=20210112 header.b=ZigLOcYq; 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"; dmarc=fail reason="SPF not aligned (relaxed)" header.from=gmail.com (policy=none) X-Migadu-Spam-Score: -2.80 X-Spam-Score: -2.80 X-TUID: Z/GYJsx09paU --0000000000007c70fa05f20f52fe Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Hi again! Your patch applies perfectly, the error was on my side. Also, a gdcm package was added to the bioinformatics module 4 days ago=3D) = It packages an older version (2.8.9) but is much more concise than my attempt. Would it be worthwhile to continue work on this package as perhaps gdcm-3.0 and move it to bioinformatics? I put it in image-processing since dcmtk was already there. Best regards Den ons 11 jan. 2023 kl 08:08 skrev Tor-bj=C3=B6rn Claesson : > > Hi! > > Tobias Geerinckx-Rice writes: > > > >> 3. It does not perform tests. > > > > OK, I'll take a look. > > > > If tests are disabled, the reason should always be noted in a comment. > > Even if it's just =E2=80=98; no test suite=E2=80=99. > > > > #:tests? #t makes the build fail with "make: *** No rule to make target > 'test'. Stop." > > GDCM has nightly regression tests > (https://open.cdash.org/index.php?project=3DGDCM), should we try to run > those when building? I have tried to find out how to do this but for now > with no success. Maybe it is obvious to more experienced people? > > >> +(define-public gdcm > > > > It used to be common to unconditionally add packages to the end of > > files, but this needlessly increased the risk of merge conflicts. > > > > Instead, just add them wherever they first fit alphabetically; here, I > > put it above =E2=80=98mia=E2=80=99. > > > > Ok, will do from now on! > > >> + (version "3.0.20") > > > > =E2=80=98guix lint=E2=80=99 says this can be updated to 3.1.0 but I did= n't try, as I'd > > rather it be tested by an actual user =E2=80=94 i.e., you. > > > > I got that too, but the latest release in git is 3.0.20 > > >> + "1w78cmm9q7aavs7svdkl4dgilcqk4yazci9m6x7icrssb7cj991i")))) > >> + (build-system cmake-build-system) > >> + (outputs '("out" "doc")) > > > > /share/doc wasn't actually installed into "doc", but to "out", so I > > set the GDCM_INSTALL_DOC_DIR configure flag. > > > >> + (arguments > >> + (list #:tests? #f > >> + #:phases #~(modify-phases %standard-phases > >> + (add-before 'configure 'set-LDFLAGS > >> + (lambda* (#:key inputs outputs > >> #:allow-other-keys) > >> + (setenv "LDFLAGS" > >> + (string-append "-Wl,-rpath=3D" > >> + #$output > >> "/lib")))) > >> + (add-before 'build 'patch-gdcm-charls.h > >> + (lambda _ > >> + (substitute* > >> "../source/Utilities/gdcm_charls.h" > >> + (("# include ") > > Ah, good catch! > > > > > Purely as a matter of taste I dropped the =E2=80=98# include =E2=80=99 = from both > > strings and escaped the =E2=80=98.=E2=80=99 in the regexp. > > > >> + "# include ")) > >> #t))) > > > > =E2=80=98#t=E2=80=99 endings are also obsolete. Just drop them entirel= y. Phases can > > now safely return anything, including nothing or undefined. > > > > I added the following phase to work around log spam, since I didn't > > find its source (nor did I look very hard) [edit: it was indeed > > graphviz, thanks]. By default, $HOME is not writable in the build > > environment. > > > > (add-before 'build 'set-HOME > > ;; The build spams =E2=80=98Fontconfig error: No writable cache > > ;; directories=E2=80=99 in a seemingly endless loop otherwise. > > (lambda _ > > (setenv "HOME" "/tmp"))) > > > >> + #:configure-flags #~(list "-DCMAKE_SKIP_RPATH:BOOL=3DYES" > > Is this needed, btw? It came from gdcm:s packaging > instructions. Removing it causes no verify-runpath issues. > > > > > I, opinionated, added newlines after #:phases and #:configure-flags. > > > > Some people like the =E2=80=98extreme indentation=E2=80=99 you get by t= hrowing away > > half of your screen width. I find it leads to cramped code and noisy > > patches once the phases need to get actual work done or an even longer > > CMAKE_ flag comes along. > > > > I also added some newlines and tried to group related flags. > > > > Thanks, I didn't know that would make the line fit better on > screen. Much neater=3D) > > >> + "-DCMAKE_C_FLAGS=3D-fvisibility=3Dhidden" > >> + "-DCMAKE_CXX_FLAGS=3D-fvisibility=3Dhidden" > > > > Should these be explained in a very brief comment? > > > > They are from https://github.com/malaterre/GDCM/blob/master/PACKAGER, > the explanation is: > "This make sure that on UNIX, the API is actually identical at what is > found on Windows." > > > > > Thank you for building with system libraries! Also remove the bundled > > copies when possible. I did so in a (rather strict) source snippet. > > > > Ok, neat=3D) > > >> + "-DGDCM_PDF_DOCUMENTATION:BOOL=3DOFF" > > > > I cannot get the man pages to build, either. They need something > > called =E2=80=98xsl-ns=E2=80=99. I've disabled GDCM_BUILD_DOCBOOK_MANP= AGES for now. > > > >> + "-DGCM_BUILD_TESTING:BOOL=3DOFF" > > > > Why is this set? It's reported by CMake as having no effect, and a > > diff of the output confirms that. > > > > From the old wiki: > > "This boolean is responsible for deciding whether or not to build/run the > nightly regression test of gdcm. Warning when turning this option on, > the size of the gdcm libraries will be bigger since some extra code are > compiled in for the testing framework (see gdcm::Testing, and the md5 > lib)." > > This seems to be incorrect then, maybe we can skip it. > > >> + (license license:bsd-3))) > > > > I still need to check this. > > > > https://github.com/malaterre/GDCM/blob/master/Copyright.txt > > I'm not able to apply your new patch, but that is probably a fault on my > part. > > Thanks a lot for sharing your time and knowledge, and for making this > patch neater! I find this a lot of fun, but have no experience > with scheme or packaging, so your explanations are very valuable to me. > > Cheers > Tor-bj=C3=B6rn > --0000000000007c70fa05f20f52fe Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Hi again!

Your patch applies= perfectly, the error was on my side.

Also, a gdcm= package was added to the bioinformatics module 4 days ago=3D) It packages = an older version (2.8.9) but is much more concise than my attempt. Would it= be worthwhile to continue work on this package as perhaps gdcm-3.0 and mov= e it to bioinformatics? I put it in image-processing since dcmtk was alread= y there.

Best regards

Den ons 11 jan. 202= 3 kl 08:08 skrev Tor-bj=C3=B6rn Claesson <tclaesson@gmail.com>:

Hi!

Tobias Geerinckx-Rice <me@tobias.gr> writes:
>
>> 3. It does not perform tests.
>
> OK, I'll take a look.
>
> If tests are disabled, the reason should always be noted in a comment.=
> Even if it's just =E2=80=98; no test suite=E2=80=99.
>

#:tests? #t makes the build fail with "make: *** No rule to make targe= t
=C2=A0'test'.=C2=A0 Stop."

GDCM has nightly regression tests
(https://open.cdash.org/index.php?project=3DGDCM)= , should we try to run
those when building? I have tried to find out how to do this but for now with no success. Maybe it is obvious to more experienced people?

>> +(define-public gdcm
>
> It used to be common to unconditionally add packages to the end of
> files, but this needlessly increased the risk of merge conflicts.
>
> Instead, just add them wherever they first fit alphabetically; here, I=
> put it above =E2=80=98mia=E2=80=99.
>

Ok, will do from now on!

>> +=C2=A0 =C2=A0 (version "3.0.20")
>
> =E2=80=98guix lint=E2=80=99 says this can be updated to 3.1.0 but I di= dn't try, as I'd
> rather it be tested by an actual user =E2=80=94 i.e., you.
>

I got that too, but the latest release in git is 3.0.20

>> + "1w78cmm9q7aavs7svdkl4dgilcqk4yazci9m6x7icrssb7cj991i"= ))))
>> +=C2=A0 =C2=A0 (build-system cmake-build-system)
>> +=C2=A0 =C2=A0 (outputs '("out" "doc")) >
> /share/doc wasn't actually installed into "doc", but to = "out", so I
> set the GDCM_INSTALL_DOC_DIR configure flag.
>
>> +=C2=A0 =C2=A0 (arguments
>> +=C2=A0 =C2=A0 =C2=A0(list #:tests? #f
>> +=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0#:phases #~(modify-phase= s %standard-phases
>> +=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 (add-before 'configure 'set-LDFLAGS
>> +=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0 (lambda* (#:key inputs outputs
>> #:allow-other-keys)
>> +=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 (setenv "LDFLAGS"
>> +=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 (string-appe= nd "-Wl,-rpath=3D"
>> +=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0#$output
>> "/lib"))))
>> +=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 (add-before 'build 'patch-gdcm-charls.h
>> +=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0 (lambda _
>> +=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 (substitute*
>> "../source/Utilities/gdcm_charls.h"
>> +=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 (("# include <CharLS/char= ls.h>")

Ah, good catch!

>
> Purely as a matter of taste I dropped the =E2=80=98# include =E2=80=99= from both
> strings and escaped the =E2=80=98.=E2=80=99 in the regexp.
>
>> +=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0"# include <charls/= charls.h>"))
>> #t)))
>
> =E2=80=98#t=E2=80=99 endings are also obsolete.=C2=A0 Just drop them e= ntirely.=C2=A0 Phases can
> now safely return anything, including nothing or undefined.
>
> I added the following phase to work around log spam, since I didn'= t
> find its source (nor did I look very hard) [edit: it was indeed
> graphviz, thanks].=C2=A0 By default, $HOME is not writable in the buil= d
> environment.
>
>=C2=A0 (add-before 'build 'set-HOME
>=C2=A0 =C2=A0 ;; The build spams =E2=80=98Fontconfig error: No writable= cache
>=C2=A0 =C2=A0 ;; directories=E2=80=99 in a seemingly endless loop other= wise.
>=C2=A0 =C2=A0 (lambda _
>=C2=A0 =C2=A0 =C2=A0 (setenv "HOME" "/tmp")))
>
>> +=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0#:configure-flags #~(lis= t "-DCMAKE_SKIP_RPATH:BOOL=3DYES"

Is this needed, btw? It came from gdcm:s packaging
instructions. Removing it causes no verify-runpath issues.

>
> I, opinionated, added newlines after #:phases and #:configure-flags. >
> Some people like the =E2=80=98extreme indentation=E2=80=99 you get by = throwing away
> half of your screen width.=C2=A0 I find it leads to cramped code and n= oisy
> patches once the phases need to get actual work done or an even longer=
> CMAKE_ flag comes along.
>
> I also added some newlines and tried to group related flags.
>

Thanks, I didn't know that would make the line fit better on
screen. Much neater=3D)

>> + "-DCMAKE_C_FLAGS=3D-fvisibility=3Dhidden"
>> + "-DCMAKE_CXX_FLAGS=3D-fvisibility=3Dhidden"
>
> Should these be explained in a very brief comment?
>

They are from https://github.com/malaterre/GDC= M/blob/master/PACKAGER,
the explanation is:
"This make sure that on UNIX, the API is actually identical at what is= found on Windows."

>
> Thank you for building with system libraries!=C2=A0 Also remove the bu= ndled
> copies when possible.=C2=A0 I did so in a (rather strict) source snipp= et.
>

Ok, neat=3D)

>> + "-DGDCM_PDF_DOCUMENTATION:BOOL=3DOFF"
>
> I cannot get the man pages to build, either.=C2=A0 They need something=
> called =E2=80=98xsl-ns=E2=80=99.=C2=A0 I've disabled GDCM_BUILD_DO= CBOOK_MANPAGES for now.
>
>> + "-DGCM_BUILD_TESTING:BOOL=3DOFF"
>
> Why is this set?=C2=A0 It's reported by CMake as having no effect,= and a
> diff of the output confirms that.
>

>From the old wiki:

"This boolean is responsible for deciding whether or not to build/run = the
nightly regression test of gdcm. Warning when turning this option on,
the size of the gdcm libraries will be bigger since some extra code are
compiled in for the testing framework (see gdcm::Testing, and the md5
lib)."

This seems to be incorrect then, maybe we can skip it.

>> +=C2=A0 =C2=A0 (license license:bsd-3)))
>
> I still need to check this.
>

https://github.com/malaterre/GDCM/blob/ma= ster/Copyright.txt

I'm not able to apply your new patch, but that is probably a fault on m= y part.

Thanks a lot for sharing your time and knowledge, and for making this
patch neater! I find this a lot of fun, but have no experience
with scheme or packaging, so your explanations are very valuable to me.

Cheers
Tor-bj=C3=B6rn
--0000000000007c70fa05f20f52fe--