Opened 4 months ago

Last modified 8 days ago

#28764 needs_revision defect

OpenSSL Build for Android

Reported by: sisbell Owned by: tbb-team
Priority: Medium Milestone:
Component: Applications/Tor Browser Version:
Severity: Normal Keywords: tbb-mobile, tbb-rbm, TorBrowserTeam201812, tbb-parity
Cc: sisbell, gk Actual Points:
Parent ID: #28704 Points:
Reviewer: Sponsor:

Description


Child Tickets

Change History (4)

comment:1 Changed 4 months ago by sisbell

Status: newneeds_review

Initial commit for review (android-1207)

comment:2 Changed 3 months ago by gk

Keywords: TorBrowserTeam201812R added; TorBrowserTeam201812 removed

comment:3 Changed 3 months ago by gk

Keywords: TorBrowserTeam201812 added; TorBrowserTeam201812R removed
Status: needs_reviewneeds_revision

Three things we should think about here:

1) The OpenSSL output is not build reproducibly. The .a creation is not deterministic as it seems to contain timestamps, see:

0007ce70: 0000 0000 0000 0000 0000 6001 0000 0b00  ..........`.....
 0007ce80: 0000 0000 0000 0000 0000 0100 0000 0000  ................
 0007ce90: 0000 7373 6c5f 7574 7374 2e6f 2f20 2020  ..ssl_utst.o/   
-0007cea0: 2020 3135 3434 3532 3132 3032 2020 3130    1544521202  10
+0007cea0: 2020 3135 3434 3532 3035 3032 2020 3130    1544520502  10
 0007ceb0: 3030 2020 3130 3030 2020 3130 3036 3434  00  1000  100644
 0007cec0: 2020 3736 3420 2020 2020 2020 600a 7f45    764       `..E
 0007ced0: 4c46 0101 0100 0000 0000 0000 0000 0100  LF..............

I wonder why that is an issue here. Maybe the cross-toolchain for Android does not have something like -Wl,--no-insert-timestamp?

2) Do we really need all the exported env variables in the build script? The build seemed to work for me e.g. without declaring RANLIB?

3) The comment in the config file should we wrapped so that we have it on two lines instead of one.

Last edited 3 months ago by gk (previous) (diff)

comment:4 Changed 8 days ago by gk

Keywords: tbb-parity added

tbb-parity items.

Note: See TracTickets for help on using tickets.