[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #17744 [Tor]: Add quotes when comparing strings in configure script
#17744: Add quotes when comparing strings in configure script
-------------------------+------------------------------------
Reporter: cypherpunks | Owner: zerosion
Type: defect | Status: needs_revision
Priority: Medium | Milestone: Tor: 0.2.8.x-final
Component: Tor | Version: Tor: 0.2.7.5
Severity: Trivial | Resolution:
Keywords: easy | Actual Points:
Parent ID: | Points:
Sponsor: |
-------------------------+------------------------------------
Comment (by cypherpunks):
Replying to [comment:14 teor]:
> I can't see the point of quoting or changing the quotes around internal
variables or internal string literals. These are the majority of the
changes in the patch, and I'm not sure if they gain us anything. They need
to be weighed against the risk that a misplaced quote breaks the build on
some obscure configuration.
> Was there a reason for these changes?
As stated in the ticket description, ticket:16651#comment:28 mentions a
case where internal variables included spaces and caused issues. This was
the reason for opening this ticket in order to prevent similar instances
from causing problems in the future.
>
> The issue with re-flowing the entire file is that it makes it hard to
tell when the last non-whitespace change was made to each line.
I believe git has options to ignore whitespace changes so they do not
interfere when tracking down changes. Furthermore, the tor codebase has
numerous commits which purely fix whitespace problems to appease `make
check-spaces`. In these cases change traceability isn't taken into account
either so IMO we shouldn't start with it now.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/17744#comment:16>
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