Subject: | Embed (or workaround) dependency on Package::Constants |
Archive::Tar::Constants uses Package::Constants to generate its @EXPORT list. And since Archive::Tar is part of the CORE, so is Package::Constants. I understand the utility of Package::Constants, but I don't feel it belongs in the core.
There are many ways to effectively declare a "constant", and Package::Constants only works for one of them. So having this package in the CORES gives it a status of authority and completeness that isn't really warranted.
As far as I can tell, Archive::Tar is the only core module that uses Package::Constant. It is a fairly small package, so I humbly suggest embedding it directly in Archive::Tar::Constants or just avoid using it entirely. That way, Package::Constants can be removed from the CORE in a future release.
Thanks for your consideration, and for this wonderful module.