[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #4727 [Tor Client]: Porting to Haiku (Patch increasing portability)
#4727: Porting to Haiku (Patch increasing portability)
---------------------------------------------+------------------------------
Reporter: martinhpedersen | Owner:
Type: enhancement | Status: needs_review
Priority: normal | Milestone:
Component: Tor Client | Version:
Keywords: haiku porting gcc-3.3 lm libm m | Parent:
Points: | Actualpoints:
---------------------------------------------+------------------------------
Changes (by Sebastian):
* status: new => needs_review
Comment:
Thanks for the patches! Please don't be disconcerted at the below
feedback, I really like your patch. If you won't have time to do what I
suggest, I'll gladly do it!
First off, please change the status of a trac ticket to "needs review"
when you have some code. That helps us find things we should look at
quickly :)
For the first patch, can you please update it to preserve order of libs?
We've had tons of trouble before where on some platforms the order was
significant, which led to hard to track down bugs, particularly on
windows. I'm unsure if that is the case here, but it might well be, so
better safe than sorry :)
As a general note, we try to keep the first line of a commit message
pretty short. If you can think of a better msg in 10 seconds, please
update it. Otherwise, just keep it this way, it's not that big a deal.
Also, if you want, you could add the changes/ files that we like to do
(see doc/HACKING for how to do it, or ask on irc).
Thanks again!
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/4727#comment:1>
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