coder / coder/coder-desktop-macos

Investigate why app & system extension shared dependencies fail to embed

Abierto
#149 0 comentarios 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
Swift
Estrellas
29
Forks
8
Merge medio
18 h 14 min
PR fusionados (30 d)
4

Descripción

In both #145 and #98, I've encountered an issue with release builds of the app. These issues arose when a dependency was required by both the App target (or any of it's dependencies), and the system extension target (and any of it's dependencies).

In #98, the issue occurred when protobuf definitions were added directly to the app target, or a new framework target that would be depended on by the app target. They were previously working fine being depended on by just `VPNLib` (which the system extension depends on).
In the first case, the network extension would crash. In the second case, the app would crash.
The crash reason was:
```
Library not loaded: @rpath/SwiftProtobuf.framework/Versions/A/SwiftProtobuf
```

Other users have reported this issue https://github.com/apple/swift-protobuf/issues/1506#issuecomment-2435125065

In #143, the issue occured when `InternalCollectionsUtilities` was depended on by both the App target, and the VPN target (via another framework, VPNLib). When a library that depended on `swift-collections` was added to the app target, the network extension would crash on startup with the error:
```
Library not loaded: @rpath/InternalCollectionsUtilities.framework/Versions/A/InternalCollectionsUtilities
```

FWICT, xcode is making some form of optimisation where it constructs a framework for code shared between two different targets. This poses an issue with a system extension, which is copied and executed outside of the app bundle. A copy of the system extension always lives in the app bundle, so it's trivial for the app to depend on a framework present within the system extension bundle (we already do this), but not the other way around. If there was some way to tell xcode to embed this new framework within the system extension bundle, we wouldn't have this issue.

Instead, the workaround we've gone with is to simply put everything that would cause this issue in `VPNLib`, as to minimise the number of times we depend on any one package.

This issue doesn't require immediate action, and is more so for book-keeping. It would be good to post this alongside an MRE on the Apple developer forums and in Apple feedback assistant.

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Comienza reproduciendo los fallos de la compilación de lanzamiento descritos en los issues #145, #98 y #143, centrándote en las dependencias compartidas por los targets de extensión del sistema de App y VPN. Prepara un ejemplo mínimo reproducible que muestre el framework integrado que falta y, después, documenta el resultado para los Apple Developer Forums y Feedback Assistant, tal como se describe en el issue.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
swift
Área
build-system, desktop
Tipo de issue
Error
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Estancado
Claridad
Necesita aclaración
Aptitud para principiantes
30/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.