Closed
Bug 241998
Opened 21 years ago
Closed 21 years ago
configure.in sets -nologo multiple times for CFLAGS and CXXFLAGS
Categories
(SeaMonkey :: Build Config, defect)
SeaMonkey
Build Config
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla1.8alpha1
People
(Reporter: u116418, Unassigned)
Details
Attachments
(2 files)
|
1.29 KB,
patch
|
cls
:
review+
|
Details | Diff | Splinter Review |
|
2.55 KB,
patch
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7) Gecko/20040428 Firehawk/0.8.0+ (Lohvarn)
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7) Gecko/20040428 Firehawk/0.8.0+ (Lohvarn)
mozilla/configure.in sets CFLAGS="$CFLAGS -TC -nologo then CFLAGS="$CFLAGS -W3
-nologo -Gy -Fd\$(PDBFILE)"
Same thing for CXXFLAGS
Reproducible: Always
Steps to Reproduce:
Attachment #147256 -
Flags: review?
|
||
Updated•21 years ago
|
Attachment #147256 -
Flags: review? → review?(bsmedberg)
Attachment #147256 -
Attachment is obsolete: true
Attachment #147439 -
Flags: review?(bsmedberg)
Attachment #147256 -
Flags: review?(bsmedberg)
|
|
||
Comment 3•21 years ago
|
||
fwiw, there's no need to change configure, it will get auto-regenerated when a
patch to configure.in gets checked in.
Attachment #147256 -
Attachment is obsolete: false
Attachment #147256 -
Flags: review?(bsmedberg-mozilla)
Attachment #147439 -
Attachment description: Modify configure → Patch for builders (modifies configure)
Attachment #147439 -
Flags: review?(bsmedberg)
Thanks for the info. I thought only configure.in needed to be changed until I
built and it was still there. I'm leaving the second patch for builders. Sorry
for the spam.
Attachment #147256 -
Flags: review?(bsmedberg-mozilla) → review+
The patch has been checked in.
Status: UNCONFIRMED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.8alpha
|
||
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•