Re: [O] add some babel supports (PHP, Lua, Redis)

2016-05-27 Thread numbch...@gmail.com
I see, I will check out them, and merge them if possible. I will PR after they are good enough. [stardiviner] GPG key ID: 47C32433 IRC(freeenode): stardiviner Twitter: @numbchild Key fingerprint = 9BAA 92BC CDDD B9EF 3B36 CB99 B8C4 B8E5 47C3 2433 Blog:

Re: [O] add some babel supports (PHP, Lua, Redis)

2016-05-27 Thread Rasmus
Hi, "numbch...@gmail.com" writes: > I modified most part of my files. Thanks. > - There are some places I can't improve because I'm not good at elisp. > (maybe other people can improve it later) Like ob-redis.el implement > ob-sql style configuration. and ob-lua.el

Re: [O] add some babel supports (PHP, Lua, Redis)

2016-05-22 Thread numbch...@gmail.com
I modified most part of my files. - There are some places I can't improve because I'm not good at elisp. (maybe other people can improve it later) Like ob-redis.el implement ob-sql style configuration. and ob-lua.el make use of lua-mode's running process. - I guess ob-php.el is the only one

Re: [O] add some babel supports (PHP, Lua, Redis)

2016-05-22 Thread Rasmus
Hi, Thanks for your patches. Sorry about the delay. I was hoping that someone with more ob knowledge would step in. Please find some comments below. I think some more work is needed before your libraries are "numbch...@gmail.com" writes: > From

Re: [O] add some babel supports (PHP, Lua, Redis)

2016-05-10 Thread numbch...@gmail.com
I moved files to contrib/lisp/. I live in China, hope FSF can assign the copyright assignment digitally. [stardiviner] GPG key ID: 47C32433 IRC(freeenode): stardiviner Twitter: @numbchild Key fingerprint = 9BAA 92BC CDDD B9EF 3B36 CB99 B8C4 B8E5 47C3 2433

Re: [O] add some babel supports (PHP, Lua, Redis)

2016-05-10 Thread Ken Mankoff
On 2016-05-10 at 11:58, Rasmus wrote: > Ken Mankoff writes: > >> On 2016-05-10 at 11:44, Rasmus wrote: >>> "numbch...@gmail.com" writes: >>> >>> Normally you "only" need to sign a piece of paper and send it by >>> snail

Re: [O] add some babel supports (PHP, Lua, Redis)

2016-05-10 Thread Rasmus
Ken Mankoff writes: > On 2016-05-10 at 11:44, Rasmus wrote: >> "numbch...@gmail.com" writes: >> >> Normally you "only" need to sign a piece of paper and send it by snail >> mail to the FSF office in the US of A. > > It is easier than this

Re: [O] add some babel supports (PHP, Lua, Redis)

2016-05-10 Thread Ken Mankoff
On 2016-05-10 at 11:44, Rasmus wrote: > "numbch...@gmail.com" writes: > > Normally you "only" need to sign a piece of paper and send it by snail > mail to the FSF office in the US of A. It is easier than this - it can be done via email. Print sign and scan

Re: [O] add some babel supports (PHP, Lua, Redis)

2016-05-10 Thread Rasmus
"numbch...@gmail.com" writes: > I tried to sign the FSF copyright assignment before, but it is very > difficult for me. So I give up. Is there other ways to merge this patches > without signing the assignment? (I can give up this right) Normally you "only" need to sign a

Re: [O] add some babel supports (PHP, Lua, Redis)

2016-05-10 Thread numbch...@gmail.com
I tried to sign the FSF copyright assignment before, but it is very difficult for me. So I give up. Is there other ways to merge this patches without signing the assignment? (I can give up this right) Second: I updated branch commit messages in the ChangeLog format. [stardiviner]

Re: [O] add some babel supports (PHP, Lua, Redis)

2016-05-10 Thread Rasmus
Hi, Thanks for your patches. Some comments follow. "numbch...@gmail.com" writes: > - ob-lua > - ob-php > - ob-redis > > https://github.com/stardiviner/org-mode > > Hope maintainer can merge those branches. First, do you have copyright assignment to the FSF or would you

[O] add some babel supports (PHP, Lua, Redis)

2016-05-10 Thread numbch...@gmail.com
I created three branches on my personal public org-mode repo on GitHub. - ob-lua - ob-php - ob-redis https://github.com/stardiviner/org-mode Hope maintainer can merge those branches. [stardiviner] GPG key ID: 47C32433 IRC(freeenode): stardiviner Twitter: