This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Make fixinc work on BeOS
- To: Daniel Berlin <dberlin at redhat dot com>
- Subject: Re: [PATCH] Make fixinc work on BeOS
- From: Bruce Korb <bkorb at sco dot COM>
- Date: Tue, 21 Nov 2000 09:07:38 -0800
- CC: GCC Patches <gcc-patches at gcc dot gnu dot org>, Bruce Korb <bkorb at cruzio dot com>
- Organization: The Santa Cruz Operation
- References: <m37l5yy42p.fsf@dan2.cygnus.com> <3A19930E.DEB409B7@sco.com> <m3hf52fc1c.fsf@dan2.cygnus.com>
- Reply-To: Bruce Korb <bkorb at cruzio dot com>
Daniel Berlin wrote:
>
> Here's a new makefile potion of the patch that should work on all platforms (I
> tested it on BeOS, and linux).
>
> We don't need to depend applyfix on anything, because if you built
> the objects for fixincl, you have the objects for applyfix.
I hope this does not feel like I am just causing trouble.
Anyway, I would rather not see convoluted shell scripts put
into MAKE variables. The result is painful to look at and
understand. I need stuff to be simple in order to understand.
anyway, how about adding a file "Makefile.BEOS" or "Makefile.BEOS.in"?
Then just modify mkfixinc.sh as in:
Index: mkfixinc.sh
===================================================================
RCS file: /cvs/gcc/egcs/gcc/fixinc/mkfixinc.sh,v
retrieving revision 1.28
diff -u -r1.28 mkfixinc.sh
--- mkfixinc.sh 2000/11/17 23:48:15 1.28
+++ mkfixinc.sh 2000/11/21 16:55:28
@@ -45,6 +45,10 @@
fixincludes=fixinc.ptx
;;
+ *-*-beos* )
+ MAKE="${MAKE} -f Makefile.BEOS"
+ ;;
+
alpha*-dec-vms* | \
arm-semi-aout | armel-semi-aout | \
arm-semi-aof | armel-semi-aof | \
@@ -55,7 +59,6 @@
hppa1.1-*-bsd* | \
hppa1.0-*-bsd* | \
hppa*-*-lites* | \
- *-*-beos* | \
*-*-gnu* | \
i?86-moss-msdos* | i?86-*-moss* | \
i?86-*-osf1* | \
NB: this assumes you add "Makefile.BEOS.in". If you add just
"Makefile.BEOS" instead, then spell the make file as in:
"`dirname $0`/Makefile.BEOS". Also, if you do the ".in" variant,
you have to add the file to the list of configured files in
the "../configure.in" file. Thanks!