search

begriffs / haskell-vim-now

One-line Haskell Vim install
MIT License
988 stars 100 forks source link

Fedora requires installing 'par' #251

Closed tsoernes closed 7 years ago

tsoernes commented 7 years ago

Also, I suggest detecting if neovim is installed and if so then skip installing vim.

begriffs commented 7 years ago

Thanks for the improvement. I don't see the part about the neovim check in the commits -- are you still working on it?

tsoernes commented 7 years ago

No, I'm unsure how to do it in a way that works across platforms

begriffs commented 7 years ago

The other case statements use cmdpkg par par, is there a reason to do it differently for fedora?

begriffs commented 7 years ago

@tsoernes any opinion about my cmdpkg question?

tsoernes commented 7 years ago

I'm not entirely sure why some packages are echoed and some are called with cmdpkg. As long as the package name is clearly mentioned if missing it's fine, bonus points if the installer script automatically calls dnf install for the required packages.

begriffs commented 7 years ago

Thanks again for pointing this issue out. I fixed it by installing par more uniformly on all platforms.