Sweet. One lil fix. #4

Merged
merged 2 commits into from Apr 19, 2012

Conversation

Projects
None yet
2 participants
Contributor

tnovelli commented Apr 19, 2012

No description provided.

bin/mad
@@ -76,7 +76,7 @@ display_mad_usage() {
#
install_all_remote() {
- local path=/usr/local/share/mad
+ local path=${MAD_PATH:-/usr/local/share/mad}
@tj

tj Apr 19, 2012

Owner

this isn't specifically one path, it's just like PATH so it's multiple colon-delimited paths. I guess we would iterate them and see which are writeable etc. Personally I hate how unix directories are structured haha but we should follow that I suppose.

@tnovelli

tnovelli Apr 19, 2012

Contributor

oh. yeah. besides, I shouldn't have to put export MAD_PATH=$HOME/.local/share/mad in my .profile. mad should just use ../share/mad. Ok, I figured it out., just gotta update my pull req... I'm still a github noob.

"hate how unix dirs are structured", yeah... heheh, those kinda comments jump right out at me now that I'm toying with a "new OS" kickstarter proposal :)
Even DOS has a better dir structure, organized by app.

@tj

tj Apr 19, 2012

Owner

yeah haha it would be much nice to keep modularity in..modules haha spreading yourself all over the FS is very rails-like

tj added a commit that referenced this pull request Apr 19, 2012

@tj tj merged commit 34e1a58 into tj:master Apr 19, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment