[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #24363 [Core Tor/Tor]: Remove /usr/athena from search path in configure.ac
#24363: Remove /usr/athena from search path in configure.ac
--------------------------+------------------------------------
Reporter: nickm | Owner: (none)
Type: defect | Status: needs_review
Priority: Medium | Milestone: Tor: 0.3.3.x-final
Component: Core Tor/Tor | Version:
Severity: Normal | Resolution:
Keywords: | Actual Points:
Parent ID: | Points:
Reviewer: | Sponsor:
--------------------------+------------------------------------
Comment (by ffmancera):
Oh thank you! I didn't know about the script. I corrected it and re-
uploaded the patch.
Replying to [comment:2 pastly]:
> Thanks for the patch!
>
> I see you included a change file. That's awesome. Nick loves that.
>
> Are you aware of the ./scripts/maint/lintChanges.py script? Your changes
file doesn't pass the script because "Minor change" isn't one of the known
headings. Peeking at the source of the script, if it was me making this
change and wanting to please the Tor ~~gods~~ maintainers, I'd probably
use "Code simplification and refactoring"
>
> It still takes me about 3 attempts to please the script :). You call it
like `./scripts/maint/lintChanges.py changes/ticket24363` and silence
indicates success.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/24363#comment:3>
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