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

Add build scripts #1

Merged
merged 5 commits into from Oct 8, 2016
Merged

Add build scripts #1

merged 5 commits into from Oct 8, 2016

Conversation

moodmosaic
Copy link
Member

This Pull Request adds Paket as a dependency manager, and FAKE - F# Make as the DSL for build tasks.

Usage

$ ./build.sh

or in the Command Prompt (Windows)

build.cmd

@jacobstanley
Copy link
Member

Sounds good, let me just check I can still build everything with this setup (I'm on OS/X, using Xamarin Studio, although I'd be happy to use vim if it comes to it, that's what I use for everything else)

@jacobstanley
Copy link
Member

This works for me after a few changes which I can make after I merge this.

Can I ask what is your setup is? are you running on Windows with msys/mingw for your shell, or just the windows command prompt?

The build.sh doesn't work for me unless I tell it to run the executable with mono.

@moodmosaic
Copy link
Member Author

I'm currently on Windows, with Visual Studio 2015 and Visual Studio Code, using both mingw and cmd.exe 😃

@moodmosaic
Copy link
Member Author

moodmosaic commented Oct 8, 2016

Hmm... I think it's normal to run the build via Mono outside Windows (?) 😕

@jacobstanley
Copy link
Member

jacobstanley commented Oct 8, 2016

So I had to change the build.sh to this to make it work for me:

#!/bin/bash

mono .paket/paket.bootstrapper.exe
exit_code=$?
if [ $exit_code -ne 0 ]; then
  exit $exit_code
fi

mono .paket/paket.exe restore
exit_code=$?
if [ $exit_code -ne 0 ]; then
  exit $exit_code
fi

mono packages/FAKE/tools/FAKE.exe

But I suspect that will break it for you if you're running build.sh on Windows

@jacobstanley
Copy link
Member

But if you're just using build.cmd then all is good, I can make the change for me and it won't affect you

@moodmosaic
Copy link
Member Author

I think that should be fine for the beginning. 👍 We may then make build.sh more generic as we go along.

@jacobstanley
Copy link
Member

ok, great 👍 I'll merge this and make my fixes

I did some googling, looks like the environment variable COMSPEC should be set on Windows alone, so we can check for that and if so run without mono

@jacobstanley jacobstanley merged commit ec3c4e8 into hedgehogqa:master Oct 8, 2016
@moodmosaic moodmosaic deleted the build-scripts branch October 8, 2016 10:19
@moodmosaic
Copy link
Member Author

Sounds good! I'm currently on my phone with limited internet access, but I'll do some googling to this too.

@ghost ghost added this to the 0.1.0 milestone Sep 22, 2021
TysonMN pushed a commit that referenced this pull request Aug 11, 2023
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.

None yet

2 participants