search

carrot / accord-cli

compile any language from the command line
Other
8 stars 2 forks source link

get rid of --out flag #1

Open notslang opened 10 years ago

notslang commented 10 years ago

'cause pipes: 

$ accord -c license.md > license.html

yay, UNIX :smile:

jescalan commented 10 years ago

Will this make it not work with windows?

notslang commented 10 years ago

Windows has those too. Of course, they're slightly different than everyone else, and doesn't support named pipes, but you don't need those to do file writing.

AFAIK, every OS made in the last couple decades has them, including other non-UNIX ones like Plan 9, QNX, and BeOS... So we're pretty safe to rely on them being there.

jescalan commented 10 years ago

Ok, sounds good to me :+1:

notslang commented 10 years ago

Oh, wait - getting rid of this might interfere with the use of --watch... and watch will likely become useful after we have dependency tracking in accord-watcher, so we probably shouldn't just get rid of it and replace it with inotifywatch / fileschanged.

We should probably keep it, but just encourage users to specify the output file when the watcher is being used.

jescalan commented 10 years ago

Ok, so should we close this issue out?

notslang commented 10 years ago

It'll be closed automatically once https://github.com/slang800/accord-cli/commit/fd0248e0ef7e7cc72d5ec169398b7549bade8520 gets merged