[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #21846 [Core Tor/Torflow]: BwAuthority can't be run out of the box without manual work
#21846: BwAuthority can't be run out of the box without manual work
-------------------------------+--------------------------------
Reporter: davidwf | Owner: aagbsn
Type: enhancement | Status: needs_revision
Priority: Medium | Milestone:
Component: Core Tor/Torflow | Version:
Severity: Normal | Resolution:
Keywords: | Actual Points:
Parent ID: | Points:
Reviewer: chelseakomlo, tom | Sponsor:
-------------------------------+--------------------------------
Comment (by davidwf):
Please see changes at 'git@xxxxxxxxxx:davidfaulkner12/bwauth.git', branch
'nicedev'.
> Why did you downgrade from SQLAlchemy 0.7.10 to 0.7.8?
> Whichever way you choose to go, please find all instances of
SQLAlchemy-0.7.* and make them consistent.
Updated -- the PYTHONPATH variables in the shell scripts had no impact
(and didn't resolve to anything on my machine at least), so I removed
them. I used 0.7.8 because that's the last documented "compatible" version
with Elixer.
> For future patches, it is ok to provide a branch and use multiple
commits - in fact, we prefer it!
Noted! It was actually always in Github, I just through patches were
preferred. :-)
> readlink -f works with gnu readlink, but not with BSD readlink. It is
possible to install BSD readlink on Linux, and GNU readlink on BSD or
macOS. So we can't depend on uname, we should just try basename if
readlink doesn't work here:
I checked to see if the command fails, and only set the variable if
necessary. This should work!
> Some OSs only have a non-versioned python (e.g. macOS), so we should
fall back to "which python" here:
Done.
> In stop_scan.sh: ...
stop_scan.sh is actually just copy and pasted from the first lines of the
run_scan.sh. I don't disagree with anything you said, but I'm not
confident that I can make those changes in a way that works reliably
without digging too deeply past the initial scope of "get this working on
a laptop".
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/21846#comment:11>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs