Skip to content

Commit

Permalink
Fix remove process - fixes #1596
Browse files Browse the repository at this point in the history
  • Loading branch information
forki committed Apr 12, 2016
1 parent 1ec6bb8 commit 34c9217
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 4 deletions.
3 changes: 3 additions & 0 deletions RELEASE_NOTES.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,6 @@
#### 2.58.15 - 12.04.2016
* BUGFIX: Remove process should remove packages from specified groups - https://github.com/fsprojects/Paket/issues/1596

#### 2.58.14 - 11.04.2016
* BUGFIX: Compare full filename for pack with template file - https://github.com/fsprojects/Paket/issues/1594

Expand Down
7 changes: 5 additions & 2 deletions src/Paket.Core/ProjectFile.fs
Original file line number Diff line number Diff line change
Expand Up @@ -501,8 +501,11 @@ module ProjectFile =
| None -> false
| Some fileName ->
let referencesFile = ReferencesFile.FromFile fileName
referencesFile.Groups.[groupName].NugetPackages
|> Seq.exists (fun p -> p.Name = package)
match referencesFile.Groups |> Map.tryFind groupName with
| None -> false
| Some group ->
group.NugetPackages
|> Seq.exists (fun p -> p.Name = package)

let deleteIfEmpty name (project:ProjectFile) =
let nodesToDelete = List<_>()
Expand Down
4 changes: 2 additions & 2 deletions src/Paket/Paket.fsproj
Original file line number Diff line number Diff line change
Expand Up @@ -41,8 +41,8 @@
<WarningLevel>3</WarningLevel>
<DocumentationFile>
</DocumentationFile>
<StartArguments>pack output out version 1.0.0 templatefile paket.template</StartArguments>
<StartWorkingDirectory>D:\code\Paket\integrationtests\scenarios\i001594-pack\temp</StartWorkingDirectory>
<StartArguments>remove nuget NuGet.CommandLine group Build</StartArguments>
<StartWorkingDirectory>D:\code\suave</StartWorkingDirectory>
</PropertyGroup>
<PropertyGroup>
<MinimumVisualStudioVersion Condition="'$(MinimumVisualStudioVersion)' == ''">11</MinimumVisualStudioVersion>
Expand Down

0 comments on commit 34c9217

Please sign in to comment.