[tor-bugs] #25862 [Applications/Tor Browser]: Clean up wrapper script/CFLAGS and friends mix on Windows
Tor Bug Tracker & Wiki
blackhole at torproject.org
Fri May 25 07:06:26 UTC 2018
#25862: Clean up wrapper script/CFLAGS and friends mix on Windows
-------------------------------------------------+-------------------------
Reporter: gk | Owner: tbb-
| team
Type: enhancement | Status:
| needs_review
Priority: Medium | Milestone:
Component: Applications/Tor Browser | Version:
Severity: Normal | Resolution:
Keywords: tbb-rbm, TorBrowserTeam201805R, | Actual Points:
boklm201805 |
Parent ID: | Points:
Reviewer: | Sponsor:
-------------------------------------------------+-------------------------
Description changed by gk:
Old description:
> Our mix of wrapper scripts and usage of CFLAGS/LDFLAGS when compiling on
> Windows has grown over the years to a point where it gets confusing. We
> should get back to sanity here.
>
> We needed the wrapper scripts to pass down all required flags to the
> depths of the Firefox build system, but meanwhile those scripts are used
> almost everywhere and `CFLAGS`/`LDFLAGS` additionally sometimes.
>
> One option boklm brought up would be to use the `CFLAGS`/`LDFLAGS`
> defined in rbm.conf and enable those script only during Firefox's build.
>
> Another just get rid of those wrapper scripts and fix the missing pieces
> in Firefox if those are still there.
>
> At any rate, we should document where we need them in the first place to
> avoid lengthy digging in case someone is wondering in the future or now,
> while looking at our build scripts.
New description:
Our mix of wrapper scripts and usage of CFLAGS/LDFLAGS when compiling on
Windows has grown over the years to a point where it gets confusing. We
should get back to sanity here.
We needed the wrapper scripts to pass down all required flags to the
depths of the Firefox build system, but meanwhile those scripts are used
almost everywhere and `CFLAGS`/`LDFLAGS` additionally sometimes.
One option boklm brought up would be to use the `CFLAGS`/`LDFLAGS` defined
in rbm.conf and enable those scripts only during Firefox's build.
Another just get rid of those wrapper scripts and fix the missing pieces
in Firefox if those are still there.
At any rate, we should document where we need them in the first place to
avoid lengthy digging in case someone is wondering in the future or now,
while looking at our build scripts.
--
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/25862#comment:7>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
More information about the tor-bugs
mailing list