December 2020 - tor-commits - lists.torproject.org

[tor/release-0.4.5] Merge branch 'tor-gitlab/mr/234' into maint-0.4.5
by ahf＠torproject.org 16 Dec '20

16 Dec '20

commit 5a2d01ae57a2b90a03895fe33f07629089eee21a Merge: b83777f7b2 ee6ad0e592 Author: Alexander Færøy <ahf(a)torproject.org> Date: Wed Dec 16 20:38:23 2020 +0000 Merge branch 'tor-gitlab/mr/234' into maint-0.4.5 changes/ticket40106 | 5 +++++ src/app/config/config.c | 3 ++- src/feature/client/transports.c | 22 ++++++++++++++++++++++ src/feature/client/transports.h | 1 + 4 files changed, 30 insertions(+), 1 deletion(-)

1 0

[tor/release-0.4.5] Merge branch 'maint-0.4.5' into release-0.4.5
by ahf＠torproject.org 16 Dec '20

16 Dec '20

commit 649ca08dd9f36b6d71034203a1fe8b57629a1665 Merge: d041f083c8 5a2d01ae57 Author: Alexander Færøy <ahf(a)torproject.org> Date: Wed Dec 16 20:39:01 2020 +0000 Merge branch 'maint-0.4.5' into release-0.4.5 changes/ticket40106 | 5 +++++ src/app/config/config.c | 3 ++- src/feature/client/transports.c | 22 ++++++++++++++++++++++ src/feature/client/transports.h | 1 + 4 files changed, 30 insertions(+), 1 deletion(-)

1 0

[tor/maint-0.4.5] Merge remote-tracking branch 'tor-gitlab/mr/238' into maint-0.4.5
by ahf＠torproject.org 16 Dec '20

16 Dec '20

commit b83777f7b2c0809d9f92ae36bd5ea7975a0a2d06 Merge: 7998ea2f18 15b0d30a55 Author: Alexander Færøy <ahf(a)torproject.org> Date: Wed Dec 16 20:31:10 2020 +0000 Merge remote-tracking branch 'tor-gitlab/mr/238' into maint-0.4.5 configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

1 0

[tor/master] Merge branch 'maint-0.4.5'
by ahf＠torproject.org 16 Dec '20

16 Dec '20

commit 66b1a9a5c101d071c88a4f6786dbed9ad4e94982 Merge: 2d68fdc94c b83777f7b2 Author: Alexander Færøy <ahf(a)torproject.org> Date: Wed Dec 16 20:31:46 2020 +0000 Merge branch 'maint-0.4.5' configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

1 0

[tor/master] Merge remote-tracking branch 'tor-gitlab/mr/238' into maint-0.4.5
by ahf＠torproject.org 16 Dec '20

16 Dec '20

commit b83777f7b2c0809d9f92ae36bd5ea7975a0a2d06 Merge: 7998ea2f18 15b0d30a55 Author: Alexander Færøy <ahf(a)torproject.org> Date: Wed Dec 16 20:31:10 2020 +0000 Merge remote-tracking branch 'tor-gitlab/mr/238' into maint-0.4.5 configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

1 0

[tor/master] configure: Revert back the static libevent path
by ahf＠torproject.org 16 Dec '20

16 Dec '20

commit 15b0d30a553e2e3e7eba2539d6531025a0272d2c Author: David Goulet <dgoulet(a)torproject.org> Date: Fri Dec 11 09:52:23 2020 -0500 configure: Revert back the static libevent path From c618c4f27986825df2026fb22b8ee222edd1afa1, we changed the static libevent.a path to be able to use the git repository or tarball directly but that broke the "make install" setup that Tor Browser is using with Android. In other words, the git repository and tarball put the "libevent.a" in ".libs/" where "make install" puts it in "lib/". Using the --with-libevent-dir=..., which is mandatory for static libevent, autoconf will take the path and use it for the includes (-I) and library (-L) for which if it finds a "include/" and a "lib/" in the root, it will use those. However, with the git repo or tarball, the "lib/" doesn't exists thus autoconf sets the library search path to be at the root and thus fails to find the libevent.a in ".libs/". This is a whole lot more work to make both cases work in our configure.ac thus I'm reverting the change here to the Tor Browser case works again and the work around for others is to either symlink the libevent.a at the root or use a temporary make install directory. One long term fix here would likely be to ask libevent to symblink the .a into the root along the .la files and likely do the same for .so. Or, use the "lib/" structure to contain the .a + .so files. Would be better than doing ninji-tsu in our configure.ac Fixes #40225 Signed-off-by: David Goulet <dgoulet(a)torproject.org> --- configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index 83f6ad1821..1c3fa7d15c 100644 --- a/configure.ac +++ b/configure.ac @@ -954,7 +954,7 @@ if test "$enable_static_libevent" = "yes"; then if test "$tor_cv_library_libevent_dir" = "(system)"; then AC_MSG_ERROR("You must specify an explicit --with-libevent-dir=x option when using --enable-static-libevent") else - TOR_LIBEVENT_LIBS="$TOR_LIBDIR_libevent/.libs/libevent.a $STATIC_LIBEVENT_FLAGS" + TOR_LIBEVENT_LIBS="$TOR_LIBDIR_libevent/libevent.a $STATIC_LIBEVENT_FLAGS" fi else if test "x$ac_cv_header_event2_event_h" = "xyes"; then

1 0

[tor/maint-0.4.5] configure: Revert back the static libevent path
by ahf＠torproject.org 16 Dec '20

