zolrath / wemux

Multi-User Tmux Made Easy
MIT License
3.63k stars 139 forks source link

Update/correct `wemux config` -> `wemux conf` #19

Closed jcamenisch closed 10 years ago

kellym commented 11 years ago

I'm not sure why this is considered a correction. I prefer the current config syntax, and pretty much every other piece of software I use makes use of config, too.

jcamenisch commented 11 years ago

Because this is the way the software is actually coded. wemux conf works. wemux config does not. This is simply updating the docs to match the code.

Unless I somehow got an outdated branch of the project. Did this change at some point?

jcamenisch commented 11 years ago

With a quick check, I also see that the README itself is currently conflicted. The first mention under Homebrew (on OS X) says wemux conf, while wemux config is used farther down in the file.

kellym commented 11 years ago

Ah, it appears the current branch allows conf*, so you can type wemux conforgetyou and it still performs the same function. Sneaky way to do it... not my favorite, but it's not my project either. See https://github.com/zolrath/wemux/blob/master/wemux#L413

jcamenisch commented 11 years ago

Interesting. My branch must have been old. I think I installed it with apt-get. I only came across this because wemux config actually didn't work for me.

Not sure I have time to investigate why. :p

zolrath commented 10 years ago

I've tagged the newest version on git so various package managers can have the newest version which should fix these inconsistencies. Sorry about that!

kellym commented 10 years ago

No worries! Thanks for writing useful software!