[Catalyst-dev] Minor Catalyst::Script::Server bug - with patch!

[Louis Erickson]

Apparently "tonight" was optimistic, as that was nearly a month ago.  I have finally submitted a pull request to you.  It should be waiting for you in github.

There was one error before and after my changes - t/author/spelling.t failed.  I'll probably submit a patch for that, too because having make test fail annoyed me.

In my environment, it's quite useful to have an additional option to the server as well, '-h' to select which hostname to use.  Testing and winding up on the wrong interface is not my favorite.  I've been adding that to each app, but it appears to me to be a pretty generic option.  If I send along a pull request to add it, will you consider it?  (I'll have to find the documentation, and update that, too.)

On Jun 2, 2014, at 2:25 PM, Louis Erickson <lerickson at rdwarf.net> wrote:

> 
> On May 29, 2014, at 7:58 AM, John Napiorkowski <jjn1056 at yahoo.com> wrote:
> 
>> Hey,
>> 
>> Looks pretty good, any chance we could get a patch via github : perl-catalyst/catalyst-runtime?
> 
> Happy to do so, if I can figure out how.
> 
> I didn't know Catalyst was using github.  The Wiki points to a git repository at shadowcat.
> 
> I've used github less than the rest of git; what's the preferred process to get you the patch?  Their documentation suggests I fork, submit to my fork, then issue a "pull request"... is that what you need?
> 
> If so, I'll try that tonight.
> 
> _______________________________________________
> Catalyst-dev mailing list
> Catalyst-dev at lists.scsys.co.uk
> http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst-dev

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.scsys.co.uk/pipermail/catalyst-dev/attachments/20140625/9ff0785e/attachment.htm>