This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Toplevel patch: Autogenerate Makefile targets 2 (*-target-*)


>> -$(CHECK_TARGET_MODULES):
>> -     @dir=`echo $@ | sed -e 's/check-target-//'`; \
>> -     if [ -f $(TARGET_SUBDIR)/$${dir}/Makefile ] ; then \
>> +[+ IF no_check +][+ ELSE check +]
>
>Is this the best place for this IF?

>> -$(INSTALL_TARGET_MODULES): installdirs
>> -     @dir=`echo $@ | sed -e 's/install-target-//'`; \
>> -     if [ -f $(TARGET_SUBDIR)/$${dir}/Makefile ] ; then \
>> +[+ ENDIF no_check +]
>
>This endif looks out of place.

These look out of place because the diffing program chose an odd
way to write things.  They are in the correct places (before and after 
the check-[+module+] target).

--Nathanael


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]