Don't overwrite the AS variable in toolchain.m4; r=glandium

RESOLVED FIXED in mozilla38

Status

()

Core
Build Config
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: Ehsan, Assigned: Ehsan)

Tracking

(Blocks: 1 bug)

unspecified
mozilla38
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
It looks like overwriting AS here is not intentional.  Before this patch,
it is impossible to override AS through mozconfig for anything that runs
past this stage in configure.
(Assignee)

Updated

3 years ago
Assignee: nobody → ehsan
Blocks: 369591
(Assignee)

Comment 1

3 years ago
Created attachment 8550745 [details] [diff] [review]
Don't overwrite the AS variable in toolchain.m4
Attachment #8550745 - Flags: review?(mh+mozilla)
Comment on attachment 8550745 [details] [diff] [review]
Don't overwrite the AS variable in toolchain.m4

Review of attachment 8550745 [details] [diff] [review]:
-----------------------------------------------------------------

You should put all those cross-msvc patches in one bug, or even one patch, because reviewing many one liners is not very efficient.
Attachment #8550745 - Flags: review?(mh+mozilla) → review+
(BTW, how did you install msvc on linux? the installer won't start in wine here)
(Assignee)

Comment 4

3 years ago
(In reply to Mike Hommey [:glandium] from comment #2)
> Comment on attachment 8550745 [details] [diff] [review]
> Don't overwrite the AS variable in toolchain.m4
> 
> Review of attachment 8550745 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> You should put all those cross-msvc patches in one bug, or even one patch,
> because reviewing many one liners is not very efficient.

These are just the easy ones.  :-)  There are more extensive changes that I have not put up yet because I'm not happy with them yet.

(In reply to Mike Hommey [:glandium] from comment #3)
> (BTW, how did you install msvc on linux? the installer won't start in wine
> here)

The installer hits https://bugs.winehq.org/show_bug.cgi?id=33576.  What I did was install things on Windows using the Chromium installation script (which is amazing!) and then copy the files over to Linux with a few tweaks.  I will blog about the instructions soon, but first I need to try to install MSVC 2013 Update 4 (what the Chromium scripts install is a previous update, it seems.)
https://hg.mozilla.org/mozilla-central/rev/6f8c5e3ddadb
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
You need to log in before you can comment on or make changes to this bug.