Skip to content

All candidate commpaths added to localprefix #233

Open

Description

@rountree

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).

Activity

  1. rountree commented on Oct 23, 2026

    @rountree
    CollaboratorAuthor

    Fixed by #232

  2. added theissue type on Oct 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions