Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Querylet CPAN distribution.

Report information
The Basics
Id: 72294
Status: open
Priority: 0/
Queue: Querylet

People
Owner: Nobody in particular
Requestors: corion [...] corion.net
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: "Pluggable" fork of Querylet on github
Date: Tue, 19 Jul 2011 22:47:12 +0200
To: Ricardo SIGNES <rjbs [...] cpan.org>
From: Max Maischein <corion [...] corion.net>
Hello Ricardo! I use Querylet a lot at $work, and I finally got around to rewriting it to be more extensible. The changes are _very_ preliminary, but I like them for now, as they will enable better error messages and better extensibility (I think - I haven't written any new features yet!). The changes currently live in the "pluggable" branch on Github [1]. The main change is to the Querylet parser, which is not one huge s/// statement anymore but splits up the sections into lightweight objects. My plans for these objects are: 1) Store line number information and include that information as #line comments, so that later error messages raised by Perl correspond to the original file and line number. 2) Add another "input" class that takes input parameters from the command line via Getopt::Long 3) Add a "title" option 4) Add a generic parser that can fetch the title and input parameters from a Querylet without running it (similar to ::CGI, but for automatic monthly reports and determining whether they need to run) 5) Add an "email the output" option 6) Add a "used parameters for this query" option that lists the parameters for _this_ result, for easy reproducibility and convenient auto-generation of filenames. All of these should be fairly easy to add once I've tested my changes. They already pass all original tests, but I haven't moved my production Querylet scripts over to the new version, so this still is somewhat theoretical. If you find time to look at my changes and whether they can make it into the mainline Querylet, that would be really great! If you have ideas on how to make the API better, these are also very welcome! Thanks for writing such a useful module! -max [1] https://github.com/Corion/querylet/tree/pluggable
Is this still something you're interested in seeing merged? -- rjbs
Subject: Re: [rt.cpan.org #72294] "Pluggable" fork of Querylet on github
Date: Thu, 31 Oct 2013 09:17:22 +0100
To: bug-Querylet [...] rt.cpan.org
From: Max Maischein <corion [...] corion.net>
Hello Ricardo, yes, I'm still interested, but my fork has not yet seen the use at $work I would like. So I'm not completely confident that my changes don't break too much. -max Ricardo Signes via RT <bug-Querylet@rt.cpan.org> schrieb: Show quoted text
><URL: https://rt.cpan.org/Ticket/Display.html?id=72294 > > >Is this still something you're interested in seeing merged? > >-- >rjbs
-- Diese Nachricht wurde von meinem Sony WM-2 Walkman gesendet.
Subject: Re: [rt.cpan.org #72294] "Pluggable" fork of Querylet on github
Date: Thu, 31 Oct 2013 10:02:45 -0400
To: Max Maischein via RT <bug-Querylet [...] rt.cpan.org>
From: Ricardo Signes <rjbs [...] cpan.org>
* Max Maischein via RT <bug-Querylet@rt.cpan.org> [2013-10-31T04:21:50] Show quoted text
> yes, I'm still interested, but my fork has not yet seen the use at $work I > would like. So I'm not completely confident that my changes don't break too > much.
Okay. At the very least, it may need rebasing atop my master, which was just dzilified. I will try to look at the branch as it stands, though, sometime *this year*. Sorry for the long delay. I'm still catching up. -- rjbs
Download signature.asc
application/pgp-signature 490b

Message body not shown because it is not plain text.

Subject: Re: [rt.cpan.org #72294] "Pluggable" fork of Querylet on github
Date: Fri, 08 Nov 2013 17:37:56 +0100
To: bug-Querylet [...] rt.cpan.org
From: Max Maischein <corion [...] corion.net>
Hello Ricardo, Show quoted text
> * Max Maischein via RT <bug-Querylet@rt.cpan.org> [2013-10-31T04:21:50]
>> yes, I'm still interested, but my fork has not yet seen the use at $work I >> would like. So I'm not completely confident that my changes don't break too >> much.
> > Okay. At the very least, it may need rebasing atop my master, which was just > dzilified. I will try to look at the branch as it stands, though, sometime > *this year*. Sorry for the long delay. I'm still catching up.
I tried rebasing, but I'm now stuck in a herd of yaks. Your new distribution uses Dist::Zilla, which does not install under Windows, because Data::Section fails its Windows-newline test on Windows. I believe that the test is simply wrong, but I don't know what the supposed test outcome is to be, and why. So until I get further tuits, I won't be rebasing my changes, as I can't find out whether Dist::Zilla would magically pull in the prerequisites I added (Module::Pluggable), or not. Sorry, -max
Subject: Re: [rt.cpan.org #72294] "Pluggable" fork of Querylet on github
Date: Fri, 8 Nov 2013 19:39:23 -0500
To: "bug-Querylet [...] rt.cpan.org" <bug-Querylet [...] rt.cpan.org>
From: Ricardo Signes <rjbs [...] cpan.org>
I will look into the DZ on Windows problem! Meantime, all I need is for `prove -lr t` to work. -- rjbs Show quoted text
> On Nov 8, 2013, at 11:38, "Max Maischein via RT" <bug-Querylet@rt.cpan.org> wrote: > > Queue: Querylet > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=72294 > > > Hello Ricardo, >
>> * Max Maischein via RT <bug-Querylet@rt.cpan.org> [2013-10-31T04:21:50]
>>> yes, I'm still interested, but my fork has not yet seen the use at $work I >>> would like. So I'm not completely confident that my changes don't break too >>> much.
>> >> Okay. At the very least, it may need rebasing atop my master, which was just >> dzilified. I will try to look at the branch as it stands, though, sometime >> *this year*. Sorry for the long delay. I'm still catching up.
> > I tried rebasing, but I'm now stuck in a herd of yaks. Your new > distribution uses Dist::Zilla, which does not install under Windows, > because Data::Section fails its Windows-newline test on Windows. I > believe that the test is simply wrong, but I don't know what the > supposed test outcome is to be, and why. > > So until I get further tuits, I won't be rebasing my changes, as I can't > find out whether Dist::Zilla would magically pull in the prerequisites I > added (Module::Pluggable), or not. > > Sorry, > -max > >