450717 - dep file for pixman-mmx.c isn't generated with GCC

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Ginn Chen]

Compile libpixman with GCC compiler depend feature

In obj/gfx/cairo/libpixman/src/.deps, there is no
pixman-mmx.pp
on both Linux and Mac OS X.

It may cause some issues with tinderbox dep builds.

In Makefile.in there's
# special rule for pixman-mmx to get the right cflags
pixman-mmx.$(OBJ_SUFFIX): pixman-mmx.c Makefile Makefile.in
        $(REPORT_BUILD)
        @$(MAKE_DEPS_AUTO_CC)
        $(ELOG) $(CC) $(OUTOPTION)$@ -c $(COMPILE_CFLAGS) $(MMX_CFLAGS) $(_VPATH
_SRCS)

Because the object name is specified, so $(*F) is empty,
but we have
OS_COMPILE_CFLAGS = $(OS_CPPFLAGS) -include $(DEPTH)/mozilla-config.h -DMOZILLA_CLIENT $(filter-out %/.pp,-Wp,-MD,$(MDDEPDIR)/$(*F).pp)

I tried
%.$(OBJ_SUFFIX): pixman-mmx.c Makefile Makefile.in
It doesn't work, the default rule wins.

Then I tried to use $(basename $(@F)) instead of  $(*F). It works.

Comment 1

[Ginn Chen]

Attachment: patch

Comment 2

[(not currently active) Ted Mielczarek]

Comment on attachment 333902 [details] [diff] [review]
patch

I really hate make's built-in variable names. It took me about 15 minutes of reading the make manual to understand what you were doing here. :)

Comment 3

[Ginn Chen]

Pushed
http://hg.mozilla.org/mozilla-central/rev/cb405eb32c76

Comment 4

[Ginn Chen]

backed out

config/Makefile.in doesn't create .deps dir, so gcc failed to create .deps/xxx.pp file for elf-dynstr-gc.

According to rules.mk, we generate dep files for
$(OBJS)$(XPIDLSRCS)$(SDK_XPIDLSRCS)$(SIMPLE_PROGRAMS)
but for gcc, the -MD option is in compiler flag, it works for every file.

config/Makefile.in doesn't have any $(OBJS)$(XPIDLSRCS)$(SDK_XPIDLSRCS)$(SIMPLE_PROGRAMS), so .dep directory is not created.

Since we only have this issue with this special directory, I suggest we workaround it.

Comment 5

[Ginn Chen]

Attachment: patch v2

Comment 6

[Ginn Chen]

Pushed:
http://hg.mozilla.org/mozilla-central/rev/35e183680fda