1236330 - Disable crtdll.obj patch check for VS2015

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Biru [:poiru]]

Attachment: Disable crtdll.obj patch check for VS2015

The crtdll.obj patch check fails when using VS2015 because its CRT .libs no longer contains crtdll.obj:

  LINK : fatal error LNK1146: no argument specified with option '/EXTRACT:'
  rm: cannot lstat `crtdll.obj': No such file or directory 

Firefox seems to run fine regardless so it seems like the CRT patch is no longer needed with the VS2015 runtime and the universal CRT.

For now, lets just skip the check on VS2015. After we switch to VS2015, we can rip out the CRT patch nastiness entirely.

Comment 1

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

Configure still succeeds without this patch, right?

Comment 2

[Biru [:poiru]]

(In reply to Kyle Huey [:khuey] ([e-mail]) from comment #1)
> Configure still succeeds without this patch, right?

Yep (tested on Try as well).

Comment 3

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

So then why take it?

Comment 4

[Biru [:poiru]]

(In reply to Kyle Huey [:khuey] ([e-mail]) from comment #3)
> So then why take it?

Mostly to get rid of the two non-fatal messages printed out by configure. See comment 0 or the 'Failure summary' tab here: https://treeherder.mozilla.org/#/jobs?repo=try&revision=b9eaaa34d9c3&selectedJob=15032019

If the build fails for other reasons, those two messages will show up before the other errors. I suspect that will confuse people.

Comment 5

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

Comment on attachment 8703409 [details] [diff] [review]
Disable crtdll.obj patch check for VS2015

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

ok

Comment 6

[Biru [:poiru]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/b333f6ccccb170d3ff677eef2a702195d0fba554
Bug 1236330 - Get rid of configure warning spew on VS2015 from unnecessary crtdll.obj check. r=khuey

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/b333f6ccccb1