[perl #32223] [PATCH] Build dynclasses by default.

[Will Coleda]

# New Ticket Created by Will Coleda
# Please include the string: [perl #32223]
# in the subject line of all future correspondence about this issue.
# <URL: http://rt.perl.org:80/rt3/Ticket/Display.html?id=32223 >

I think we should be building dynclasses by default.

oolong:~/research/parrot/config/gen/makefiles coke$ cvs diff root.in
Index: root.in
===================================================================
RCS file: /cvs/public/parrot/config/gen/makefiles/root.in,v
retrieving revision 1.254
diff -b -u -r1.254 root.in
--- root.in 12 Oct 2004 09:00:16 -0000 1.254
+++ root.in 30 Oct 2004 03:31:44 -0000
@@ -463,7 +463,7 @@
#
###############################################################################

-all : flags_dummy $(TEST_PROG) runtime/parrot/include/parrotlib.pbc runtime/parrot/include/config.fpmc docs $(LIBNCI_SO) $(GEN_LIBRARY)
+all : flags_dummy $(TEST_PROG) runtime/parrot/include/parrotlib.pbc runtime/parrot/include/config.fpmc docs $(LIBNCI_SO) $(GEN_LIBRARY) dynclasses_dummy

.SUFFIXES : .c .h .pmc .dump $(O) .str .imc .pbc

@@ -581,6 +581,9 @@
@echo Compiling with:
@$(PERL) tools/dev/cc_flags.pl ./CFLAGS echo $(CC) $(CFLAGS) -I$(@D) ${cc_o_out} xx$(O) -c xx.c

+dynclasses_dummy :
+ cd dynclasses && $(MAKE)
+
runtime/parrot/include/parrotlib.pbc: runtime/parrot/library/parrotlib.imc $(TEST_PROG)
./parrot -o $@ runtime/parrot/library/parrotlib.imc

[Leopold Toetsch]

Will Coleda <parrotbug...@parrotcode.org> wrote:

> I think we should be building dynclasses by default.

Before applying it, some response from various platforms might be
helpful. Please folks either apply that patch, or run the make
command manually:

> + cd dynclasses && $(MAKE)

Do we have some tests for dynclasses?

leo

[Sam Ruby]

Leopold Toetsch wrote:

> Will Coleda <parrotbug...@parrotcode.org> wrote:
>
>>I think we should be building dynclasses by default.
>
> Before applying it, some response from various platforms might be
> helpful. Please folks either apply that patch, or run the make
> command manually:
>
>>+ cd dynclasses && $(MAKE)

Works for me. uname --all:

Linux rubix 2.4.26-1-386 #1 Tue Aug 24 13:31:19 JST 2004 i686 GNU/Linux

> Do we have some tests for dynclasses?

Not much. By making the following change to each of the
dynclasses/*.pasm files, the output of each becomes predictable:

- print I0
+ typeof S0, I0
+ print S0

- Sam Ruby

[Brent 'Dax' Royal-Gordon]

Leopold Toetsch <l...@toetsch.at> wrote:
> Will Coleda <parrotbug...@parrotcode.org> wrote:
>
> > I think we should be building dynclasses by default.
>
> Before applying it, some response from various platforms might be
> helpful. Please folks either apply that patch, or run the make
> command manually:

Compiles here, unsurprisingly.

$ uname -a
Linux brent-linux 2.6.9-gentoo-r1amd64-1 #2 Tue Oct 26 23:15:26 UTC
2004 x86_64 AMD Athlon(tm) 64 Processor 3000+ AuthenticAMD GNU/Linux

--
Brent 'Dax' Royal-Gordon <br...@brentdax.com>
Perl and Parrot hacker

There is no cabal.