* vm merged to master
@ 2009-03-13 23:04 Andy Wingo
2009-03-14 13:19 ` Neil Jerram
` (2 more replies)
0 siblings, 3 replies; 8+ messages in thread
From: Andy Wingo @ 2009-03-13 23:04 UTC (permalink / raw)
To: guile-devel
Hey all,
I went ahead and merged VM to master. Things went fine after a couple of
manual merges, though `(procedure-property apply 'arity)' needs some
work -- see the log for 5442b3a5989a118f2addf3190eab2e27d65e0658.
I should be happy but really I'm just tired ;) Can people please check
this out, give it some pounding and report any errors to the list.
Errors running external code modules are especially appreciated --
existing code should still work, with very few exceptions.
Andy
--
http://wingolog.org/
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: vm merged to master
2009-03-13 23:04 vm merged to master Andy Wingo
@ 2009-03-14 13:19 ` Neil Jerram
2009-03-16 22:48 ` Andy Wingo
2009-03-14 14:36 ` Ludovic Courtès
2009-03-15 0:01 ` Greg Troxel
2 siblings, 1 reply; 8+ messages in thread
From: Neil Jerram @ 2009-03-14 13:19 UTC (permalink / raw)
To: Andy Wingo; +Cc: guile-devel
Andy Wingo <wingo@pobox.com> writes:
> Hey all,
>
> I went ahead and merged VM to master. Things went fine after a couple of
> manual merges, though `(procedure-property apply 'arity)' needs some
> work -- see the log for 5442b3a5989a118f2addf3190eab2e27d65e0658.
>
> I should be happy but really I'm just tired ;) Can people please check
> this out, give it some pounding and report any errors to the list.
See
http://www.ossau.uklinux.net/guile/snapshots/guile-master-20090314.log.
AFAICT from a quick look:
- The main build looks fine...
- ...but the usual make check tests don't seem to be attempted anymore.
- make distcheck fails; looks like because vm-i-*.i not included in
the distribution (i.e. EXTRA_DIST).
Regards (and thanks!),
Neil
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: vm merged to master
2009-03-13 23:04 vm merged to master Andy Wingo
2009-03-14 13:19 ` Neil Jerram
@ 2009-03-14 14:36 ` Ludovic Courtès
2009-03-16 22:54 ` Andy Wingo
2009-03-15 0:01 ` Greg Troxel
2 siblings, 1 reply; 8+ messages in thread
From: Ludovic Courtès @ 2009-03-14 14:36 UTC (permalink / raw)
To: guile-devel
Hello!
Andy Wingo <wingo@pobox.com> writes:
> I went ahead and merged VM to master.
Great!
> Things went fine after a couple of
> manual merges, though `(procedure-property apply 'arity)' needs some
> work -- see the log for 5442b3a5989a118f2addf3190eab2e27d65e0658.
Right, that won't work if `apply' is a program. The question is: what
should we do with procedure properties? Before the VM, they could be
used both with subrs and Scheme closures. I think it would be nice if
we could extend them so that they worked transparently with programs as
well, maybe by attaching per-program procedure properties lazily. What
do you think?
> I should be happy but really I'm just tired ;) Can people please check
> this out, give it some pounding and report any errors to the list.
>
> Errors running external code modules are especially appreciated --
> existing code should still work, with very few exceptions.
The only troubles I had so far are:
- the `object-ref' limitation to 256 objects;
- `current-reader' not taken into account;
- circular references among modules (previously worked around via
autoloads) not supported---but this is a good thing, it's an
opportunity to remove kludges. :-)
I'll eventually proceed with the merge in the BDW-GC branches.
Thanks,
Ludo'.
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: vm merged to master
2009-03-13 23:04 vm merged to master Andy Wingo
2009-03-14 13:19 ` Neil Jerram
2009-03-14 14:36 ` Ludovic Courtès
@ 2009-03-15 0:01 ` Greg Troxel
2009-03-16 21:53 ` Andy Wingo
2 siblings, 1 reply; 8+ messages in thread
From: Greg Troxel @ 2009-03-15 0:01 UTC (permalink / raw)
To: Andy Wingo; +Cc: guile-devel
[-- Attachment #1: Type: text/plain, Size: 254 bytes --]
My autobuild failed this morning (1000Z) during the build phase:
http://autobuild.josefsson.org/guile/log-200903140604763094000.txt
It had previously been failing in make check:
http://autobuild.josefsson.org/guile/log-200903130603258959000.txt
[-- Attachment #2: Type: application/pgp-signature, Size: 193 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: vm merged to master
2009-03-15 0:01 ` Greg Troxel
@ 2009-03-16 21:53 ` Andy Wingo
0 siblings, 0 replies; 8+ messages in thread
From: Andy Wingo @ 2009-03-16 21:53 UTC (permalink / raw)
To: Greg Troxel; +Cc: guile-devel
On Sun 15 Mar 2009 01:01, Greg Troxel <gdt@ir.bbn.com> writes:
> My autobuild failed this morning (1000Z) during the build phase:
I believe this problem is fixed. Please let me know if you find others.
Andy
--
http://wingolog.org/
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: vm merged to master
2009-03-14 13:19 ` Neil Jerram
@ 2009-03-16 22:48 ` Andy Wingo
2009-03-19 0:51 ` Neil Jerram
0 siblings, 1 reply; 8+ messages in thread
From: Andy Wingo @ 2009-03-16 22:48 UTC (permalink / raw)
To: Neil Jerram; +Cc: guile-devel
Hi,
On Sat 14 Mar 2009 14:19, Neil Jerram <neil@ossau.uklinux.net> writes:
> - make distcheck fails; looks like because vm-i-*.i not included in
> the distribution (i.e. EXTRA_DIST).
I have not yet poked distcheck, but non-srcdir builds do work as of now.
I don't know what the deal with your make check issue is; can you
reproduce it with a fresh checkout?
Andy
--
http://wingolog.org/
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: vm merged to master
2009-03-14 14:36 ` Ludovic Courtès
@ 2009-03-16 22:54 ` Andy Wingo
0 siblings, 0 replies; 8+ messages in thread
From: Andy Wingo @ 2009-03-16 22:54 UTC (permalink / raw)
To: Ludovic Courtès; +Cc: guile-devel
Heya Ludo,
On Sat 14 Mar 2009 15:36, ludo@gnu.org (Ludovic Courtès) writes:
>> `(procedure-property apply 'arity)' needs some work
>
> Right, that won't work if `apply' is a program. The question is: what
> should we do with procedure properties?
You jump questions ;)
1) Procedure properties do work on programs. They just aren't
serialized to disk. But some serializations set properties on the
resulting procedure -- e.g. define*.
2) Programs do have properties -- program-properties, i think the
accessor is.
3) Should arity be accessed via procedure-properties? I think not.
Logically properties are cached values, not computed values, whereas
arity can be computed.
3.5) Should arity even be represented like that? The recent addition to
(ice-9 session), procedure-arguments, is another take on that
question.
4) For some reason I like numbered lists recently? Please take it as
some way of organizing my thoughts, it's not intended to be pedantic
;)
> Before the VM, [procedure-properties] could be used both with subrs
> and Scheme closures.
They still can.
> - the `object-ref' limitation to 256 objects;
We spoke of this on IRC, for those that weren't there. Actually, for
those that don't know: #guile is a friendly place. We actually have a
night shift going during the week, if you drop in between 21h and 24h
CET, people are likely hacking away :-)
> - `current-reader' not taken into account;
I want to fix this soon.
> - circular references among modules (previously worked around via
> autoloads) not supported---but this is a good thing, it's an
> opportunity to remove kludges. :-)
Is this really the case? Can you make a test case?
> I'll eventually proceed with the merge in the BDW-GC branches.
Great, I'm looking forward to that :-))
Cheers,
Andy
--
http://wingolog.org/
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: vm merged to master
2009-03-16 22:48 ` Andy Wingo
@ 2009-03-19 0:51 ` Neil Jerram
0 siblings, 0 replies; 8+ messages in thread
From: Neil Jerram @ 2009-03-19 0:51 UTC (permalink / raw)
To: Andy Wingo; +Cc: guile-devel
Andy Wingo <wingo@pobox.com> writes:
> Hi,
>
> On Sat 14 Mar 2009 14:19, Neil Jerram <neil@ossau.uklinux.net> writes:
>
>> - make distcheck fails; looks like because vm-i-*.i not included in
>> the distribution (i.e. EXTRA_DIST).
>
> I have not yet poked distcheck, but non-srcdir builds do work as of now.
> I don't know what the deal with your make check issue is; can you
> reproduce it with a fresh checkout?
No I can't. A fresh checkout (a couple of days ago) worked fine.
(i.e. all the tests were run and passed).
In fact I realize now that I was being quite stupid... my script for
generating the daily snapshots just doesn't run "make check" at all.
Sorry!
Regards,
Neil
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2009-03-19 0:51 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-03-13 23:04 vm merged to master Andy Wingo
2009-03-14 13:19 ` Neil Jerram
2009-03-16 22:48 ` Andy Wingo
2009-03-19 0:51 ` Neil Jerram
2009-03-14 14:36 ` Ludovic Courtès
2009-03-16 22:54 ` Andy Wingo
2009-03-15 0:01 ` Greg Troxel
2009-03-16 21:53 ` Andy Wingo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).