-
Notifications
You must be signed in to change notification settings - Fork 6
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
Ready to Merge: Get Install on Linux Working #35
Conversation
@lfilho Also how do I change the prompt. With That does not work with your fork. I would really like to be able to change the prompt but I am locked into the below and cannot figure out how to change it: |
Took me a while to figure out I had to run |
Hey @daxgames thanks a lot for the contribution. Last time (long time ago) i installed it on linux it worked, possibly Brew changed their install path in the mean time. I'll just trust your changes and tests :) I try to install them afresh in my linux box soon. As for the Brewfile changes: i opted for alphabetical in order to minimize future diffs and make it easier to quickly see if a particular brew formula is or is not in there. If you needed to run bundle install a second time anyway (which is odd), should we keep the previous alphabetical order then? Aside this nit pick on the Brewfile, I'm ready to merge your PR, thanks! As for changing the prompt, oof, good question. It's been a while since i needed to change prompts or looked at that area of the code, so I don't know what's happening. I'll try to have a go at it (separate from this PR of course). |
@lfilho I just found some things I did wring and fixed them. I also figured out how to change the prompt about an hour ago. It's not pretty. See below(from my personal branch Not the one for this PR):
Only then could I change the prompt using the normal methods described above. I am switching back and forth between yours and mine until I am happy by simply checking out the branch I want to run and running Something is still off in my prompt after changing it but it is a LOT closer to what I am looking for. I think I am done with the PR though. |
I see. Cheer! I'm merging the PR but FYI will revert the Brewfile to be sorted again until we have better evidence that the sorting was causing any issues, ok? Thanks a lot for your contribution. Let me know if you any more good ideas :) |
No problem. Will do. |
I have been using skwp/dotfiles since 2013 and have been wanting to play with
nvim
.I use skwp/dotfiles on both MacOS and Linux and would like to have a single solution that can work with
nvim
on both as well as skwp/dotfiles does.I was aware of your fork and have tried it before without much success on Linux. Today I decided to try and fix the issues I saw when installing just so I could actually start using it and see if I want to move to it or create my own fork of it as I did for skwp/dotfiles.
What was not working
Failing during install on Linux because
brew
was not in the path after installing it.How it works now
The change is this PR allows
install.sh
on Linux to complete without error. As I have modified things it now installs on Linux and only:PlugInstall
oncenvim
is running and it is completely functional as far as I can tell.Please let me knwo If I missed anything.
Note: I renamed my skwp/dotfiles
~/.yadr
folder and installed using my changes from another branch in my fork. I have not done a fresh install on a clean system.Tested On
Questionable Changes
Linux brew env setup during the rakefle run is not as dynamic as I would like.
.yadr/Brewfile
Changed the order from alphabetical. Doubt this matters but below are my observations.
ffmpeg
,rename
Hung FOREVER on first runinstall.sh
and eventually failed to install.brew bundle install
manually a second time installed both successfully./home/linuxbrew
folder and reranyadr
installffmpeg
installed this timerename
did not install this time.brew bundle install
manually a second time installedrename
successfully.If I have put things in the wrong place or any changes to the PR are required please let me know and I will fix ASAP.
Thanks.
Dax