-
Notifications
You must be signed in to change notification settings - Fork 153
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
[Suggestion] Support for multiple projects with deliverable output #116
Changes from 24 commits
2f11147
0fa87c4
fc90f53
669ce37
3847785
cf5ce8e
bfe5e39
ff404ad
23e912a
482d2bf
382db99
509543f
59934d6
8f85784
7f9a9a8
f0a6b5c
fcbc5fd
21392ea
e4d793d
df9317f
80fb905
1efbffc
01537a3
0eec754
a89baab
8c3ccbe
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,7 +3,6 @@ | |
// -------------------------------------------------------------------------------------- | ||
|
||
#r @"packages/FAKE/tools/FakeLib.dll" | ||
|
||
open Fake | ||
open Fake.Git | ||
open Fake.AssemblyInfoFile | ||
|
@@ -100,6 +99,16 @@ Target "AssemblyInfo" (fun _ -> | |
csProjs |> Seq.iter genCSAssemblyInfo | ||
) | ||
|
||
// Copies binaries from default VS location to exepcted bin folder | ||
// But keeps a subdirectory structure for each project in the | ||
// src folder to support multiple project outputs | ||
Target "CopyBinaries" (fun _ -> | ||
directoryInfo "src" | ||
|> subDirectories | ||
|> Array.map (fun d -> (d.Name, sprintf "%s/bin/Release" d.FullName)) | ||
|> Array.iter (fun (n, p) -> CopyDir (sprintf "bin/%s" n) p (fun _ -> true)) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same here please use path.combine or @@ |
||
) | ||
|
||
// -------------------------------------------------------------------------------------- | ||
// Clean build results | ||
|
||
|
@@ -333,6 +342,7 @@ Target "All" DoNothing | |
"Clean" | ||
==> "AssemblyInfo" | ||
==> "Build" | ||
==> "CopyBinaries" | ||
==> "RunTests" | ||
=?> ("GenerateReferenceDocs",isLocalBuild) | ||
=?> ("GenerateDocs",isLocalBuild) | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -19,19 +19,19 @@ | |
<DebugType>full</DebugType> | ||
<Optimize>false</Optimize> | ||
<Tailcalls>false</Tailcalls> | ||
<OutputPath>..\..\bin</OutputPath> | ||
<OutputPath>.\bin\Debug</OutputPath> | ||
<DefineConstants>DEBUG;TRACE</DefineConstants> | ||
<WarningLevel>3</WarningLevel> | ||
<DocumentationFile>..\..\bin\FSharp.ProjectTemplate.xml</DocumentationFile> | ||
<DocumentationFile>.\bin\Debug\FSharp.ProjectTemplate.xml</DocumentationFile> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. should be FSharp.ProjectScaffold, right? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ok. my question is more: this gets correctly replaced, right? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. yes, from: https://github.com/fsprojects/ProjectScaffold/blob/master/init.fsx#L138 |> replace projectTemplateName projectName
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 👍 |
||
</PropertyGroup> | ||
<PropertyGroup Condition=" '$(Configuration)|$(Platform)' == 'Release|AnyCPU' "> | ||
<DebugType>pdbonly</DebugType> | ||
<Optimize>true</Optimize> | ||
<Tailcalls>true</Tailcalls> | ||
<OutputPath>..\..\bin</OutputPath> | ||
<OutputPath>.\bin\Release</OutputPath> | ||
<DefineConstants>TRACE</DefineConstants> | ||
<WarningLevel>3</WarningLevel> | ||
<DocumentationFile>..\..\bin\FSharp.ProjectTemplate.xml</DocumentationFile> | ||
<DocumentationFile>.\bin\Release\FSharp.ProjectTemplate.xml</DocumentationFile> | ||
</PropertyGroup> | ||
<ItemGroup> | ||
<Reference Include="mscorlib" /> | ||
|
@@ -72,4 +72,4 @@ | |
</Target> | ||
--> | ||
<Import Project="..\..\.paket\paket.targets" /> | ||
</Project> | ||
</Project> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use Path.Combine