[PATCH] gnu: texlive-xiften: Readd need input propagation

  • Done
  • quality assurance status badge
Details
3 participants
  • Isaac van Bakel
  • Nicolas Goaziou
  • Steve George
Owner
unassigned
Submitted by
Isaac van Bakel
Severity
normal
I
I
Isaac van Bakel wrote on 6 Dec 2023 15:01
(address . guix-patches@gnu.org)(name . Isaac van Bakel)(address . isaac.vanbakel@inf.ethz.ch)
9af482bdc36dfc17d767d7c1cc282e71bf529b68.1701871261.git.isaac.vanbakel@inf.ethz.ch
* gnu/packages/tex.scm (texlive-xfithen)[propagated-inputs]:
Readd propagation of `texlive-ifmtarg` to fix usage of package.

Change-Id: I9e5e3cfa0fad3dc9fd3b4950eedf7443b254d2f1
---
This was removed as a "spurious" propagated input by the
texlive-team-next changes, but in my usage it was not spurious and the
package propagation is needed to use `xifthen` correctly.

gnu/packages/tex.scm | 1 +
1 file changed, 1 insertion(+)

Toggle diff (16 lines)
diff --git a/gnu/packages/tex.scm b/gnu/packages/tex.scm
index 0d76710daa..934cdb2a80 100644
--- a/gnu/packages/tex.scm
+++ b/gnu/packages/tex.scm
@@ -70883,6 +70883,7 @@ (define-public texlive-xifthen
"0b33mlmnxsj5mi06v2w2zgamk51mgv1lxdr1cax8nkpn9g7n9axw")))
(outputs '("out" "doc"))
(build-system texlive-build-system)
+ (propagated-inputs (list texlive-ifmtarg))
(home-page "https://ctan.org/pkg/xifthen")
(synopsis "Extended conditional commands")
(description

base-commit: 8c294e43eb0b0ea61da7c0570872e0908f64c8ed
--
2.41.0
N
N
Nicolas Goaziou wrote on 6 Dec 2023 22:39
(name . Isaac van Bakel)(address . isaac.vanbakel@inf.ethz.ch)
87sf4fm2o9.fsf@nicolasgoaziou.fr
Hello,

Isaac van Bakel <isaac.vanbakel@inf.ethz.ch> writes:

Toggle quote (22 lines)
> * gnu/packages/tex.scm (texlive-xfithen)[propagated-inputs]:
> Readd propagation of `texlive-ifmtarg` to fix usage of package.
>
> Change-Id: I9e5e3cfa0fad3dc9fd3b4950eedf7443b254d2f1
> ---
> This was removed as a "spurious" propagated input by the
> texlive-team-next changes, but in my usage it was not spurious and the
> package propagation is needed to use `xifthen` correctly.
>
> gnu/packages/tex.scm | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/gnu/packages/tex.scm b/gnu/packages/tex.scm
> index 0d76710daa..934cdb2a80 100644
> --- a/gnu/packages/tex.scm
> +++ b/gnu/packages/tex.scm
> @@ -70883,6 +70883,7 @@ (define-public texlive-xifthen
> "0b33mlmnxsj5mi06v2w2zgamk51mgv1lxdr1cax8nkpn9g7n9axw")))
> (outputs '("out" "doc"))
> (build-system texlive-build-system)
> + (propagated-inputs (list texlive-ifmtarg))

This is a limitation in TeX Live, not Guix's, package definitions. The
dependency is not reported in their "texlive.tlpdb" file.

Note there are hundreds of such issues. We cannot realistically fix them
manually ; for example this change would need to be tested every time
Guix TeX packages are updated.

I suggest to use collections of packages to avoid messing too much with
individual dependencies, and possibly report those missing dependencies
to the TeX Live project.

Regards,
--
Nicolas Goaziou
N
N
Nicolas Goaziou wrote on 7 Dec 2023 12:52
(name . Isaac van Bakel)(address . isaac.vanbakel@inf.ethz.ch)
87o7f2mdpm.fsf@nicolasgoaziou.fr
Hello,

Isaac van Bakel <isaac.vanbakel@inf.ethz.ch> writes:

Toggle quote (14 lines)
> On 2023/12/06 22:39, Nicolas Goaziou wrote:
>> This is a limitation in TeX Live, not Guix's, package definitions. The
>> dependency is not reported in their "texlive.tlpdb" file.
>>
>> Note there are hundreds of such issues. We cannot realistically fix them
>> manually ; for example this change would need to be tested every time
>> Guix TeX packages are updated.
>
> Is there no way to declare that a package auto-generated from TeX Live
> should be "patched" in some way (in this case, to include the
> dependency)? I'm aware that Nix does something like this for their
> various package sets. Perhaps this would be more sustainable than
> a fully-manual process.

I don't know what Nix does. However, considering the sheer number of
packages involved, I don't believe in the sustainability of anything
even remotely manual.

Now let's suppose we solve this. E.g., the importer greps through ".ins"
or ".dtx" files to identify and declare a list of dependencies. Then
we'll face another, greater, issue: it will be no longer possible to
build TeX Live packages because of the dependencies loops we will have
just introduced. Let me insist of the fact that I'm not talking about an
hypothetical problem: I encountered it when I started to package the
missing TeX Live bits a few months ago.

If there's no solution, then there's no problem. ;)

