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

[Louis Erickson]

I think I've done this right.  Git is making me slightly crazy; I've been using Perforce for ten years, and there are things that feel similar but aren't.

I've submitted a pull request for the stopwords to get t/author/spelling.t fixed.  Trivial, but it makes the tests pass.

I misremembered what it is I've been having to override Script/Server.pm to provide; it isn't an option for the hostname; there already is a -h option for that.  It's an environment variable to set the hostname, like you can set the port number.  I've got a change for that, too.  Shall I send the pull request for that?

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

> 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
> 
> _______________________________________________
> 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/20140626/2eaf0248/attachment.htm>