Opened 2 years ago

Last modified 12 days ago

#17015 new enhancement

chutney should try harder to find a stable/different version of tor

Reported by: teor Owned by:
Priority: Medium Milestone:
Component: Core Tor/Chutney Version:
Severity: Normal Keywords: SponsorS testing
Cc: Actual Points:
Parent ID: #16949 Points:
Reviewer: Sponsor:

Description

Currently, chutney tries to find a stable version of tor under tor-stable anywhere in the path, then stops.

We could:

  • Preserve the old path before test-network.sh modifies it, and use the first tor in that path
  • Check for /usr/bin/tor and /usr/local/bin/tor (And maybe /opt ?)
  • use each version in a round-robin fashion (Split to #17014, child of #16954)

Child Tickets

Change History (7)

comment:1 Changed 2 years ago by teor

Type: defectenhancement

comment:2 Changed 2 years ago by kernelcorn

The second option seems like the simplest and generally reliable approach. Chutney should use the system installation of Tor (/usr/bin/tor) or one that someone installed themselves (/usr/local/bin/tor).

However, the first option seems more flexible and is the behavior probably expected by most Chutney users. It allows someone to install their own Tor build in a custom location and then change their path accordingly, which is perfectly common approach for other Linux applications.

comment:3 Changed 2 years ago by teor

I think a combination of the two options would work best - try a few well-known paths after the user-supplied path. That said, if a user doesn't have /usr/local/bin in their path, should we really be looking in it?

I'll work on this after we have tor 027 out.

comment:4 Changed 2 years ago by nickm

I vote for principle-of-least-surprise: option 1 seems best to me. Looking outside of the user's path seems likely, sooner or later, to surprise the user in a way they won't like.

comment:5 Changed 18 months ago by nickm

Owner: nickm deleted
Status: newassigned

Remove myself as chutney ticket owner. Default owners are trouble.

comment:6 Changed 18 months ago by nickm

Status: assignednew

comment:7 Changed 12 days ago by teor

Severity: Normal

Set all open tickets without a severity to "Normal"

Note: See TracTickets for help on using tickets.