Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tor: update to version 0.4.1.6 #5317

Merged
merged 6 commits into from
Oct 6, 2019
Merged

Conversation

svschmel
Copy link
Contributor

Hello,

with PR 11695 i want to update tor to version 0.4.1.5.
On 2019-09-19 the Tor Project released the new version 0.4.1.6.
With this PR we can offer the new one.

New in this version:

  • The tor binary will now be installed in the "/usr/bin" directory.
  • In the old versions it was "/usr/lib"
  • The script "tor.sh" from the manifest was adjusted.

On https://metrics.torproject.org/rs.html#details/AD0FCFD83EA4899DB34735E599701592DEA13690 you can the see new version running.

With "arm" you can check connections made by tor.
See the attachments.

IMHO we can close the PR 11695 and use this one to put tor V. 0.4.1.6 to oi-userland .

Screenshot_20190921_205842
Screenshot_20190921_210012

Copy link
Contributor

@Mno-hime Mno-hime left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please do a bit of a clean-up of the Makefile:

  • move REQUIRED_PACKAGES += text/gnu-diffutils to the rest of REQUIRED_PACKAGES;
  • remove the commented line #CONFIGURE_OPTIONS+= --enable-linker-hardening;
  • drop prep, configure, and ips includes and replace them with common.mk include, drop "common targets" build, install, and test and put BUILD_BITS=64 before the first include on top of the file, CONFIGURE_BINDIR.64 = $(USRBINDIR) likely should be dropped.

When done, rune gmake clean sample-manifest and then vimdiff tor.p5m manifest/sample-manifest.p5m and check that nothing got broken and all files we likely want to deliver are delivered as part of the tor.p5m file.

Did you run the test target? Did it pass?

components/network/tor/tor.p5m Outdated Show resolved Hide resolved
@svschmel
Copy link
Contributor Author

svschmel commented Sep 22, 2019

Hello,
i made the changes in the Makefile and run "gmake clean sample-manifest"

vimdiff => The main difference is the following block:

sample-manifest.p5m:
file path=etc/tor/torrc.sample
file path=usr/bin/tor

tor.p5m:
file files/auth_attr path=etc/security/auth_attr.d/tor
file files/prof_attr path=etc/security/prof_attr.d/tor
file files/tor.xml
path=lib/svc/manifest/network/tor.xml
file files/tor.sh path=/lib/svc/method/tor.sh mode=0555
dir path=var/lib/tor group=daemon mode=0755 owner=daemon
dir path=var/log/tor group=daemon mode=0755 owner=daemon
file etc/tor/torrc.sample path=etc/tor/torrc mode=0644 preserve=true
file usr/bin/tor path=usr/bin/tor mode=0555

It looks ok for me. I don't think, that the sample-manifest-file knows something about the service-manifest files. => files-directory.
And i can't find something about this files-directory in the make-rules.

------
test target:

One test failed:

============================================================================
Testsuite summary for tor 0.4.1.6
============================================================================
# TOTAL: 20
# PASS:  16
# SKIP:  3
# XFAIL: 0
# FAIL:  1
# XPASS: 0
# ERROR: 0
============================================================================
See ./test-suite.log
============================================================================

/oi-userland/components/network/tor/build/amd64$ grep -v OK test-suite.log 

....
FAIL: src/test/test
===================

util/load_win_lib: SKIPPED
util/log_mallinfo: SKIPPED
util/map_anon_nofork: 
  FAIL /export/home/svschmel/oi-userland/components/network/tor/tor-0.4.1.6/src/test/test_util.c:6223: assert(buf[0] OP_EQ 0xd0): -48 vs 208
  [map_anon_nofork FAILED]
1/1360 TESTS FAILED. (2 skipped)
FAIL src/test/test (exit status: 1)
....

At the moment i don't know, if this is critical because the build version is up and running.
But it seems that this test have problems in the 0.4.x series:
https://trac.torproject.org/projects/tor/ticket/29529
https://trac.torproject.org/projects/tor/ticket/29534

@Mno-hime
Copy link
Contributor

It looks ok for me. I don't think, that the sample-manifest-file knows something about the service-manifest files. => files-directory.
And i can't find something about this files-directory in the make-rules.

Both are OK. Files from files directory are addressed directly, see lines in tor.p5m starting file files/.

At the moment i don't know, if this is critical because the build version is up and running.
But it seems that this test have problems in the 0.4.x series:
https://trac.torproject.org/projects/tor/ticket/29529
https://trac.torproject.org/projects/tor/ticket/29534

They eventually enabled it again (https://github.com/torproject/tor/blob/master/src/test/test_util.c#L6397), please file bug upstream and note it above the test target in the Makefile.

(Please, don't create a new PR, push to this one. Thanks!)

@svschmel
Copy link
Contributor Author

For the "map_anon_nofork"- test error i created the following ticket:
https://trac.torproject.org/projects/tor/ticket/31897

@svschmel
Copy link
Contributor Author

Hello,

i have updated the Makefile.
I mentioned the issue request at torproject.org

@Mno-hime Mno-hime merged commit d054262 into OpenIndiana:oi/hipster Oct 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants