aurora / rmate

Remote TextMate 2 implemented as shell script

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Support for opening more than one file at the same time

miceno opened this issue · comments

When invoking rmate with more than one file location, the script complains:

server# rmate conf/last_httpd.conf conf/last_httpd.include 
dirname: extra operand `conf/last_httpd.include'
Try `dirname --help' for more information.
/root/bin/rmate: line 226: [: conf/last_httpd.conf: 
  binary operator expected

#46 added this support.

+1 for this to be merged (in a way that is supported by Sublime)

Ok ... i think i'll have some time to look into this end of january. I am still a little unhappy about the amount of changes required ... isn't there a more simple solution?

Actually, I didn't change a lot of code, however, I have to wrap a chunk of code in a function in order to dispatch it multiple times. I can leave the code inside the function un-intented if you prefer.

No, the indent is absolutely ok ... it's the usage of bash arrays i dislike a little bit ;) ... but i have to admit, that this is only personal taste ... ok. Let's give me some more days, let's say until end of january. I hadn't have time yet to test this. But i'll do so and if everything works finde, i'll going to merge this!

Actually, I also want to know if there are any possible alternatives to bash arrays.

Ok, i was not able to make a decision by end of january. I am sorry about this. But the more i think about the feature, the more i agree, that it would be useful to have. So currently i am working on it, but with a little different approach as yours @randy3k . It's currently to early to show something, but i just wanted to let you know, that i am working on it ...

Thanks, I am looking forward to the update.

This was implemented in #46.