Re: Patch 8.0.1458

2018-02-03 Fir de Conversatie Matěj Cepl
On 2018-02-03, 21:35 GMT, Bram Moolenaar wrote:
> I notice that "js" is another name for nodejs.  I don't see
> "js{version}", would one use that in a script shebang line?

Probably not. I admit these scripts are quite rare 
(surprisingly), but I guess it can be alias to 
$DEFAULT_JS_INTEPRETER quite easily, so it is probably The Right 
Thing™ to check for.

Best,

Matěj
-- 
http://matej.ceplovi.cz/blog/, Jabber: mceplceplovi.cz
GPG Finger: 3C76 A027 CA45 AD70 98B5  BC1D 7920 5802 880B C9D8
 
"Push to test." (click) "Release to detonate..."
 -- from a bugzilla quip list

-- 
-- 
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups 
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Re: Patch 8.0.1458

2018-02-03 Fir de Conversatie Bram Moolenaar

Matěj Cepl wrote:

> On 2018-02-03, 14:56 GMT, Bram Moolenaar wrote:
> > +   \ 'python': [['#!/path/python']],
> 
> What about /usr/bin/python2 and /usr/bin/python3

We can check for those too.

> > +   \ 'javascript': [['#!/path/node'],
> > +   \['#!/path/nodejs'],
> > +   \['#!/path/rhino']],
> 
> /usr/bin/js and /usr/bin/js{version} ? (We have Mozilla's 
> Spidermonkey packaged as such in Fedora/RHEL, but I guess it is 
> available for other Linuces as well, I see it as libmozjs-*-bin 
> packages on Debian as well)

I notice that "js" is another name for nodejs.  I don't see
"js{version}", would one use that in a script shebang line?

-- 
ARTHUR:  Who are you?
TALL KNIGHT: We are the Knights Who Say "Ni"!
BEDEVERE:No!  Not the Knights Who Say "Ni"!
 "Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD

 /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net   \\\
///sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\  an exciting new programming language -- http://www.Zimbu.org///
 \\\help me help AIDS victims -- http://ICCF-Holland.org///

-- 
-- 
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups 
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Re: Patch 8.0.1458

2018-02-03 Fir de Conversatie Matěj Cepl
On 2018-02-03, 14:56 GMT, Bram Moolenaar wrote:
> +   \ 'python': [['#!/path/python']],

What about /usr/bin/python2 and /usr/bin/python3

> +   \ 'javascript': [['#!/path/node'],
> +   \['#!/path/nodejs'],
> +   \['#!/path/rhino']],

/usr/bin/js and /usr/bin/js{version} ? (We have Mozilla's 
Spidermonkey packaged as such in Fedora/RHEL, but I guess it is 
available for other Linuces as well, I see it as libmozjs-*-bin 
packages on Debian as well)

Best,

Matěj
-- 
http://matej.ceplovi.cz/blog/, Jabber: mceplceplovi.cz
GPG Finger: 3C76 A027 CA45 AD70 98B5  BC1D 7920 5802 880B C9D8
 
Afraid to die alone?
Become a bus driver.
  -- alleged easter egg in notepad++

-- 
-- 
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups 
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Patch 8.0.1458

2018-02-03 Fir de Conversatie Bram Moolenaar

Patch 8.0.1458
Problem:Filetype detection test does not check all scripts.
Solution:   Add most scripts to the test
Files:  src/testdir/test_filetype.vim


*** ../vim-8.0.1457/src/testdir/test_filetype.vim   2018-01-29 
09:17:28.670732722 +0100
--- src/testdir/test_filetype.vim   2018-02-03 15:54:29.878525301 +0100
***
*** 542,547 
--- 542,581 
\ 'strace': [['execve("/usr/bin/pstree", ["pstree"], 0x7ff0 /* 63 vars 
*/) = 0'],
\   ['15:17:47 execve("/usr/bin/pstree", ["pstree"], ... 
"_=/usr/bin/strace"]) = 0'],
\   ['__libc_start_main and something']],
+   \ 'clojure': [['#!/path/clojure']],
+   \ 'scala': [['#!/path/scala']],
+   \ 'tcsh': [['#!/path/tcsh']],
+   \ 'zsh': [['#!/path/zsh']],
+   \ 'tcl': [['#!/path/tclsh'],
+   \ ['#!/path/wish'],
+   \ ['#!/path/expectk'],
+   \ ['#!/path/itclsh'],
+   \ ['#!/path/itkwish']],
+   \ 'expect': [['#!/path/expect']],
+   \ 'gnuplot': [['#!/path/gnuplot']],
+   \ 'make': [['#!/path/make']],
+   \ 'pike': [['#!/path/pike'],
+   \  ['#!/path/pike0'],
+   \  ['#!/path/pike9']],
+   \ 'lua': [['#!/path/lua']],
+   \ 'perl6': [['#!/path/perl6']],
+   \ 'perl': [['#!/path/perl']],
+   \ 'php': [['#!/path/php']],
+   \ 'python': [['#!/path/python']],
+   \ 'groovy': [['#!/path/groovy']],
+   \ 'ruby': [['#!/path/ruby']],
+   \ 'javascript': [['#!/path/node'],
+   \['#!/path/nodejs'],
+   \['#!/path/rhino']],
+   \ 'bc': [['#!/path/bc']],
+   \ 'sed': [['#!/path/sed']],
+   \ 'ocaml': [['#!/path/ocaml']],
+   \ 'awk': [['#!/path/awk']],
+   \ 'wml': [['#!/path/wml']],
+   \ 'scheme': [['#!/path/scheme']],
+   \ 'cfengine': [['#!/path/cfengine']],
+   \ 'erlang': [['#!/path/escript']],
+   \ 'haskell': [['#!/path/haskell']],
\ }
  
  func Test_script_detection()
*** ../vim-8.0.1457/src/version.c   2018-02-03 15:43:11.031507374 +0100
--- src/version.c   2018-02-03 15:55:21.649850216 +0100
***
*** 773,774 
--- 773,776 
  {   /* Add new patch number below this line */
+ /**/
+ 1458,
  /**/

-- 
E  M  A  C  S
s  e  l  o  h
c  t  t  n  i
a  a t  f
pr  t
eo
 l

 /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net   \\\
///sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\  an exciting new programming language -- http://www.Zimbu.org///
 \\\help me help AIDS victims -- http://ICCF-Holland.org///

-- 
-- 
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups 
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.