Add installer

[syumai]

Related to #471 .

What this does

• Add command deno_install.
• This enables installing executable deno script in a easy way.

Usage

# Install deno script
deno_install https://deno.land/std/http/file_server.ts
# Now, you can run installed command
file_server

• For the details, please read the README.
• If you want to try behavior of this command, run the command below.

deno -A \
https://raw.githubusercontent.com/syumai/deno_std/add-installer/installer/deno_install.ts \
https://raw.githubusercontent.com/syumai/deno_std/add-installer/installer/deno_install.ts

TODO

• Remove wget.
  • This is necessary for parsing target script's shebang.
  • If deno's fetch supports redirect, this can be removed.
• Add tests.

Why I needed wget?

• deno's fetch can't get data from scripts using redirect server like deno.land. (e.g. https://deno.land/std/http/file_server.ts)
• fetch returns blank data.
• redirect destination script's body is needed to parse shebang.

[CLAassistant]

All committers have signed the CLA.

[ry]

Thanks @syumai

[kt3k]

@syumai
There seem to be a few lint errors:

/home/vsts/work/1/s/installer/mod.ts
  109:1   warning  Missing return type on function                                                                 @typescript-eslint/explicit-function-return-type
  128:1   warning  Missing return type on function                                                                 @typescript-eslint/explicit-function-return-type
  157:29  error    Array type using 'Array<Permission>' is forbidden for simple types. Use 'Permission[]' instead  @typescript-eslint/array-type

/home/vsts/work/1/s/installer/shebang.ts
   3:9   error    Array type using 'Array<string>' is forbidden for simple types. Use 'string[]' instead  @typescript-eslint/array-type
   8:25  error    Array type using 'Array<string>' is forbidden for simple types. Use 'string[]' instead  @typescript-eslint/array-type
  11:39  warning  Missing return type on function

[syumai]

@kt3k
Thanks! I'll fix them.

[bartlomieju]

This message should be displayed to user after step 1.

Also I think it makes sense to make it a param if message should be shown - if we add this script to deno as deno install then .deno/bin will already be in PATH.

How about Windows?

[bartlomieju]

It's pretty cool, but... not all scripts have shebang line - then it deno_install should handle manually specified perm flags as described by @ry:

So now people can run this:

deno_install file_server https://deno.land/std/http/file_server.ts --allow-write --allow-net

This will create a new executable shell script called $PATH/file_server, which would have the following contents:

#!/bin/sh
deno run --allow-write --allow-net https://deno.land/std/http/file_server.ts $@

If we skip this feature in first version we can drop wget as well

[bartlomieju]

Using DENO_DIR instead of default ~/.deno/bin will require to add $DENO_DIR/bin/ to path as well to run scripts from shell - maybe we can skip this step for now?

[bartlomieju]

What if installed module is JavaScript file?

[bartlomieju]

Won't second call overwrite first one? Maybe use template string?

const tpl = `#/bin/sh
${commands.join(' ')}`;
writeFileSync(FILE_PATH, encoder.encode(tpl));

[bartlomieju]

Will it work when path is empty?

[syumai]

@bartlomieju
I'm sorry for being late fixing my PR.
I've confirmed this PR. thanks!
#489
Should I close this PR?

[bartlomieju]

@syumai I wanted to get it up to speed but I can't push to your branch :) that's why I opened another one - we can merge them later

[syumai]

#489 completed implementing this. I close this PR.