`pkg_rpm` cannot set different permissions to the executable in `pkg_files` and to its runfiles
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Área
- build-system
Línea de trabajo
Comienza con la lógica de detección de ejecutables y configuración del modo en pkg/private/pkg_files.bzl, en las ubicaciones enlazadas, y después sigue cómo pkg_rpm consume PackageFilesInfo y gestiona include_runfiles. Revisa las pruebas relevantes de pkg_rpm o añade una reproducción específica para un ejecutable con runfiles. Se considera terminado cuando el ejecutable y sus runfiles pueden recibir modos diferentes, y el ejecutable conserva su bit de ejecución previsto.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The problem
pkg_tar automatically detects the executable in pkg_files:
https://github.com/bazelbuild/rules_pkg/blob/c53ff51070c96b368188f057a6683d97c31b77f7/pkg/private/pkg_files.bzl#L395
And sets its execution bit:
https://github.com/bazelbuild/rules_pkg/blob/c53ff51070c96b368188f057a6683d97c31b77f7/pkg/private/pkg_files.bzl#L411
pkg_rpm does not seem to do anything similar, so executables end up with the same file mode as the rest of the files in the provider, which is usually a mode without the execution bit.
Workaround
A workaround is to set the execution bit for the pkg_files target, which will set it for the executable and all its runfiles, but this is not ideal. It would be better if the default was to preserve the file mode of the source file, I think that would work reasonably well.
Future work
This highlights a general issue with the include_runfiles attribute in pkg_files: because the executable and all its runfiles are grouped inside a single PackageFilesInfo provider, there is no granularity to have different attributes or prefixes for different files.
There is also the issue of executables being included twice, as the main target, and as one of the runfiles, which makes a warning show up (file mapped to two different destinations). This should probably have its own issue though.
- Lenguaje dominante
- Starlark
- Estrellas
- 254
- Forks
- 225
- Merge medio
- 2 d 4 h
- PR fusionados (30 d)
- 3
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de bazelbuild/rules_pkg
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
bazelbuild/rules_pkg#1093 ·
Los mantenedores suelen responder en 1 día
-
Targets rely on implicit `__init__.py` creation, causing rules_python deprecation warningsPosiblemente ocupada @onnosteenbergen-samotics la tomó hace 31 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
bazelbuild/rules_pkg#1081 · 3 comentarios ·
Los mantenedores suelen responder en 1 día
-
design feature-request
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
bazelbuild/rules_pkg#1080 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
bazelbuild/rules_pkg#1075 · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
bazelbuild/rules_pkg#1066 · 1 reacción ·
Los mantenedores suelen responder en 1 día
Todos los issues de bazelbuild/rules_pkg
Issues similares
-
orchestration.migration.data.tasks.retry values are ignored by the 8.8 data migration jobPosiblemente ocupada @irisfield la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
camunda/camunda-platform-helm#7773 ·
Los mantenedores suelen responder en 1 día
-
0.kind: enhancement 6.topic: reproducible builds
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
NixOS/nixpkgs#580765 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
win_build: parallel PostBuildEvent race in example_app_atiopencl and example_app_nvopencl copying openclapp_kernels.clPosiblemente ocupada @qixuancao la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 73/100
Los mantenedores suelen responder en 1 día
-
area: workspace needs: repro priority: medium
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
[Bug]: CSS extract HMR never removes old stylesheet when the filename contains a queryPosiblemente ocupada @ebidel la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
web-infra-dev/rspack#16345 ·
Los mantenedores suelen responder en 1 día