[perl #39850] [TODO] [CAGE] lack Parrot::Config confusing....

[Will Coleda]

# New Ticket Created by Will Coleda
# Please include the string: [perl #39850]
# in the subject line of all future correspondence about this issue.
# <URL: http://rt.perl.org/rt3/Ticket/Display.html?id=39850 >

To de-confuse and cut back on one of the most common recent FAQs:

1) Create a stub module of the same name and check it into svn. When
used, the module should:

a) check to see if Parrot::Config::Generated exists. If so, this
module should act as a proxy for that object.
b) If not, die with a helpful error message, like, "Unavailable
until parrot is configured."

2) Change the build process to generate Parrot::Configure::Generated
instead of Parrot::Config
3) Change the clean process to remove Parrot::Configure::Generated
instead of Parrot::Config

For extra points, the cage cleaner will quietly not install the proxy
class, and instead install the generated config as Parrot::Config.

--
Will "Coke" Coleda
wi...@coleda.com

[Will Coleda via RT]

Thanks, applied as r13647!

minor modification: makefile deps changed to be on Config/Generated.pm instead of just
Config/

Regards.

On Fri Jul 28 03:09:56 2006, guest wrote:

> On Sun Jul 16 10:06:46 2006, coke wrote:
> > To de-confuse and cut back on one of the most common recent FAQs:
> >
> > 1) Create a stub module of the same name and check it into svn. When
> > used, the module should:
> >
> > a) check to see if Parrot::Config::Generated exists. If so, this
> > module should act as a proxy for that object.
> > b) If not, die with a helpful error message, like, "Unavailable
> > until parrot is configured."
> >
> > 2) Change the build process to generate Parrot::Configure::Generated
> > instead of Parrot::Config
> > 3) Change the clean process to remove Parrot::Configure::Generated
> > instead of Parrot::Config

> I updated the MANIFEST to include lib/Parrot/Config.pm. I changed the
> generators for the makefiles to account for lib/Parrot/Config.pm and
> the object it proxies, lib/Parrot/Config/Generated.pm. The svn:ignore
> property has to be edited by a committer after the patch is applied
> because 'Config.pm' is added to the repository.

>
> > For extra points, the cage cleaner will quietly not install the proxy
> > class, and instead install the generated config as Parrot::Config.

> I did a dry run of make install yet I didn't notice
> lib/Parrot/Config.pm being installed anywhere, nor any of the other
> files under lib/Parrot.
>
> diffstat (doesn't show propedit for lib/Parrot):
> MANIFEST | 1 +
> MANIFEST.SKIP | 4 ++--
> config/gen/config_pm.pm | 12 ++++++++++--
> config/gen/config_pm/Config_pm.in | 15 +++++----------
> config/gen/makefiles/root.in | 2 +-
> lib/Parrot/Config.pm | 33 +++++++++++++++++++++++++++++++++
> 6 files changed, 52 insertions(+), 15 deletions(-)