Skip to content
This repository has been archived by the owner on Oct 7, 2020. It is now read-only.

Add cmd script to install with cabal #1542

Merged
merged 6 commits into from Jan 4, 2020
Merged

Conversation

jneira
Copy link
Member

@jneira jneira commented Jan 4, 2020

Sorry for restore an os specific script but i've got tired of typing or copypasting it

@fendor
Copy link
Collaborator

fendor commented Jan 4, 2020

Well, I am fine with a single line script.

@alanz
Copy link
Collaborator

alanz commented Jan 4, 2020

I am happy with this too. Should we just put it there, or should there be something about it in the README?

When you invoke it, do you type cabal-install, or cabal-install.cmd? I am worried about the name, with respect to the package cabal-install too.

@jneira
Copy link
Member Author

jneira commented Jan 4, 2020

A comment in the readme could be useful, i'll do. I'll change it for install-with-cabal too.
You can invoke it without ext: install-with-cabal hie-8.6.5 f.e.

@fendor
Copy link
Collaborator

fendor commented Jan 4, 2020

That is a bit ambiguous, imo. Maybe a name like cabal-hie-install or something, making clear this is a hie specific installation script?

@alanz alanz merged commit e016450 into haskell:master Jan 4, 2020
@jneira jneira deleted the win-install branch January 4, 2020 23:56
@samuelpilz
Copy link
Contributor

I guess we should poke the cabal team to allow setting the cabal project file inside the script. Hvr has hinted that this should be no problem to support but nobody has done that yet

@jneira
Copy link
Member Author

jneira commented Jan 5, 2020

I guess we should poke the cabal team to allow setting the cabal project file inside the script. Hvr has hinted that this should be no problem to support but nobody has done that yet

The issue i opened has not much feedback 🤔

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants