On Wed, Dec 15, 2004 at 11:10:49PM -0500, Bill Catlan wrote:
> In response to a couple of requests, I have packaged a release of FlexMart
> (formerly, DMart) and put it on SourceForge at flexmart-0.1. Here's the
> file download URL:
Wow, I haven't even been able to get your previous release tested and
you are rolling out another one.
> I welcome any questions, comments and feedback.
Praise:
- Nice error messages.
- Clean code
Unfortunately, I can't get any further because CGI::Session::MySQL is
looking for an autoload file that does not exist in the CGI::Session
distribution---set_table_n.al. Any ideas? I've appended the full error
message at the end of this reply.
Criticisms:
- Tedious, manual install.
- I don't use MySQL so had to install it and figure out how to
configure it in order to continue; support for alternative databases
such as PostgreSQL or SQLLite (which would require no external
database engine to be configured) would be great
- There are several module dependenices which I did not have installed
(DBD::MySQL, CGI::Session::MySQL, Error to name a few). A Build.PL
would be helpful to let me know what I need to install on my local
system. Including some of the pure Perl modules that have
appropriate licenses in a PAR file would be even better.
- You do not have any automated tests to confirm the installation or
the logic of your code. This concerns me and would prevent me from
going further with this codebase. I'd be willing to help you get the
Apache::Test framework working and writing some basic tests to get
you started.
- I don't understand why you have split the distribution into two
folders. I suppose it's for environments where the user does not
have access to the global httpd.conf file. However, having two sets
of configuration files is confusing. From the looks of things, it
appears that the files in fmdemo subclass/override those in the
flexmart directory. If this is the case, why have a global config
file at all? It seems like you could get rid of the pgman and conf
directories in flexmart and have fmdemo subclass your Flexmart
libraries.
Thanks,
William
----------
Uh oh ... there seems to have been an error. We apologize for any inconvenience.
The following error has occurred:
Can't locate auto/CGI/Session/MySQL/set_table_n.al in @INC (@INC contains: /home/william/local/flexmart-0.1/flexmart/lib /home/william/local/flexmart-0.1/fmdemo/lib/cgiapp /etc/perl /usr/local/lib/perl/5.8.4 /usr/local/share/perl/5.8.4 /usr/lib/perl5 /usr/share/perl5 /usr/lib/perl/5.8 /usr/share/perl/5.8 /usr/local/lib/site_perl /usr/local/lib/perl/5.8.3 /usr/local/share/perl/5.8.3 /usr/local/lib/perl/5.8.2 /usr/local/share/perl/5.8.2 . /usr/local/apache/ /usr/local/apache/lib/perl) at /home/william/local/flexmart-0.1/flexmart/lib/FlexMart/Session.pm line 35
The erroneous input:
Not available.
Suggested help:
The previous request generated a fatal error. The error has been logged and a system administrator has been notified. We apologize for any inconvenience.
----------
--
Knowmad Services Inc.
http://www.knowmad.com
---------------------------------------------------------------------
Web Archive: http://www.mail-archive.com/suppressed/
http://marc.theaimsgroup.com/?l=cgiapp&r=1&w=2
To unsubscribe, e-mail: suppressed
For additional commands, e-mail: suppressed
Mail converted by mhonarc 2.6.15
This archive provided courtesy of JSW4.NET, Internet Hosting Services for Small Business.