On Tue, Jun 14, 2011 at 4:22 AM, Dima Kogan <[email protected]> wrote:
>
> I made some general error handling improvements. Existence of a
> runnable gnuplot is now verified in both Makefile.PL and at
> module-loading time. Supported gnuplot cmdline options are detected and
> only those that exist are used. No errors are read from the process
> itself. That is coming, once I figure out how to do that efficiently.
I'm not sure that there is an easy cross-platform
way to deal with process information. That tends
to be OS and command shell dependent---at least!
> I'll update the docs to conform with PDL::Doc guidelines.
Great. Accessible on-line docs make it much easier
to work with a large software package like PDL and
its component modules.
> It looks like scantree.pl finds the directory that PDL.pm lives in, and
> scans it for documentation. This is insufficient at least with Debian's
> install policy; possibly with others also. Debian places binary
> system-dependent libraries in /usr/lib/perl, and pure-perl system
> independent ones in /usr/share/perl. PDL.pm ends up in /usr/lib and
> PDL::Graphics::Gnuplot in /usr/share, so scantree.pl doesn't find it on
> my machine. How are you installing the module? Are you seeing a similar
> issue?
>
> I think scantree.pl should do something like
>
> foreach my $inc(@INC)
> {
> my $candidateDir = "$inc/PDL";
> scantree($candidateDir) if -d $candidateDir;
> }
>
> Sounds reasonable? Want a patch?
I think scantree.pl needs to be more flexible and documented
before general release as part of the PDL install process.
At least it would be nice to be able to give a list of directories
to search for docs to add to an existing database.
My initial focus is on the best way to allow PDL module
developers to add their docs to the online database as
part of the install process. It might make more sense
to expose the functionality needed as a routine in PDL::Doc
and then scantree.pl would be just a command line
wrapper for that. The function could be based on the
existing PDL::Doc::scantree() routine.
We should probably add the location of the PDL documentation
database to PDL::Config on install for easy reference.
--Chris
_______________________________________________
Perldl mailing list
[email protected]
http://mailman.jach.hawaii.edu/mailman/listinfo/perldl
