Hi Dima-

I cloned the repo and and it built out of the box for my
cygwin install.  Pretty impressive watching all the gnuplot
window appear when running tst.pl!  The build and tests
passed for win32 ActiveState perl but the tst.pl program
failed with a bunch of messages like:

  'gnuplot' is not recognized as an internal or external command,
  operable program or batch file.

So somehow the build and tests pass without the module
being able to run.  I suggest adding an INSTALL file with
directions on where to obtain gnuplot and that it is required
for the module to work.

Also, it is very helpful to module users if the module
install process is "friendly".  At the least, the tests should
indicate that gnuplot was not found and refer the user
to where/how to get it.

I had to change the git repository URL to http:// from
https:// before it would work.  Maybe that would make
sense for the meta data and your github resource.

Looking forward to seeing this on CPAN.  If the platform
issues can be resolved, it could be a strong addition
to the PDL development environment.

--Chris


Ideally, you would have a module like Alien::GnuPlot
to probe, configure, or build/install gnuplot as part of
the perl Makefile.PL stage of the build.
On Sun, Jun 12, 2011 at 6:26 PM, Dima Kogan <[email protected]> wrote:
> Hi. I wrote a Gnuplot plotter for PDL. PDL::Graphics::Gnuplot is
> currently at https://github.com/dkogan/PDL-Graphics-Gnuplot . I'd like
> to give this list a chance to comment before pushing this up to CPAN.
> The code is complete, for the most part. There are some improvements
> that I will make at some point, but those will be incremental.
> Some improvements currently on the list:
>
> - More specialized API functions, as more convenient interfaces to plot(). For
>  example plot3d(...) could be shorthand for plot(3d => 1, ...)
> - Better error handling by reading replies from the gnuplot child process
> - Examples section in the POD
>
> Comments welcome.
>
> dima
>
> _______________________________________________
> Perldl mailing list
> [email protected]
> http://mailman.jach.hawaii.edu/mailman/listinfo/perldl
>

_______________________________________________
Perldl mailing list
[email protected]
http://mailman.jach.hawaii.edu/mailman/listinfo/perldl

Reply via email to