Internal headers (`crypto`, `internal` and `prov`) in `zig-out/include`

Abierto Apto para principiantes
#7 1 comentario 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
64/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Tranquilo
Stack tecnológico
zig
Área
build-system

Línea de trabajo

Comienza en build.zig alrededor de la línea 1255, donde el issue identifica la instalación de los headers de OpenSSL. Compara el árbol de include instalado del paquete con el árbol de include de OpenSSL upstream y comprueba si los consumidores requieren crypto, internal o prov. Se considera terminado cuando se confirmen los headers exportados previstos y se verifique el layout resultante de zig-out/include.

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

Descripción

Hi,
I'm new to Zig (2 days now) and I'm working on making https://github.com/kubernetes-client/c/ working with Zig as I'd like to rewrite some of my scripts into Zig as learning projects.

But the library depends on OpenSSL and it imports it in its public header files so when not using system's OpenSSL I do

b.installArtifact(openssl_lib);

so that the library is distributed in a working state when using vendored OpenSSL.

However the OpenSSL library installs some internal include directories, as far as I understand, which I don't think should be installed.

zig-out/include/
├── crypto
├── curl
├── internal
├── kubernetes
├── openssl
└── prov

7 directories, 0 files

The directories have some generic names and as far as my research goes these headers are internal and upstream only exports the openssl directory

/nix/store/dy64cxaygvmjfznysgxk501yds8jij6s-openssl-3.6.1-dev/include
└── openssl

2 directories

So is there a reason why

https://github.com/allyourcodebase/openssl/blob/6b318b447c8ff7529e3a2337a1df2a9b4816fee2/build.zig#L1255

isn't

lib.installHeadersDirectory(b.path("include/openssl"), "openssl", .{});

I'm opening an issue and not a PR as I'm not sure whether I'm missing something. If there are no requirements for those extra directories to be installed I'll gladly open a PR to make that change.

Lenguaje dominante
Assembly
Estrellas
28
Forks
18
Métricas de merge de PR
Sin PR fusionados en 30 d

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de allyourcodebase/openssl

Todos los issues de allyourcodebase/openssl

Issues similares

Más issues de Build System

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.