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

POSIX.pm 1.47+ exports C99 math functions #14688

Closed
p5pRT opened this issue May 3, 2015 · 5 comments
Closed

POSIX.pm 1.47+ exports C99 math functions #14688

p5pRT opened this issue May 3, 2015 · 5 comments

Comments

@p5pRT
Copy link

p5pRT commented May 3, 2015

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

Searchable as RT124446$

@p5pRT
Copy link
Author

p5pRT commented May 3, 2015

From @tokuhirom

POSIX.pm 1.47+ exports C99 math functions by default.

It breaks code the code like following​:

```
package Foo;
sub round { }
package Bar;
use POSIX;
use parent qw(Foo);
__PACKAGE__->round();
```

Because C99 math functions have too generic name.

I suggest to remove these functions from default imports.

The issue happened on real code​:
yappo/p5-Geo-Coordinates-Converter#1

Issue is introduced from​:
7965ede#diff-08dc17f086c8dfcda1469a5d1ab1ad41

--
tokuhirom

@p5pRT
Copy link
Author

p5pRT commented May 8, 2015

From @rjbs

Thanks, this is fixed by a patch that will land today.

--
rjbs

@p5pRT
Copy link
Author

p5pRT commented May 8, 2015

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

@p5pRT
Copy link
Author

p5pRT commented May 9, 2015

From @rjbs

Applied. Exports have been restored to their perl 5.20.0 "glory."

--
rjbs

@p5pRT
Copy link
Author

p5pRT commented May 9, 2015

@rjbs - Status changed from 'open' to 'resolved'

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

No branches or pull requests

1 participant