Closed
Bug 879916
Opened 11 years ago
Closed 11 years ago
gfx/tests/Makefile.in - commented CMMSRCS=
Categories
(Firefox Build System :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla24
People
(Reporter: joey, Assigned: BenWa)
Details
Attachments
(1 file)
1.85 KB,
patch
|
joe
:
review+
|
Details | Diff | Splinter Review |
gfx/tests/Makefile.in Commented entries in the makefile will need to be cleaned up/purged or re-enabled. changeset: 70003:8e846d7f21ea date: Mon May 23 18:54:47 2011 +0200 summary: Bug 648911 - Remove support for non-libxul builds. r=ted,joedrew #ifeq ($(MOZ_WIDGET_TOOLKIT),cocoa) #DISABLED_CMMSRCS = gfxTestCocoaHelper.mm #HELPER_OBJS = gfxTestCocoaHelper.$(OBJ_SUFFIX) #EXTRA_DEPS += gfxTestCocoaHelper.$(OBJ_SUFFIX) #endif # CPP_UNIT_TESTS disabled for now because they dont work in libxul builds. #ifndef BUILD_STATIC_LIBS # #CPP_UNIT_TESTS = \ # TestColorNames.cpp \ #LIBS = \ # $(HELPER_OBJS) \ # $(call EXPAND_LIBNAME_PATH,thebes,../thebes) \ # $(call EXPAND_LIBNAME_PATH,gkgfx,../src) \ # $(MOZ_UNICHARUTIL_LIBS) \
Comment 1•11 years ago
|
||
This is just another case of standalone C++ tests that we disabled when we removed --disable-libxul. These could all probably become gtests.
Assignee | ||
Comment 2•11 years ago
|
||
I have a patch to fix some CPPSRCS tests but not all. For CMMSRCS I only see 'gfxTestCocoaHelper.mm' which is a simple and trivial memory pool test. I don't see any value in it and perhaps we should just delete it rather then port it. Are there any other files I'm not seeing or does this bug just cover this file.
Assignee | ||
Comment 3•11 years ago
|
||
Trivial test I'm not interested in porting.
Updated•11 years ago
|
Attachment #758770 -
Flags: review?(joe) → review+
Assignee | ||
Comment 4•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/ee5b6ada4a03
Comment 5•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/ee5b6ada4a03
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla24
Updated•6 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•