Remove use of atomics library, use instead a Lock for the access.

Author: Nacho Bonafonte

Package.resolved

@@ -35,15 +35,13 @@ let package = Package(
         .package(name: "Thrift", url: "https://github.com/undefinedlabs/Thrift-Swift", from: "1.1.1"),
         .package(name: "swift-nio", url: "https://github.com/apple/swift-nio.git", from: "2.0.0"),
         .package(name: "grpc-swift", url: "https://github.com/grpc/grpc-swift.git", from: "1.0.0"),
-        .package(name: "swift-atomics", url: "https://github.com/apple/swift-atomics.git", "0.0.2"..."2.0.0"),
         .package(name: "swift-metrics", url: "https://github.com/apple/swift-metrics.git", from: "2.1.1"),
     ],
     targets: [
         .target(name: "OpenTelemetryApi",
                 dependencies: []),
         .target(name: "OpenTelemetrySdk",
-                dependencies: ["OpenTelemetryApi",
-                               .product(name: "Atomics", package: "swift-atomics")]),
+                dependencies: ["OpenTelemetryApi"]),
         .target(name: "ResourceExtension",
                 dependencies: ["OpenTelemetrySdk"],
                 path: "Sources/Instrumentation/SDKResourceExtension",

Package.swift

@@ -3,7 +3,6 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-import Atomics
 import Foundation
 import OpenTelemetryApi
 
@@ -73,9 +72,12 @@ public class RecordEventsReadableSpan: ReadableSpan {
     /// The end time of the span.
     public private(set) var endTime: Date?
     /// True if the span is ended.
-    fileprivate var endAtomic = ManagedAtomic<Bool>(false)
+    fileprivate var endAtomic = false
+    private let endSyncLock = Lock()
     public var hasEnded: Bool {
-        return self.endAtomic.load(ordering: .relaxed)
+        endSyncLock.withLock {
+            return endAtomic
+        }
     }
 
     private let eventsSyncLock = Lock()
@@ -260,9 +262,15 @@ public class RecordEventsReadableSpan: ReadableSpan {
     }
 
     public func end(time: Date) {
-        if endAtomic.exchange(true, ordering: .relaxed) {
+        var wasAlreadyEnded = false
+        endSyncLock.withLockVoid {
+            wasAlreadyEnded = endAtomic
+            endAtomic = true
+        }
+        if wasAlreadyEnded {
             return
         }
+
         eventsSyncLock.withLockVoid {
             attributesSyncLock.withLockVoid {
                 isRecording = false