[syslinux] [PATCH] Dont bypass compiler driver for Dependency generation options

[Gene Cumm]

On Thu, Dec 10, 2015 at 6:39 AM, Gene Cumm <gene.cumm at gmail.com> wrote:
> On Mon, Sep 14, 2015 at 1:36 AM, Khem Raj <raj.khem at gmail.com> wrote:
>>
>>> On Sep 13, 2015, at 11:04 AM, Gene Cumm <gene.cumm at gmail.com> wrote:
>>>
>>> On Thu, Sep 10, 2015 at 1:55 AM, Khem Raj via Syslinux
>>> <syslinux at zytor.com> wrote:
>>>> We can let compiler driver pass the right options to preprocessor after
>>>> processing -Mxy options, right now its bypassing the gcc driver and
>>>> handing them straight to cpp
>>>>
>>>> This also helps in other compilers processing these options correctly
>>>> for their preprocessors consumption
>>>>
>>>> Signed-off-by: Khem Raj <raj.khem at gmail.com>
>>>> ---
>>>> mk/syslinux.mk | 4 ++--
>>>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>>>
>>>> diff --git a/mk/syslinux.mk b/mk/syslinux.mk
>>>> index 337560b..9ca74b1 100644
>>>> --- a/mk/syslinux.mk
>>>> +++ b/mk/syslinux.mk
>>>> @@ -82,11 +82,11 @@ ARCH ?= $(strip $(SUBARCH))
>>>> GCCWARN  = -W -Wall -Wstrict-prototypes $(DEBUGOPT)
>>>>
>>>> # Common stanza to make gcc generate .*.d dependency files
>>>> -MAKEDEPS = -Wp,-MT,$@,-MD,$(dir $@).$(notdir $@).d
>>>> +MAKEDEPS = -MT $@ -MD
>>>>
>>>> # Dependencies that exclude system headers; use whenever we use
>>>> # header files from the platform.
>>>> -UMAKEDEPS = -Wp,-MT,$@,-MMD,$(dir $@).$(notdir $@).d
>>>> +UMAKEDEPS = -MT $@ -MMD
>>>>
>>>> # Items that are only appropriate during development; this file is
>>>> # removed when tarballs are generated.
>>>
>>> It looks like these were introduced in MCONFIG back in commit ID
>>> fe938522 and originally in a bunch of other commits like fbfc9121 and
>>> the earliest instance appears to be 17f96764.
>>>
>>> Khem, at first I didn't quite understand the part about other
>>> compilers then it struck me that you're probably trying to build
>>> linux/syslinux and extlinux/extlinux without a gcc dependence.
>>
>> I was using clang to compile it :). than I ran into further issues which I am tackling
>> and if any useful patch results, I will send upstream
>
> Thanks.  This is now merged.
>
> --
> -Gene

Instead of dropping everything for the filename generation (as they're
referenced throughout numerous make files), I added another commit
that preserves the old filenames, 721a0af and here's the effective
diff.  Could you let me know how this works for your environment?


diff --git a/mk/syslinux.mk b/mk/syslinux.mk
index 337560b..46355c9 100644
--- a/mk/syslinux.mk
+++ b/mk/syslinux.mk
@@ -82,11 +82,11 @@ ARCH ?= $(strip $(SUBARCH))
 GCCWARN  = -W -Wall -Wstrict-prototypes $(DEBUGOPT)

 # Common stanza to make gcc generate .*.d dependency files
-MAKEDEPS = -Wp,-MT,$@,-MD,$(dir $@).$(notdir $@).d
+MAKEDEPS = -MT $@ -MD -MF $(dir $@).$(notdir $@).d

 # Dependencies that exclude system headers; use whenever we use
 # header files from the platform.
-UMAKEDEPS = -Wp,-MT,$@,-MMD,$(dir $@).$(notdir $@).d
+UMAKEDEPS = -MT $@ -MMD -MF $(dir $@).$(notdir $@).d

 # Items that are only appropriate during development; this file is
 # removed when tarballs are generated.


-- 
-Gene