Opened 7 months ago

Closed 7 months ago

#29143 closed defect (fixed)

Building obfs4 in tor-browser-build nightlies is broken due to uTLS support

Reported by: gk Owned by: tbb-team
Priority: Medium Milestone:
Component: Applications/Tor Browser Version:
Severity: Normal Keywords: tbb-rbm, TorBrowserTeam201901R
Cc: boklm, yawning Actual Points:
Parent ID: Points:
Reviewer: Sponsor:


Starting build: Mon Jan 21 13:50:13 2019
/var/tmp/dist/gopath/src/ cannot find package "" in any of:
	/var/tmp/dist/go/src/ (from $GOROOT)
	/var/tmp/dist/gopath/src/ (from $GOPATH)
/var/tmp/dist/gopath/src/ cannot find package "" in any of:
	/var/tmp/dist/go/src/ (from $GOROOT)
	/var/tmp/dist/gopath/src/ (from $GOPATH)
/var/tmp/dist/gopath/src/ cannot find package "" in any of:
	/var/tmp/dist/go/src/ (from $GOROOT)
	/var/tmp/dist/gopath/src/ (from $GOPATH)
/var/tmp/dist/gopath/src/ cannot find package "" in any of:
	/var/tmp/dist/go/src/ (from $GOROOT)
	/var/tmp/dist/gopath/src/ (from $GOPATH)

Child Tickets

Change History (9)

comment:1 Changed 7 months ago by boklm

(the nightly build from today did not fail because it was started before the obfs4 commit was pushed)

comment:2 Changed 7 months ago by yawning

Well, yeah. There's more dependencies, and some of the existing ones may need to be bumped. Eventually all the meek components are going to start requiring some of these as well. v0.0.0-20190117054722-15f83653abbc v0.0.0-20170116200512-5312a6153412 v1.2.1 v0.0.0-20190105024908-a89e7e6da482 v0.0.0-20190103213133-ff983b9c42bc v0.0.0-20190119204137-ed066c81e75e v0.0.0-20190116161447-11f53e031339 // indirect v0.3.0 // indirect

Note: Depending on how much goes into utls, the version required to build the eventual tagged release may differ.

comment:5 in reply to:  3 Changed 7 months ago by yawning

Replying to dcf:

Edit: I don't know what's going on with the dependency. For whatever reason, I didn't need to package that one.

arnhem :: Documents/Development/obfs4 ‹master*› % go mod why -m 

comment:6 Changed 7 months ago by boklm

In branch bug_29143_v3 I use var/go_lib_deps instead of some tar commands in var/build_go_lib_pre (after being reminded about it by dcf's patch):

This patch is fixing the build for me, however I did not check yet that it is running correctly.

I also see dcf's patch is adding and to go_lib_install in projects/goxcrypto/config.

comment:7 Changed 7 months ago by boklm

Keywords: TorBrowserTeam201901R added; TorBrowserTeam201901 removed
Status: newneeds_review

There is a new revision of the patch, for review, in branch bug_29143_v5:

I checked that it is running correctly on Linux, in the alpha and nightly builds.

comment:9 Changed 7 months ago by gk

Resolution: fixed
Status: needs_reviewclosed

Looks good now. Thanks for the work while being busy with all the other important stuff, boklm! Merged to master (commit f27f11201d86b8bb909bad8b4c04f841b5487ddb).

Note: See TracTickets for help on using tickets.