Toggle quote (15 lines)
>> I suggest to use collections of packages to avoid messing too much with
>> individual dependencies, and possibly report those missing dependencies
>> to the TeX Live project.
>
> Unfortunately this particular package only appears in an unopinionated
> collection in latexextra, which is very heavy for something that
> I have to put in my manifest just to fix a dependency issue.
>
> I'll look into getting the dependency registered on TL, but it's worth
> noting that they also don't want the responsibility of dependency
> tracking. They also don't recommend that package authors bother to put
> dependency information in their packages.[0] I'm conscious that this
> is a complex burden, but ideally *someone* would take it on so that
> the software works.

The software works; the burden in on the user who wants to optimize
their TeX Live installation. I think this is a reasonable, and common,
expectation. I don't think the TeX Live installer takes a different
approach. On a different context, compare a user installing a full blown
DE, such as Gnome, to another one who picks a window manager, a file
manager, etc., and needs to glue these parts together to make them
behave as a coherent environment.

As a data point, I have a 20-ish TeX Live packages from latexetra
collection explicitly installed from my manifest as required
dependencies. This list hardly ever increases or changes, so this is
acceptable, at least for me.

Sorry for being pessimistic. Maybe Someone™ will find an unexpected
solution. Meanwhile, adding manually dependencies is, IMO, a no go.

Regards,
--
Nicolas Goaziou
I
I
Isaac van Bakel wrote on 7 Dec 2023 12:06
(name . Nicolas Goaziou)(address . mail@nicolasgoaziou.fr)
4ab29d4c-2876-9bdd-d880-a1f909cd8956@inf.ethz.ch
Hello,

On 2023/12/06 22:39, Nicolas Goaziou wrote:
Toggle quote (7 lines)
> This is a limitation in TeX Live, not Guix's, package definitions. The
> dependency is not reported in their "texlive.tlpdb" file.
>
> Note there are hundreds of such issues. We cannot realistically fix them
> manually ; for example this change would need to be tested every time
> Guix TeX packages are updated.

Is there no way to declare that a package auto-generated from TeX Live
should be "patched" in some way (in this case, to include the
dependency)? I'm aware that Nix does something like this for their
various package sets. Perhaps this would be more sustainable than a
fully-manual process.

Toggle quote (4 lines)
> I suggest to use collections of packages to avoid messing too much with
> individual dependencies, and possibly report those missing dependencies
> to the TeX Live project.

Unfortunately this particular package only appears in an unopinionated
collection in latexextra, which is very heavy for something that I have
to put in my manifest just to fix a dependency issue.

I'll look into getting the dependency registered on TL, but it's worth
noting that they also don't want the responsibility of dependency
tracking. They also don't recommend that package authors bother to put
dependency information in their packages.[0] I'm conscious that this is
a complex burden, but ideally *someone* would take it on so that the
software works.

Best,

Isaac

S
S
Steve George wrote on 31 Oct 14:44 +0100
closing 67660
(address . control@debbugs.gnu.org)(address . 67660-submitter@debbugs.gnu.org)
1730382169-1986-bts-steve@futurile.net
close 67660
Closing as for now we've decided that (a) we can't manually patch a plethora
of tex packages and (b) there's no-one working on automation (whether similar
or different to the Nix solution)
thanks
?
Your comment

Commenting via the web interface is currently disabled.

To comment on this conversation send an email to 67660@debbugs.gnu.org

To respond to this issue using the mumi CLI, first switch to it
mumi current 67660
Then, you may apply the latest patchset in this issue (with sign off)
mumi am -- -s
Or, compose a reply to this issue
mumi compose
Or, send patches to this issue
mumi send-email *.patch