Skip to content

Fix compilation warnings - #2235

Merged
dkovba merged 1 commit into
mainfrom
warnings
Sep 8, 2026
Merged

dkovba merged 1 commit into
mainfrom
warnings

Conversation

@dkovba

@dkovba dkovba commented Sep 3, 2026 •

Copy link
Copy Markdown
Contributor

Type of Change

  • Bug fix

Motivation and Context

Fix compilation warnings below. Similar to #2119.

/Users/Dmitry/Apple/container/Sources/Plugins/NetworkVmnet/NetworkVmnetHelper+Start.swift:70:13: warning: cannot use struct 'FilePath' in a property declaration member of a type not marked '@_implementationOnly'; 'SystemPackage' was not imported by this file
 68 |         }()
 69 | 
 70 |         var logRoot = LogRoot.path
    |             `- warning: cannot use struct 'FilePath' in a property declaration member of a type not marked '@_implementationOnly'; 'SystemPackage' was not imported by this file
 71 | 
 72 |         func run() async throws {

SystemPackage.FilePath:2:15: note: struct declared here
1 | @available(macOS 10.10, iOS 8.0, watchOS 2.0, tvOS 9.0, visionOS 1.0, *)
2 | public struct FilePath : Sendable {
  |               `- note: struct declared here
3 |     public init()
4 | }
/Users/Dmitry/Apple/container/Sources/Plugins/RuntimeLinux/RuntimeLinuxHelper+Start.swift:46:13: warning: cannot use struct 'FilePath' in a property declaration member of a type not marked '@_implementationOnly'; 'SystemPackage' was not imported by this file
 44 |         var root: String
 45 | 
 46 |         var logRoot = LogRoot.path
    |             `- warning: cannot use struct 'FilePath' in a property declaration member of a type not marked '@_implementationOnly'; 'SystemPackage' was not imported by this file
 47 | 
 48 |         var machServiceLabel: String {

SystemPackage.FilePath:2:15: note: struct declared here
1 | @available(macOS 10.10, iOS 8.0, watchOS 2.0, tvOS 9.0, visionOS 1.0, *)
2 | public struct FilePath : Sendable {
  |               `- note: struct declared here
3 |     public init()
4 | }

Testing

  • Tested locally

@dkovba
dkovba requested a review from jglogan September 3, 2026 20:41
@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown

Code Coverage

Tier Line Coverage
Unit 24.87%
Integration 66.65%
Combined 76.14%

@dkovba
dkovba merged commit 57ddcb3 into main Sep 8, 2026
4 checks passed
@dkovba
dkovba deleted the warnings branch September 8, 2026 19:41
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.

2 participants