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

I18N::Langinfo not documented as not available on MSWin32 #14571

Closed
p5pRT opened this issue Mar 9, 2015 · 11 comments
Closed

I18N::Langinfo not documented as not available on MSWin32 #14571

p5pRT opened this issue Mar 9, 2015 · 11 comments

Comments

@p5pRT
Copy link

@p5pRT p5pRT commented Mar 9, 2015

Migrated from rt.perl.org#124032 (status was 'resolved')

Searchable as RT124032$

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Mar 9, 2015

From @karenetheridge

I18N​::Langinfo is a core module, but apparently not available on Windows. It would be extremely helpful for module authors if this fact were noted in its documentation, so as to avoid the unfortunate situation of someone adding it as a prerequisite to their distribution, and therefore preventing installation on MSWin32.

(As well, could it be explained *why* it's not available? Perhaps a workaround can be found? or some interfaces stubbed out rather than the entire module not being installed?)

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Mar 9, 2015

From @karenetheridge

On Mon Mar 09 11​:12​:37 2015, ether wrote​:

I18N​::Langinfo is a core module, but apparently not available on
Windows. It would be extremely helpful for module authors if this fact
were noted in its documentation, so as to avoid the unfortunate
situation of someone adding it as a prerequisite to their
distribution, and therefore preventing installation on MSWin32.

(As well, could it be explained *why* it's not available? Perhaps a
workaround can be found? or some interfaces stubbed out rather than
the entire module not being installed?)

History of previous changes​: http​://www.nntp.perl.org/group/perl.perl5.changes/2005/04/msg11975.html

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Mar 9, 2015

From @karenetheridge

On Mon Mar 09 11​:38​:38 2015, ether wrote​:

On Mon Mar 09 11​:12​:37 2015, ether wrote​:

I18N​::Langinfo is a core module, but apparently not available on
Windows. It would be extremely helpful for module authors if this
fact
were noted in its documentation, so as to avoid the unfortunate
situation of someone adding it as a prerequisite to their
distribution, and therefore preventing installation on MSWin32.

This also raises another question -- do we have *anywhere* where we document
the matrix of module / platform compatibilities? If this was available
programmatically, we could write an author test or toolchain plugin around
it -- e.g. "assert that this distribution does not pull in prereqs that are
unavailable on certain platforms".

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Mar 10, 2015

From @jkeenan

On Mon Mar 09 11​:12​:37 2015, ether wrote​:

I18N​::Langinfo is a core module, but apparently not available on
Windows.

Can you identify the mechanism by which its unavailability on Windows is enforced?

#####
$ ack -i win ext/I18N-Langinfo
ext/I18N-Langinfo/Langinfo.c
65​:# if defined(__CYGWIN__) && defined(USE_DYNAMIC_LOADING)

ext/I18N-Langinfo/Langinfo.pm
98​:The following example will import the langinfo() function itself and
#####

I couldn't find anything specific in the code.

Thank you very much.

--
James E Keenan (jkeenan@​cpan.org)

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Mar 10, 2015

The RT System itself - Status changed from 'new' to 'open'

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Mar 10, 2015

From @karenetheridge

On Mon Mar 09 18​:02​:19 2015, jkeenan wrote​:

On Mon Mar 09 11​:12​:37 2015, ether wrote​:

I18N​::Langinfo is a core module, but apparently not available on
Windows.

Can you identify the mechanism by which its unavailability on Windows
is enforced?

Everything I know is in this ticket​: perl-pod/Pod-Spell#18

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Mar 10, 2015

From @jandubois

On Mon, Mar 9, 2015 at 6​:47 PM, Karen Etheridge via RT
<perlbug-followup@​perl.org> wrote​:

On Mon Mar 09 18​:02​:19 2015, jkeenan wrote​:

On Mon Mar 09 11​:12​:37 2015, ether wrote​:

I18N​::Langinfo is a core module, but apparently not available on
Windows.

Can you identify the mechanism by which its unavailability on Windows
is enforced?

Everything I know is in this ticket​: perl-pod/Pod-Spell#18

I18N​::Langinfo is not built on Windows because nl_langinfo() does not exist.

If you manually build and install it, it will just croak at runtime​:

SV*
langinfo(code)
  int code
  PROTOTYPE​: _
  CODE​:
#ifdef HAS_NL_LANGINFO
  RETVAL = newSVpv(nl_langinfo(code), 0);
#else
  croak("nl_langinfo() not implemented on this architecture");
#endif
  OUTPUT​:
  RETVAL

Cheers,
-Jan

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Mar 2, 2018

From @khwilliamson

As of 5.28, this module will be available on all platforms
--
Karl Williamson

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Mar 2, 2018

@khwilliamson - Status changed from 'open' to 'pending release'

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Jun 23, 2018

From @khwilliamson

Thank you for filing this report. You have helped make Perl better.

With the release yesterday of Perl 5.28.0, this and 185 other issues have been
resolved.

Perl 5.28.0 may be downloaded via​:
https://metacpan.org/release/XSAWYERX/perl-5.28.0

If you find that the problem persists, feel free to reopen this ticket.

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Jun 23, 2018

@khwilliamson - Status changed from 'pending release' to 'resolved'

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

No branches or pull requests

1 participant