-
Notifications
You must be signed in to change notification settings - Fork 146
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
diff sub command #84
Comments
Hm, interesting feature. That'd be quite useful. What should the syntax of the output look like? Just like p.s.: What's a "homestick" :-) |
I was thinking Maybe a DIFFTOOL env variable and/or --difftool='' flag would be useful. |
Hm, ok. I thought you meant that homeshick should show what was going to be symlinked and which files would conflict. I don't think it falls into the purview of homeshick to do such a fine-grained diff like you are suggesting. It'd be nice to somehow be able to integrate homeshick with a git alias though and through that obtain your suggested feature. To do that #70 ("make homeshick output machine readable") needs to be fixed first, then you could write all kinds of cool little scripts that combine homeshick and git features.
Did you know you can do this now?
That's a rather handy feature they introduced. |
Thanks for the tip on gitconfig-local. Already doing _local's with .bashrc and .vimrc. Homeshick already prompts to replace existing files with symlinks. But I never want to just say yes, I want to compare the files so I know what i'm losing if anything. Right now I frequently do So I figured I don't think git helps, since it's diffing a file in the repo to a static file outside of version control. If you think it's out of scope though, go ahead and close this. |
Ah, ok. So additionally to the |
That would be awesome! If it picked up an environment variable for the actual diff command that would be even better. I can take a look at the implementation as well. |
homeshick diff <castle> [<file1> <file2>]
Should diff the current files against the ones in the castle. useful before linking to know what's getting clobbered or should be merged in.
The text was updated successfully, but these errors were encountered: