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
[PATCH] use Importer to enhance Exporter in blead #15151
Comments
From @exodistAttached is a patch to perl blead. This patch brings in the Importer Key things to note: * Nothing existing needs to change. -Chad |
From @karenetheridgeSince this is adding a new module to core, Module::CoreList also needs to I'm presuming that the discussion of whether to adding Importer to core has On Tue, Jan 26, 2016 at 8:07 PM, Chad Granum <perlbug-followup@perl.org>
|
The RT System itself - Status changed from 'new' to 'open' |
From @exodistStrange, no CoreList tests failed. So yeah, the patch will need that added. There has not been a discussion about adding Importer to core, I expected that to happen in the comments on this ticket. What has happened is a discussion about improving Exporter.pm, that discussion also produced concerns that consumers of exports will have the burden of depending on a specific exporter version, having the Importer module is intended to solve that secondary problem. |
From @tonycozOn Tue Jan 26 20:07:39 2016, exodist7@gmail.com wrote:
This doesn't really solve the Exporter new features problem. It assumes that a module uses the Exporter package variables (@EXPORT etc), but a module might choose to supply extra export features by switching to Exporter::Tidy or Sub::Exporter instead, neither of which use the Exporter package variables*. The only way either new Exporter features or Importer are safe to use is for the upstream package author to promise that such features are available (and they implement them either by a dependency on the newer Exporter or by setting the variables Importer needs). Tony * from a quick look at the documentation |
From @rjbs* Chad Granum <perlbug-followup@perl.org> [2016-01-26T23:07:40]
I don't think I can get behind this. It's "I came up with an interface a couple weeks ago and now want to tie one of The docs on Importer.pm only mention that it has renaming as a thing, but it's This patch seems to me to introduce a piece of machinery that may be useful, -- |
From @exodistI think I did fail to communicate adequately about this, and that is on me. *(Skip to the end if you do not want the explanation, but want some other - After proposing the addition of '-as' to Exporter.pm there was a lot of - Someone pointed out that @ISA and base/parent seem to be a similar - I did not tackle Importer as something new, but instead as a refactor of - I used the Importer namespace for several reasons: Once I had it finished to my specification I decided I needed to be SURE it Once I had this I realized that the new Exporter.pm version was notably - It is faster Now, to answer a specific question: No, the patch to Exporter.pm is not I am not going to push to hard on this. I see a lot of benefit, and the *Other Options:* Take the refactor work form Importer.pm, but put it mostly into I can also strip out the extra features Importer.pm has if the problem is Import can be core'd without changing Exporter.pm, but it might as well -Chad On Sun, Feb 7, 2016 at 7:10 PM, Ricardo Signes <perl.p5p@rjbs.manxome.org>
|
Migrated from rt.perl.org#127384 (status was 'open')
Searchable as RT127384$
The text was updated successfully, but these errors were encountered: