All candidate commpaths added to localprefix

Abierto
  • ¿Algo no está bien?
#233 1 comentario 0 reacciones 1 asignado Ver en GitHub

Los mantenedores suelen responder en 1 día

@rountree ya está trabajando en esto.

Desde el 23/10/2026.

Evaluación

Este issue todavía no se ha evaluado.

Descripción

bug

The selected commpath should be added to localprefix. Testing the multiple commpaths case (where some or all of the candidate commpaths could be unusable) flushed out a bug where all of the candidate commpaths were getting added to localprefix, including / (which was intended to be discarded by the selection process). That led to all files being considered local, and several obscure test failures.

On a related note: the cachepath should also be added to localprefix (to cover the pathological case of a user keeping local files in Spindle's cache).

Lenguaje dominante
C
Estrellas
110
Forks
35
Merge medio
1 d 3 h
PR fusionados (30 d)
10

Preparar el entorno

Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.

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 llnl/Spindle

Todos los issues de llnl/Spindle

Issues similares

Más issues de C

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.