{"push_id":2977157944,"size":5,"distinct_size":5,"ref":"refs/heads/master","head":"fb090aa622bcfd0431b6a4b97883bddf29ad8a0e","before":"95918754d2ba7e6bffe0fc74bebed60bd917c10c","commits":[{"sha":"09c74f71217bb6c614b8597041aacf830160f3a0","author":{"name":"Azat Khuzhin","email":"fa577405315cc9b8efd2145a48904e0c6dc2fb8f@gmail.com"},"message":"Fix an error for debug locking in dns/getaddrinfo_race_gotresolve\n\nWhen there is no /etc/services file evdns_getaddrinfo() will fail (with\nservice=\"ssh\") and hence it will go to then \"end\" label with locked\nrp.lock which in case of debug locking checks will bail with:\n  [err] ../evthread.c:220: Assertion lock->count == 0 failed in debug_lock_free\n\nSo add rp.locked flag, and unlock the lock before freeing it if it is in\nlocked state.\n\nAnd here is how you can reproduce the issue:\n  $ docker run -e LD_LIBRARY_PATH=$PWD/lib -e PATH=/usr/bin:/bin:$PWD/bin -v $PWD:$PWD --rm -it debian:testing regress dns/getaddrinfo_race_gotresolve\n(since debian:testing does not have /etc/services)","distinct":true,"url":"https://api.github.com/repos/transmission/libevent/commits/09c74f71217bb6c614b8597041aacf830160f3a0"},{"sha":"7198bbb8f13fe0caab310b2fb6dd618c61893f9a","author":{"name":"Azat Khuzhin","email":"fa577405315cc9b8efd2145a48904e0c6dc2fb8f@gmail.com"},"message":"Turn off dns/getaddrinfo_race_gotresolve by default\n\nIt is:\n- pretty internal regression\n- CPU bound\n- right now failed on travis-ci machines","distinct":true,"url":"https://api.github.com/repos/transmission/libevent/commits/7198bbb8f13fe0caab310b2fb6dd618c61893f9a"},{"sha":"2cb8eae7958fd9adceba32e67a146925880d8571","author":{"name":"Azat Khuzhin","email":"fa577405315cc9b8efd2145a48904e0c6dc2fb8f@gmail.com"},"message":"Do not rely on getservbyname() for most of the dns regression tests\n\nThere is only one test that uses service name getaddrinfo_async, which\nmanually check whether it works or not, other should not assume that it\nis available and works.\n\nThere was already an attempt to overcome some possible limitations, like\nlack of \"http\" in /etc/services in\nd6bafbbeb27ff3943d6f3b6783bcded76384c31e (\"test/dns: replace servname\nsince solaris does not have \"http\"\")","distinct":true,"url":"https://api.github.com/repos/transmission/libevent/commits/2cb8eae7958fd9adceba32e67a146925880d8571"},{"sha":"bd2184d8d42f721fa523d01b9a9250c75fedb855","author":{"name":"Azat Khuzhin","email":"fa577405315cc9b8efd2145a48904e0c6dc2fb8f@gmail.com"},"message":"Merge branch 'regress-dns-fixes'\n\n* regress-dns-fixes:\n  Do not rely on getservbyname() for most of the dns regression tests\n  Turn off dns/getaddrinfo_race_gotresolve by default\n  Fix an error for debug locking in dns/getaddrinfo_race_gotresolve","distinct":true,"url":"https://api.github.com/repos/transmission/libevent/commits/bd2184d8d42f721fa523d01b9a9250c75fedb855"},{"sha":"fb090aa622bcfd0431b6a4b97883bddf29ad8a0e","author":{"name":"Azat Khuzhin","email":"fa577405315cc9b8efd2145a48904e0c6dc2fb8f@gmail.com"},"message":"Revert \"travis-ci: use travis_retry to reduce number of problems with flacky tests\"\n\nThat was a bad idea, because all this patch did is just uses extra CPU\ntime on travis-ci workers and eventually fail the build anyway.\n\nIt also increases time of executing one matrix entry (x3).\n\nThis reverts commit 7004ee8e869faabe3a284fb3d7eb859fb50ef90c.","distinct":true,"url":"https://api.github.com/repos/transmission/libevent/commits/fb090aa622bcfd0431b6a4b97883bddf29ad8a0e"}]}
{"actor":{"display_login":"mikedld"}}