Attached is a patch that adds a simple fibers API to v8cgi. It probably doesn't work on all platforms but we have tested it on the mac and linux. And it's a module so it shouldn't hurt people who don't want it.
> Attached is a patch that adds a simple fibers API to v8cgi. It probably doesn't work on all platforms but we have tested it on the mac and linux. And it's a module so it shouldn't hurt people who don't want it.
> -- > You received this message because you are subscribed to the Google Groups "v8cgi" group. > To post to this group, send email to v8cgi@googlegroups.com. > To unsubscribe from this group, send email to v8cgi+unsubscribe@googlegroups.com. > For more options, visit this group at http://groups.google.com/group/v8cgi?hl=en. > <fibers_patch.txt>
On Wed, Oct 20, 2010 at 15:44, Darrell Brogdon <darr...@brogdon.net> wrote: > Curious. What are Fibers? I'd usually search before asking but I suspect > I'd get a massively un-useable result on a word like that. :)
> -D
> On Oct 20, 2010, at 4:32 PM, Malcolm Handley wrote:
> > Attached is a patch that adds a simple fibers API to v8cgi. It probably > doesn't work on all platforms but we have tested it on the mac and linux. > And it's a module so it shouldn't hurt people who don't want it.
> > -- > > You received this message because you are subscribed to the Google Groups > "v8cgi" group. > > To post to this group, send email to v8cgi@googlegroups.com. > > To unsubscribe from this group, send email to > v8cgi+unsubscribe@googlegroups.com <v8cgi%2Bunsubscribe@googlegroups.com>. > > For more options, visit this group at > http://groups.google.com/group/v8cgi?hl=en. > > <fibers_patch.txt>
> -- > You received this message because you are subscribed to the Google Groups > "v8cgi" group. > To post to this group, send email to v8cgi@googlegroups.com. > To unsubscribe from this group, send email to > v8cgi+unsubscribe@googlegroups.com <v8cgi%2Bunsubscribe@googlegroups.com>. > For more options, visit this group at > http://groups.google.com/group/v8cgi?hl=en.
> Attached is a patch that adds a simple fibers API to v8cgi. It probably > doesn't work on all platforms but we have tested it on the mac and linux. > And it's a module so it shouldn't hurt people who don't want it.
> -- > You received this message because you are subscribed to the Google Groups > "v8cgi" group. > To post to this group, send email to v8cgi@googlegroups.com. > To unsubscribe from this group, send email to > v8cgi+unsubscribe@googlegroups.com <v8cgi%2Bunsubscribe@googlegroups.com>. > For more options, visit this group at > http://groups.google.com/group/v8cgi?hl=en.
Sorry. I think that both of these problems were caused by my trying to split our changes into smaller patches to give you. I can still do that if you want but this time I'm attaching a patch with almost all of our changes.
Here's what the patch does:
- fixes DOMParser.prototype._unescapeEntities to globally unescape (versus just the first match). (dom.js) - adds a flush() method to File. (fs.js) - adds a fiber api (v8cgi.cc, src/lib/fibers, SConstruct) - adds support for enabling v8's profiler from JS (src/lib/profiler, SConstruct) - adds a memcached client (src/lib/memcached, SConstruct) - adds support for passing an environment when running subprocesses (process.cc) - adds getpid (system.cc) - links against the debug version of v8 when built with the debug flag (SConstruct). This may not be what you want but we find it useful to use a debug v8 when testing changes to v8cgi. (We like to get real stack traces, for example.)
I have applied this patch myself.
We have tests for many of these changes but they are written in our on JS testing framework. You are welcome to them if you like, though.
>> Attached is a patch that adds a simple fibers API to v8cgi. It probably >> doesn't work on all platforms but we have tested it on the mac and linux. >> And it's a module so it shouldn't hurt people who don't want it.
>> -- >> You received this message because you are subscribed to the Google Groups >> "v8cgi" group. >> To post to this group, send email to v8cgi@googlegroups.com. >> To unsubscribe from this group, send email to >> v8cgi+unsubscribe@googlegroups.com <v8cgi%2Bunsubscribe@googlegroups.com> >> . >> For more options, visit this group at >> http://groups.google.com/group/v8cgi?hl=en.
> -- > You received this message because you are subscribed to the Google Groups > "v8cgi" group. > To post to this group, send email to v8cgi@googlegroups.com. > To unsubscribe from this group, send email to > v8cgi+unsubscribe@googlegroups.com <v8cgi%2Bunsubscribe@googlegroups.com>. > For more options, visit this group at > http://groups.google.com/group/v8cgi?hl=en.
thanks a lot for your contribution. I just commited it with only very minor cosmetic changes, such as removal of unnecessary includes. I also need to test your new libraries (fibes, profiler, memcached) on Windows and adjust the build process accordingly.
I now need to properly document your three new modules. Are you able to provide the documentation (similar as other modules) in Google Code Wiki format? Alternatively, I can make you a project memeber, so you can edit wiki yourself. This seems to be a better alternative, as it allows you to commit fixes for these modules when necessary...
> Sorry. I think that both of these problems were caused by my trying to > split our changes into smaller patches to give you. I can still do that if > you want but this time I'm attaching a patch with almost all of our changes.
> Here's what the patch does:
> - fixes DOMParser.prototype._unescapeEntities to globally unescape (versus > just the first match). (dom.js) > - adds a flush() method to File. (fs.js) > - adds a fiber api (v8cgi.cc, src/lib/fibers, SConstruct) > - adds support for enabling v8's profiler from JS > (src/lib/profiler, SConstruct) > - adds a memcached client (src/lib/memcached, SConstruct) > - adds support for passing an environment when running subprocesses > (process.cc) > - adds getpid (system.cc) > - links against the debug version of v8 when built with the debug flag > (SConstruct). This may not be what you want but we find it useful to use a > debug v8 when testing changes to v8cgi. (We like to get real stack traces, > for example.)
> I have applied this patch myself.
> We have tests for many of these changes but they are written in our on JS > testing framework. You are welcome to them if you like, though.
> On Wed, Oct 20, 2010 at 23:43, Ondřej Žára <ondrej.z...@gmail.com> wrote:
>> Hi Malcom,
>> thanks a lot! This looks like a very nice and cool contribution.
>> 1) applying with patch -p0 < ... results in >> patch: **** malformed patch at line 34: def build_mysql(env):
>> what am I doing wrong?
>> 2) your patch also modifies SConstruct so it expects "profile" and >> "memcached" modules to be defined. Are you able to supply also these? :)
>>> Attached is a patch that adds a simple fibers API to v8cgi. It probably >>> doesn't work on all platforms but we have tested it on the mac and linux. >>> And it's a module so it shouldn't hurt people who don't want it.
>>> -- >>> You received this message because you are subscribed to the Google Groups >>> "v8cgi" group. >>> To post to this group, send email to v8cgi@googlegroups.com. >>> To unsubscribe from this group, send email to >>> v8cgi+unsubscribe@googlegroups.com<v8cgi%2Bunsubscribe@googlegroups.com> >>> . >>> For more options, visit this group at >>> http://groups.google.com/group/v8cgi?hl=en.
>> -- >> You received this message because you are subscribed to the Google Groups >> "v8cgi" group. >> To post to this group, send email to v8cgi@googlegroups.com. >> To unsubscribe from this group, send email to >> v8cgi+unsubscribe@googlegroups.com <v8cgi%2Bunsubscribe@googlegroups.com> >> . >> For more options, visit this group at >> http://groups.google.com/group/v8cgi?hl=en.
> -- > You received this message because you are subscribed to the Google Groups > "v8cgi" group. > To post to this group, send email to v8cgi@googlegroups.com. > To unsubscribe from this group, send email to > v8cgi+unsubscribe@googlegroups.com <v8cgi%2Bunsubscribe@googlegroups.com>. > For more options, visit this group at > http://groups.google.com/group/v8cgi?hl=en.
On Mon, Oct 25, 2010 at 06:49, Ondřej Žára <ondrej.z...@gmail.com> wrote: > Hi Malcom,
> thanks a lot for your contribution. I just commited it with only very minor > cosmetic changes, such as removal of unnecessary includes. I also need to > test your new libraries (fibes, profiler, memcached) on Windows and adjust > the build process accordingly.
Excellent. Thanks!
> I now need to properly document your three new modules. Are you able to > provide the documentation (similar as other modules) in Google Code Wiki > format? Alternatively, I can make you a project memeber, so you can edit > wiki yourself. This seems to be a better alternative, as it allows you to > commit fixes for these modules when necessary...
I'd be happy to be a project member and add the docs and commit changes directly. The one problem is that I don't have any Windows machines to test on so I'd be worried about breaking the build for Windows.
> Sorry. I think that both of these problems were caused by my trying to >> split our changes into smaller patches to give you. I can still do that if >> you want but this time I'm attaching a patch with almost all of our changes.
>> Here's what the patch does:
>> - fixes DOMParser.prototype._unescapeEntities to globally unescape (versus >> just the first match). (dom.js) >> - adds a flush() method to File. (fs.js) >> - adds a fiber api (v8cgi.cc, src/lib/fibers, SConstruct) >> - adds support for enabling v8's profiler from JS >> (src/lib/profiler, SConstruct) >> - adds a memcached client (src/lib/memcached, SConstruct) >> - adds support for passing an environment when running subprocesses >> (process.cc) >> - adds getpid (system.cc) >> - links against the debug version of v8 when built with the debug flag >> (SConstruct). This may not be what you want but we find it useful to use a >> debug v8 when testing changes to v8cgi. (We like to get real stack traces, >> for example.)
>> I have applied this patch myself.
>> We have tests for many of these changes but they are written in our on JS >> testing framework. You are welcome to them if you like, though.
>> On Wed, Oct 20, 2010 at 23:43, Ondřej Žára <ondrej.z...@gmail.com> wrote:
>>> Hi Malcom,
>>> thanks a lot! This looks like a very nice and cool contribution.
>>> 1) applying with patch -p0 < ... results in >>> patch: **** malformed patch at line 34: def build_mysql(env):
>>> what am I doing wrong?
>>> 2) your patch also modifies SConstruct so it expects "profile" and >>> "memcached" modules to be defined. Are you able to supply also these? :)
>>>> Attached is a patch that adds a simple fibers API to v8cgi. It probably >>>> doesn't work on all platforms but we have tested it on the mac and linux. >>>> And it's a module so it shouldn't hurt people who don't want it.
>>>> -- >>>> You received this message because you are subscribed to the Google >>>> Groups "v8cgi" group. >>>> To post to this group, send email to v8cgi@googlegroups.com. >>>> To unsubscribe from this group, send email to >>>> v8cgi+unsubscribe@googlegroups.com<v8cgi%2Bunsubscribe@googlegroups.com> >>>> . >>>> For more options, visit this group at >>>> http://groups.google.com/group/v8cgi?hl=en.
>>> -- >>> You received this message because you are subscribed to the Google Groups >>> "v8cgi" group. >>> To post to this group, send email to v8cgi@googlegroups.com. >>> To unsubscribe from this group, send email to >>> v8cgi+unsubscribe@googlegroups.com<v8cgi%2Bunsubscribe@googlegroups.com> >>> . >>> For more options, visit this group at >>> http://groups.google.com/group/v8cgi?hl=en.
>> -- >> You received this message because you are subscribed to the Google Groups >> "v8cgi" group. >> To post to this group, send email to v8cgi@googlegroups.com. >> To unsubscribe from this group, send email to >> v8cgi+unsubscribe@googlegroups.com <v8cgi%2Bunsubscribe@googlegroups.com> >> . >> For more options, visit this group at >> http://groups.google.com/group/v8cgi?hl=en.
> -- > You received this message because you are subscribed to the Google Groups > "v8cgi" group. > To post to this group, send email to v8cgi@googlegroups.com. > To unsubscribe from this group, send email to > v8cgi+unsubscribe@googlegroups.com <v8cgi%2Bunsubscribe@googlegroups.com>. > For more options, visit this group at > http://groups.google.com/group/v8cgi?hl=en.
> I now need to properly document your three new modules. Are you able to >> provide the documentation (similar as other modules) in Google Code Wiki >> format? Alternatively, I can make you a project memeber, so you can edit >> wiki yourself. This seems to be a better alternative, as it allows you to >> commit fixes for these modules when necessary...
> I'd be happy to be a project member and add the docs and commit changes > directly. The one problem is that I don't have any Windows machines to test > on so I'd be worried about breaking the build for Windows.
As for Windows build, no problem, I can handle that. I have a virtualized windows machine which I use for building v8cgi...
Please let me know what your gmail e-mail is, so I can add you as a project member.
>> Sorry. I think that both of these problems were caused by my trying to >>> split our changes into smaller patches to give you. I can still do that if >>> you want but this time I'm attaching a patch with almost all of our changes.
>>> Here's what the patch does:
>>> - fixes DOMParser.prototype._unescapeEntities to globally unescape >>> (versus just the first match). (dom.js) >>> - adds a flush() method to File. (fs.js) >>> - adds a fiber api (v8cgi.cc, src/lib/fibers, SConstruct) >>> - adds support for enabling v8's profiler from JS >>> (src/lib/profiler, SConstruct) >>> - adds a memcached client (src/lib/memcached, SConstruct) >>> - adds support for passing an environment when running subprocesses >>> (process.cc) >>> - adds getpid (system.cc) >>> - links against the debug version of v8 when built with the debug flag >>> (SConstruct). This may not be what you want but we find it useful to use a >>> debug v8 when testing changes to v8cgi. (We like to get real stack traces, >>> for example.)
>>> I have applied this patch myself.
>>> We have tests for many of these changes but they are written in our on JS >>> testing framework. You are welcome to them if you like, though.
>>> On Wed, Oct 20, 2010 at 23:43, Ondřej Žára <ondrej.z...@gmail.com>wrote:
>>>> Hi Malcom,
>>>> thanks a lot! This looks like a very nice and cool contribution.
>>>> 1) applying with patch -p0 < ... results in >>>> patch: **** malformed patch at line 34: def build_mysql(env):
>>>> what am I doing wrong?
>>>> 2) your patch also modifies SConstruct so it expects "profile" and >>>> "memcached" modules to be defined. Are you able to supply also these? :)
>>>>> Attached is a patch that adds a simple fibers API to v8cgi. It >>>>> probably doesn't work on all platforms but we have tested it on the mac and >>>>> linux. And it's a module so it shouldn't hurt people who don't want it.
>>>>> -- >>>>> You received this message because you are subscribed to the Google >>>>> Groups "v8cgi" group. >>>>> To post to this group, send email to v8cgi@googlegroups.com. >>>>> To unsubscribe from this group, send email to >>>>> v8cgi+unsubscribe@googlegroups.com<v8cgi%2Bunsubscribe@googlegroups.com> >>>>> . >>>>> For more options, visit this group at >>>>> http://groups.google.com/group/v8cgi?hl=en.
>>>> -- >>>> You received this message because you are subscribed to the Google >>>> Groups "v8cgi" group. >>>> To post to this group, send email to v8cgi@googlegroups.com. >>>> To unsubscribe from this group, send email to >>>> v8cgi+unsubscribe@googlegroups.com<v8cgi%2Bunsubscribe@googlegroups.com> >>>> . >>>> For more options, visit this group at >>>> http://groups.google.com/group/v8cgi?hl=en.
>>> -- >>> You received this message because you are subscribed to the Google Groups >>> "v8cgi" group. >>> To post to this group, send email to v8cgi@googlegroups.com. >>> To unsubscribe from this group, send email to >>> v8cgi+unsubscribe@googlegroups.com<v8cgi%2Bunsubscribe@googlegroups.com> >>> . >>> For more options, visit this group at >>> http://groups.google.com/group/v8cgi?hl=en.
>> -- >> You received this message because you are subscribed to the Google Groups >> "v8cgi" group. >> To post to this group, send email to v8cgi@googlegroups.com. >> To unsubscribe from this group, send email to >> v8cgi+unsubscribe@googlegroups.com <v8cgi%2Bunsubscribe@googlegroups.com> >> . >> For more options, visit this group at >> http://groups.google.com/group/v8cgi?hl=en.
> -- > You received this message because you are subscribed to the Google Groups > "v8cgi" group. > To post to this group, send email to v8cgi@googlegroups.com. > To unsubscribe from this group, send email to > v8cgi+unsubscribe@googlegroups.com <v8cgi%2Bunsubscribe@googlegroups.com>. > For more options, visit this group at > http://groups.google.com/group/v8cgi?hl=en.
On Mon, Oct 25, 2010 at 11:05, Ondřej Žára <ondrej.z...@gmail.com> wrote:
> I now need to properly document your three new modules. Are you able to >>> provide the documentation (similar as other modules) in Google Code Wiki >>> format? Alternatively, I can make you a project memeber, so you can edit >>> wiki yourself. This seems to be a better alternative, as it allows you to >>> commit fixes for these modules when necessary...
>> I'd be happy to be a project member and add the docs and commit changes >> directly. The one problem is that I don't have any Windows machines to test >> on so I'd be worried about breaking the build for Windows.
> As for Windows build, no problem, I can handle that. I have a virtualized > windows machine which I use for building v8cgi...
> Please let me know what your gmail e-mail is, so I can add you as a project > member.
> You received this message because you are subscribed to the Google Groups > "v8cgi" group. > To post to this group, send email to v8cgi@googlegroups.com. > To unsubscribe from this group, send email to > v8cgi+unsubscribe@googlegroups.com <v8cgi%2Bunsubscribe@googlegroups.com>. > For more options, visit this group at > http://groups.google.com/group/v8cgi?hl=en.
