On Mon, Sep 06, 2004 at 11:09:01PM +0000, Mark Stosberg wrote:
> Yes, it was missing from the MANIFEST. This time I actually tested the
> distribution as it is distributed to insure this kind of problem is
> gone.
> 
> Here's an update with the missing file:
> 
> http://mark.stosberg.com/perl/CGI-Application-Plugin-ConfigAuto-0.21.tar.gz

That one tested fine.



> Let me know what you think. It's a simple enough module I think I'll go ahead and
> put it on CPAN unless I get some adverse feedback in the next couple days.

I think that I'm going to like Config::Auto. One problem I have with the
plugin is that there are places (e.g., tests and the Build.PL) where I
need data in the config file. What I've done is modify my subroutine
that retrieves the configuration data to check whether webapp is set. If
it is, then we use the plugin; if it isn't, then I use Config::Auto. I'm
sure this isn't ideal but I'm trying to refactor as little code as
possible while I get comfortable with this new system.

Also, your pod mentions that the cfg_file method should be called in the
instance script. All 674 of my tests are still passing even though I
don't call cfg_file until my setup() method is called. What are your
reasons for making the following statement:

    This plugin asks you to declare your config file locations in the
    instance scripts, where it will have minimum impact on your
    application.

What impact could it have that I may be overlooking? I've had no problem
loading my config file in the setup method so far.


Thanks,
William

-- 
Knowmad Services Inc.
http://www.knowmad.com

---------------------------------------------------------------------
Web Archive:  http://www.mail-archive.com/[EMAIL PROTECTED]/
              http://marc.theaimsgroup.com/?l=cgiapp&r=1&w=2
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to