Skip to content
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

Added clean with platform agnostic workaround #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

RemcoSchrijver
Copy link
Contributor

@RemcoSchrijver RemcoSchrijver commented Jun 7, 2022

Yeah this isn't a pretty script. I first wanted to use ifeq like a in a proper Makefile but then windows started acting up with not being able to find that specific command. Read on stack overflow that it could be several issues with the path so I decided I wouldn't bother and work around it for now. (Because fixing path issues would make running this template a lot worse)

So right now I just added an or true to the back of the clean operations for both unix and windows and the only ugly thing remaining is that your terminal complains that it it doesn't recognize the command. No clue how to suppress that so if you know if that's possible that would be great.

This does fix the issue of lingering code in your build output and should also prevent people from writing huge swats of haskell code in the lib build folder.

Also added .vscode to gitignore. (I was also thinking of adding the lib file to gitignore because it's build output but that might cause some confusion?)

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

Successfully merging this pull request may close these issues.

1 participant