Fix bug that causes ignored 'exclude' directive

Sources/PackageDescription/Package.swift

@@ -108,12 +108,12 @@ extension Package: TOMLConvertible {
         }
         result += "]\n"
 
-        for target in targets {
+        result += "\n" + "exclude = \(exclude)" + "\n"
 
+        for target in targets {
             result += "[[package.targets]]\n"
             result += target.toTOML()
         }
-        result += "\n" + "exclude = \(exclude)"
 
         return result
     }

Tests/dep/Fixtures/29_exclude_directory/Package.swift

@@ -1,7 +1,6 @@
 import PackageDescription
 
 let package = Package(
-
     name: "29_exclude_directory",
     exclude: ["FooLib"]
-)
+)

Tests/dep/Fixtures/32_exclude_directory_with_targets/BarLib/Bar.swift

@@ -0,0 +1,3 @@
+class Bar {
+    var bar: Int = 0
+}

Tests/dep/Fixtures/32_exclude_directory_with_targets/FooBarLib/FooBar.swift

@@ -0,0 +1,3 @@
+class FooBar{
+    var bar: Int = 0 
+}

Tests/dep/Fixtures/32_exclude_directory_with_targets/FooLib/Foo.swift

@@ -0,0 +1,3 @@
+class Foo {
+    var bar: Int = 0
+}

Tests/dep/Fixtures/32_exclude_directory_with_targets/Package.swift

@@ -0,0 +1,7 @@
+import PackageDescription
+
+let package = Package(
+    name: "32_exclude_directory_with_targets",
+    targets: [Target(name: "FooLib")],
+    exclude: ["FooLib"]
+)

Tests/dep/FunctionalBuildTests.swift

@@ -46,6 +46,7 @@ class FunctionalBuildTests: XCTestCase, XCTestCaseProvider {
             ("testExecDeps", testExecDeps),
             ("testMultDeps", testMultDeps),
             ("testExcludeDirs", testExcludeDirs),
+            ("testExcludeWithTarget", testExcludeWithTarget),
             ("test_exdeps", test_exdeps),
             ("test_exdeps_canRunBuildTwice", test_exdeps_canRunBuildTwice),
             ("test_get_ExternalDeps", test_get_ExternalDeps),
@@ -391,6 +392,18 @@ class FunctionalBuildTests: XCTestCase, XCTestCaseProvider {
         }
     }
 
+    // 32 exclude directories (with targets)
+    // (see https://github.com/apple/swift-package-manager/pull/83)
+    func testExcludeWithTarget() {
+        let filesToVerify = ["BarLib.a", "FooBarLib.a"]
+        let filesShouldNotExist = ["FooLib.a"]
+        fixture(name: "32_exclude_directory_with_targets") { prefix in
+            XCTAssertNotNil(try? executeSwiftBuild(prefix))
+            XCTAssertTrue(self.verifyFilesExist(filesToVerify, fixturePath: prefix))
+            XCTAssertFalse(self.verifyFilesExist(filesShouldNotExist, fixturePath: prefix))
+        }
+    }
+
     func test_exdeps() {
         fixture(name: "102_mattts_dealer") { prefix in
             let prefix = Path.join(prefix, "app")