16 Dec '20

commit 15b0d30a553e2e3e7eba2539d6531025a0272d2c Author: David Goulet <dgoulet(a)torproject.org> Date: Fri Dec 11 09:52:23 2020 -0500 configure: Revert back the static libevent path From c618c4f27986825df2026fb22b8ee222edd1afa1, we changed the static libevent.a path to be able to use the git repository or tarball directly but that broke the "make install" setup that Tor Browser is using with Android. In other words, the git repository and tarball put the "libevent.a" in ".libs/" where "make install" puts it in "lib/". Using the --with-libevent-dir=..., which is mandatory for static libevent, autoconf will take the path and use it for the includes (-I) and library (-L) for which if it finds a "include/" and a "lib/" in the root, it will use those. However, with the git repo or tarball, the "lib/" doesn't exists thus autoconf sets the library search path to be at the root and thus fails to find the libevent.a in ".libs/". This is a whole lot more work to make both cases work in our configure.ac thus I'm reverting the change here to the Tor Browser case works again and the work around for others is to either symlink the libevent.a at the root or use a temporary make install directory. One long term fix here would likely be to ask libevent to symblink the .a into the root along the .la files and likely do the same for .so. Or, use the "lib/" structure to contain the .a + .so files. Would be better than doing ninji-tsu in our configure.ac Fixes #40225 Signed-off-by: David Goulet <dgoulet(a)torproject.org> --- configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index 83f6ad1821..1c3fa7d15c 100644 --- a/configure.ac +++ b/configure.ac @@ -954,7 +954,7 @@ if test "$enable_static_libevent" = "yes"; then if test "$tor_cv_library_libevent_dir" = "(system)"; then AC_MSG_ERROR("You must specify an explicit --with-libevent-dir=x option when using --enable-static-libevent") else - TOR_LIBEVENT_LIBS="$TOR_LIBDIR_libevent/.libs/libevent.a $STATIC_LIBEVENT_FLAGS" + TOR_LIBEVENT_LIBS="$TOR_LIBDIR_libevent/libevent.a $STATIC_LIBEVENT_FLAGS" fi else if test "x$ac_cv_header_event2_event_h" = "xyes"; then

1 0

[tor/release-0.4.5] Merge remote-tracking branch 'tor-gitlab/mr/238' into maint-0.4.5
by ahf＠torproject.org 16 Dec '20

16 Dec '20

commit b83777f7b2c0809d9f92ae36bd5ea7975a0a2d06 Merge: 7998ea2f18 15b0d30a55 Author: Alexander Færøy <ahf(a)torproject.org> Date: Wed Dec 16 20:31:10 2020 +0000 Merge remote-tracking branch 'tor-gitlab/mr/238' into maint-0.4.5 configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

1 0

[tor/release-0.4.5] configure: Revert back the static libevent path
by ahf＠torproject.org 16 Dec '20

16 Dec '20

commit 15b0d30a553e2e3e7eba2539d6531025a0272d2c Author: David Goulet <dgoulet(a)torproject.org> Date: Fri Dec 11 09:52:23 2020 -0500 configure: Revert back the static libevent path From c618c4f27986825df2026fb22b8ee222edd1afa1, we changed the static libevent.a path to be able to use the git repository or tarball directly but that broke the "make install" setup that Tor Browser is using with Android. In other words, the git repository and tarball put the "libevent.a" in ".libs/" where "make install" puts it in "lib/". Using the --with-libevent-dir=..., which is mandatory for static libevent, autoconf will take the path and use it for the includes (-I) and library (-L) for which if it finds a "include/" and a "lib/" in the root, it will use those. However, with the git repo or tarball, the "lib/" doesn't exists thus autoconf sets the library search path to be at the root and thus fails to find the libevent.a in ".libs/". This is a whole lot more work to make both cases work in our configure.ac thus I'm reverting the change here to the Tor Browser case works again and the work around for others is to either symlink the libevent.a at the root or use a temporary make install directory. One long term fix here would likely be to ask libevent to symblink the .a into the root along the .la files and likely do the same for .so. Or, use the "lib/" structure to contain the .a + .so files. Would be better than doing ninji-tsu in our configure.ac Fixes #40225 Signed-off-by: David Goulet <dgoulet(a)torproject.org> --- configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index 83f6ad1821..1c3fa7d15c 100644 --- a/configure.ac +++ b/configure.ac @@ -954,7 +954,7 @@ if test "$enable_static_libevent" = "yes"; then if test "$tor_cv_library_libevent_dir" = "(system)"; then AC_MSG_ERROR("You must specify an explicit --with-libevent-dir=x option when using --enable-static-libevent") else - TOR_LIBEVENT_LIBS="$TOR_LIBDIR_libevent/.libs/libevent.a $STATIC_LIBEVENT_FLAGS" + TOR_LIBEVENT_LIBS="$TOR_LIBDIR_libevent/libevent.a $STATIC_LIBEVENT_FLAGS" fi else if test "x$ac_cv_header_event2_event_h" = "xyes"; then

1 0

[tor/release-0.4.5] Merge branch 'maint-0.4.5' into release-0.4.5
by ahf＠torproject.org 16 Dec '20

16 Dec '20

commit d041f083c8c7e95bbcac408d6b30a516a0c0af76 Merge: 6ffa0f3e6b b83777f7b2 Author: Alexander Færøy <ahf(a)torproject.org> Date: Wed Dec 16 20:31:46 2020 +0000 Merge branch 'maint-0.4.5' into release-0.4.5 configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

1 0