Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ExtUtils::XSSymSet.3pm manual page is installed without installing the module #17424

Closed
ppisar opened this issue Jan 15, 2020 · 0 comments · Fixed by #17432
Closed

ExtUtils::XSSymSet.3pm manual page is installed without installing the module #17424

ppisar opened this issue Jan 15, 2020 · 0 comments · Fixed by #17432

Comments

@ppisar
Copy link
Contributor

ppisar commented Jan 15, 2020

I noticed that "make install" on Linux installs a manual page for ExtUtils::XSSymSet (e.g. /usr/local/share/man/man3/ExtUtils::XSSymSet.3), but the corresponding Perl module is missing (ExtUtils/XSSymSet.pm).

The explanation is in ./installperl:

sub installlib {
    ...
    return if $name eq 'ExtUtils/XSSymSet.pm' and !$Is_VMS;
    ...
}

It seems the same exception is not applied on manual pages.

Another possible fix would be removing the exception and installing the module unconditionally since the module can be used as a general string shortener:

"ExtUtils::XSSymSet" provides functions to
shorten names in a consistent fashion, and to track a set of names to
insure that each is unique. While designed with xsubpp in mind, it may
be used with any set of strings.
tonycoz added a commit to tonycoz/perl5 that referenced this issue Jan 20, 2020
This module is only installed on VMS, so there's not much point in
installing the man page.

An alternative would be to install the module on VMS, but it tries
to use configuration only set on VMS.

fixes Perl#17424
tonycoz added a commit that referenced this issue Jan 27, 2020
This module is only installed on VMS, so there's not much point in
installing the man page.

An alternative would be to install the module on VMS, but it tries
to use configuration only set on VMS.

fixes #17424
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants