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

[META] encoding.pm issues #14543

Closed
p5pRT opened this issue Feb 26, 2015 · 8 comments
Closed

[META] encoding.pm issues #14543

p5pRT opened this issue Feb 26, 2015 · 8 comments
Labels

Comments

@p5pRT
Copy link

@p5pRT p5pRT commented Feb 26, 2015

Migrated from rt.perl.org#123945 (status was 'rejected')

Searchable as RT123945$

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Feb 26, 2015

From @tonycoz

encoding.pm is deprecated because its behaviour is just too strange to support.

This meta ticket lists tickets that report on that strange behaviour, or even bugs[1] in that behaviour, to be closed once the feature is removed.

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Feb 26, 2015

From @tonycoz

On Thu Feb 26 15​:39​:53 2015, tonyc wrote​:

... bugs[1] ...

[1] if you can tell what's buggy and what's expected behaviour

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Jun 16, 2015

From @tonycoz

On Thu Feb 26 15​:39​:53 2015, tonyc wrote​:

encoding.pm is deprecated because its behaviour is just too strange to
support.

This meta ticket lists tickets that report on that strange behaviour,
or even bugs[1] in that behaviour, to be closed once the feature is
removed.

Removed 31923, while the original ticket was about C<use encoding> other tickets more directly related to PerlIO​::encoding (which is the
based cause of the issue) have been merged into it.

Tony

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Mar 18, 2016

From @jkeenan

On Thu Feb 26 15​:39​:53 2015, tonyc wrote​:

encoding.pm is deprecated because its behaviour is just too strange to
support.

This meta ticket lists tickets that report on that strange behaviour,
or even bugs[1] in that behaviour, to be closed once the feature is
removed.

1. I don't see an RT listed in this meta ticket specifically for the removal of encoding.pm. Is there such a ticket? If not, when/where was the decision made to remove it?

2. Also, *which* 'encoding.pm' are we looking at? Once I build blead I see​:

#####
$ ls -l cpan/Encode/encoding.pm lib/encoding.pm ext/PerlIO-encoding/encoding.pm
-rw-rw-r-- 1 jkeenan jkeenan 22452 Feb 24 22​:25 cpan/Encode/encoding.pm
-rw-rw-r-- 1 jkeenan jkeenan 1219 Feb 24 22​:25 ext/PerlIO-encoding/encoding.pm
-r--r--r-- 1 jkeenan jkeenan 22452 Feb 24 22​:25 lib/encoding.pm
#####
... where, I presume, the first and third in the listing above are the same file.

Thank you very much.

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

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Mar 18, 2016

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

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Mar 18, 2016

From @rjbs

* James E Keenan via RT <perlbug-followup@​perl.org> [2016-03-18T07​:48​:34]

1. I don't see an RT listed in this meta ticket specifically for the removal
of encoding.pm. Is there such a ticket? If not, when/where was the decision
made to remove it?

I'd have to look, but it was decided within the last four years.

2. Also, *which* 'encoding.pm' are we looking at? Once I build blead I see​:

#####
$ ls -l cpan/Encode/encoding.pm lib/encoding.pm ext/PerlIO-encoding/encoding.pm
-rw-rw-r-- 1 jkeenan jkeenan 22452 Feb 24 22​:25 cpan/Encode/encoding.pm
-rw-rw-r-- 1 jkeenan jkeenan 1219 Feb 24 22​:25 ext/PerlIO-encoding/encoding.pm

The first, encoding.pm shipped with Encode. The second is PerlIO​::encoding,
the "encoding" PerlIO layer.

--
rjbs

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Jul 15, 2016

From @cpansprout

The encoding.pm functionality was removed in the branch merged as a9cb10c.

--

Father Chrysostomos

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Jul 15, 2016

@cpansprout - Status changed from 'open' to 'rejected'

@p5pRT p5pRT closed this Jul 15, 2016
@p5pRT p5pRT added the Severity Low label Oct 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant