[bug#65745] [PATCH] gnu: yelp: Provide dependencies to display "man:" URIs.

[Maxim Cournoyer]

Hi,

Liliana Marie Prikler <liliana.prikler@gmail.com> writes:

> Am Samstag, dem 09.09.2023 um 18:45 +0200 schrieb Ludovic Courtès:
>> Hi,
>> 
>> Maxim Cournoyer <maxim.cournoyer@gmail.com> skribis:
>> 
>> > > +                   ;; Create a wrapper for 'man' that knows
>> > > where to find
>> > > +                   ;; 'groff' and 'gunzip'.
>> > > +                   (define libexecdir
>> > > +                     (string-append #$output "/libexec/yelp"))
>> > > +                   (define wrapper
>> > > +                     (string-append libexecdir "/man"))
>> > > +
>> > > +                   (mkdir-p libexecdir)
>> > > +                   (symlink (search-input-file inputs "bin/man")
>> > > wrapper)
>> > > +                   (wrap-program wrapper
>> > > +                     `("PATH" ":" =
>> > > +                       ,(map (lambda (program)
>> > > +                               (dirname (search-input-file
>> > > inputs program)))
>> > > +                             '("bin/groff" "bin/gunzip"))))
>> > > +
>> > > +                   ;; Have Yelp use that 'man' wrapper instead
>> > > of searching
>> > > +                   ;; for 'man' in $PATH.
>> > > +                   (substitute* "libyelp/yelp-man-parser.c"
>> > > +                     (("\"man\"")
>> > > +                      (string-append "\"" wrapper "\""))))))))
>> > 
>> > Couldn't we more simply wrap the yelp command with the 'man',
>> > 'groff' and
>> > 'gunzip' command locations added to PATH?
>> 
>> Yeah it’s convoluted; the idea was that the wrapper for ‘man’ is
>> something that should be fixed in ‘man-db’ proper but that we cannot
>> fix
>> in ‘master’ because ‘man-db’ has too many dependents.
>> 
>> Maybe I should start with it though…
> Can we graft our man-db to get the same effect?

We also have a core-updates branch; it could go there and we could merge
it with the docbook stuff that's going to go there.

I'd prefer this than convoluted solutions I don't understand :-).

-- 
Thanks,
Maxim