Re: [PATCH] Completion: Improve _man

[dana <dana@dana.is>]

On 10 Jun 2018, at 08:07, Oliver Kiddle <okiddle@yahoo.co.uk> wrote:
>Is this the line where you removed * from:
> dirs=( $^_manpath/(sman|man|cat)${^sects}*/ )
>
>I think I've seen systems with directories named man1.Z/cat1.Z etc.
>HP/UX perhaps. Perhaps also locales have occured such as man3f.fi_FI
>I suspect the * was to handle something like that rather than for the
>undesirable functionality you describe.

It is, and that would make sense. If that is indeed the reason the pattern was
there, maybe it would be more accurate like this?

  dirs=( $^_manpath/(sman|man|cat)${^sects}(|.*)/ )

But then i think we'd need to strip the .* back out afterwards too.

On 10 Jun 2018, at 08:07, Oliver Kiddle <okiddle@yahoo.co.uk> wrote:
>It looks like the sections will be changing with Solaris 11.4:
>
>https://blogs.oracle.com/solaris/normalizing-man-page-section-numbers-in-solaris-114-v2

lol. Well... that's time well spent then.

Also i guess that change is going to introduce another issue like the current
_shutdown function has where relying on OSTYPE alone produces false positives
(because all of the Solaris variants report solaris2.11 or whatever, but some of
them are quite different from each other — now they'll have different man-page
structures).

>This results in all sections for which we don't have a description
>getting an empty description. The descriptions that we have are then
>pushed right over to the right like this:

Yeahhh. I identified that problem at some point, but then i forgot about the
actual implications. I like the _describe solution.

I'll try to fix any remaining issues with this later. Thanks!

dana