[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #25862 [Applications/Tor Browser]: Clean up wrapper script/CFLAGS and friends mix on Windows
#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
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs