[racket-dev] bug in match docs

2011-02-02 Thread Robby Findler
I see this in the match docs:

  > (match (make-hat 23 'bowler)
  [(struct shoe (sz col)) sz]
  [(struct hat (sz stl)) sz])

  reference to undefined identifier: make-hat

and I think that's a bug in the docs.

Robby
_
  For list-related administrative tasks:
  http://lists.racket-lang.org/listinfo/dev


Re: [racket-dev] Pre-Release Checklist for v5.1

2011-02-02 Thread Jon Rafkind

> * Jon Rafkind 
>   Release tests for (one of the) linux releases:
>   - Test that the `racket' and `racket-textual' source releases
> compile fine
>   - Test that the binary installers for both work, try each one in
> both normal and unix-style installation modes. (just ubuntu)
>   [Note: get the release candidates from the URL in this email. Use
>the 'static table' link to see a list of all tar files available]
>

Running the plt binary installer:
http://pre.racket-lang.org/release/installers/plt-5.0.99.900-bin-i386-linux-ubuntu-jaunty.sh

Checking the integrity of the binary archive... ok.
Unpacking into "/tmp/work/x/plt/racket-tmp-install"... done.
Cannot handle distribution of shared-libraries (yet)

 === context ===
/tmp/work/x/plt/racket-tmp-install/collects/setup/unixstyle-install.rkt:383:0:
move/copy-distribution
/tmp/work/x/plt/racket-tmp-install/collects/setup/unixstyle-install.rkt:
[running body]

Error: installation failed
  (Removing installation files in /tmp/work/x/plt/racket-tmp-install)

_
  For list-related administrative tasks:
  http://lists.racket-lang.org/listinfo/dev


Re: [racket-dev] Pre-Release Checklist for v5.1

2011-02-02 Thread Eli Barzilay
A few seconds ago, Matthew Flatt wrote:
> At Wed, 2 Feb 2011 16:01:42 -0500, Eli Barzilay wrote:
> > I have a change that does that (disables plot if the directory is
> > missing).  Is it a problem if configure becomes the result of Autoconf
> > 2.61 (it's currently 2.66).
> 
> Yes, it's best if we stick to 2.66, so I'm happy to do that part.

Actually I got it backwards -- it's now 2.61, and I can make a 2.66
configure script from it.  I'll push it now.

-- 
  ((lambda (x) (x x)) (lambda (x) (x x)))  Eli Barzilay:
http://barzilay.org/   Maze is Life!
_
  For list-related administrative tasks:
  http://lists.racket-lang.org/listinfo/dev


Re: [racket-dev] Pre-Release Checklist for v5.1

2011-02-02 Thread Matthew Flatt
At Wed, 2 Feb 2011 16:01:42 -0500, Eli Barzilay wrote:
> I have a change that does that (disables plot if the directory is
> missing).  Is it a problem if configure becomes the result of Autoconf
> 2.61 (it's currently 2.66).

Yes, it's best if we stick to 2.66, so I'm happy to do that part.

_
  For list-related administrative tasks:
  http://lists.racket-lang.org/listinfo/dev


Re: [racket-dev] Pre-Release Checklist for v5.1

2011-02-02 Thread Matthew Flatt
At Wed, 02 Feb 2011 13:58:56 -0700, Jon Rafkind wrote:
> On 02/02/2011 01:26 PM, Matthew Flatt wrote:
> > At Wed, 02 Feb 2011 12:20:52 -0700, Jon Rafkind wrote:
> >> This patch makes configure ignore plot if plot/Makefile is not found. Ok
> >> to commit?
> > Don't modify "configure" directly. Its source is "racket/configure.in".
> >
> 
> My patch was for src/racket/configure.ac, the only configure.in files I
> see are
>   ./racket/gc/configure.in
>   ./lt/configure.in

Oh, right --- I misread the patch and also gave the wrong path.

I expected to see both "configure.ac" and "configure" change in the
patch, but I can apply either your patch or Eli's to "configure.in" and
generate "configure".

> > But the problem for the releaseis that "src/plot" should be included,
> > right?
> >
> 
> Well according to Eli plot is supposed to be left out of the textual
> distribution.

Ok, I see that I missed the "mz-" before, too.

_
  For list-related administrative tasks:
  http://lists.racket-lang.org/listinfo/dev


Re: [racket-dev] Pre-Release Checklist for v5.1

2011-02-02 Thread Eli Barzilay
30 minutes ago, Matthew Flatt wrote:
> At Wed, 02 Feb 2011 12:20:52 -0700, Jon Rafkind wrote:
> > This patch makes configure ignore plot if plot/Makefile is not found. Ok
> > to commit?
> 
> Don't modify "configure" directly. Its source is
> "racket/configure.in".

I have a change that does that (disables plot if the directory is
missing).  Is it a problem if configure becomes the result of Autoconf
2.61 (it's currently 2.66).



> But the problem for the releaseis that "src/plot" should be
> included, right?

The problem was in the mz distro, which doesn't include plot.

-- 
  ((lambda (x) (x x)) (lambda (x) (x x)))  Eli Barzilay:
http://barzilay.org/   Maze is Life!
_
  For list-related administrative tasks:
  http://lists.racket-lang.org/listinfo/dev


Re: [racket-dev] Pre-Release Checklist for v5.1

2011-02-02 Thread Jon Rafkind
On 02/02/2011 01:26 PM, Matthew Flatt wrote:
> At Wed, 02 Feb 2011 12:20:52 -0700, Jon Rafkind wrote:
>> This patch makes configure ignore plot if plot/Makefile is not found. Ok
>> to commit?
> Don't modify "configure" directly. Its source is "racket/configure.in".
>

My patch was for src/racket/configure.ac, the only configure.in files I
see are
  ./racket/gc/configure.in
  ./lt/configure.in

> But the problem for the releaseis that "src/plot" should be included,
> right?
>

Well according to Eli plot is supposed to be left out of the textual
distribution.
_
  For list-related administrative tasks:
  http://lists.racket-lang.org/listinfo/dev


Re: [racket-dev] Pre-Release Checklist for v5.1

2011-02-02 Thread Matthew Flatt
At Wed, 02 Feb 2011 12:20:52 -0700, Jon Rafkind wrote:
> This patch makes configure ignore plot if plot/Makefile is not found. Ok
> to commit?

Don't modify "configure" directly. Its source is "racket/configure.in".

But the problem for the releaseis that "src/plot" should be included,
right?

_
  For list-related administrative tasks:
  http://lists.racket-lang.org/listinfo/dev


Re: [racket-dev] Pre-Release Checklist for v5.1

2011-02-02 Thread Jon Rafkind
On 02/01/2011 01:05 PM, Jon Rafkind wrote:
>> * Jon Rafkind 
>>   Release tests for (one of the) linux releases:
>>   - Test that the `racket' and `racket-textual' source releases
>> compile fine
>>   - Test that the binary installers for both work, try each one in
>> both normal and unix-style installation modes. (just ubuntu)
>>   [Note: get the release candidates from the URL in this email. Use
>>the 'static table' link to see a list of all tar files available]
>>
> Using mz-5.0.99.900-src-unix.tgz, configure error'd out
>   config.status: error: cannot find input file: plot/Makefile.in
>
> _
>   For list-related administrative tasks:
>   http://lists.racket-lang.org/listinfo/dev
This patch makes configure ignore plot if plot/Makefile is not found. Ok
to commit?
diff --git a/src/racket/configure.ac b/src/racket/configure.ac
index 861fce6..bc1e4f6 100644
--- a/src/racket/configure.ac
+++ b/src/racket/configure.ac
@@ -1259,7 +1259,7 @@ fi
 makefiles="$makefiles foreign/Makefile"
 ac_configure_args="$ac_configure_args$SUB_CONFIGURE_EXTRAS"
 
-if test "${enable_plot}" = "yes" ; then
+if test "${enable_plot}" = "yes" && test -e plot/Makefile ; then
   makefiles="$makefiles
  plot/Makefile"
   MAKE_PLOT=plot
_
  For list-related administrative tasks:
  http://lists.racket-lang.org/listinfo/dev

Re: [racket-dev] Pre-Release Checklist for v5.1

2011-02-02 Thread Casey Klein
On Mon, Jan 31, 2011 at 4:50 PM, Ryan Culpepper  wrote:
>
> * Casey Klein 
>  - Redex Tests

Done.

>  Updates:
>  - Redex Updates: update HISTORY
>  (updates should show v5.1 as the most current version; email me
>  to pick the changes when they're done, or tell me if there are no such
>  changes.)
>

Done. Please merge 309bb47c7e6dc0f4cdd0bcd6c3dfd426abb3f377.

_
  For list-related administrative tasks:
  http://lists.racket-lang.org/listinfo/dev


Re: [racket-dev] Pre-Release Checklist for v5.1

2011-02-02 Thread Matthew Flatt
At Wed, 2 Feb 2011 08:44:53 -0500, Sam Tobin-Hochstadt wrote:
> On Wed, Feb 2, 2011 at 8:09 AM, Matthew Flatt  wrote:
> >
> > Recent versions of Ubuntu are configured disable images that accompany
> > text labels within buttons. For now, I made `racket/gui' enable button
> > images globally within the application, although it should really
> > should be button-specific (and only for buttons, like the ones in "PLT
> > Games", that are configured to show images independent of the user's
> > preferences).
> 
> I suspect that the "correct" way of doing this from GTK's perspective
> is to make the image a child of the button, but not part of the label.
>  I don't know if this would be a problem from the perspective of
> alignment/sizing though.

The relevant code is "collects/mred/private/wx/gtk/button.rkt".

Thanks!

_
  For list-related administrative tasks:
  http://lists.racket-lang.org/listinfo/dev


Re: [racket-dev] Pre-Release Checklist for v5.1

2011-02-02 Thread Sam Tobin-Hochstadt
On Wed, Feb 2, 2011 at 8:09 AM, Matthew Flatt  wrote:
>
> Recent versions of Ubuntu are configured disable images that accompany
> text labels within buttons. For now, I made `racket/gui' enable button
> images globally within the application, although it should really
> should be button-specific (and only for buttons, like the ones in "PLT
> Games", that are configured to show images independent of the user's
> preferences).

I suspect that the "correct" way of doing this from GTK's perspective
is to make the image a child of the button, but not part of the label.
 I don't know if this would be a problem from the perspective of
alignment/sizing though.
-- 
sam th
sa...@ccs.neu.edu
_
  For list-related administrative tasks:
  http://lists.racket-lang.org/listinfo/dev


Re: [racket-dev] Pre-Release Checklist for v5.1

2011-02-02 Thread Matthew Flatt
At Tue, 1 Feb 2011 22:27:50 -0500, Eli Barzilay wrote:
> Two hours ago, Eli Barzilay wrote:
> > 
> > Seems like there's a bigger problem -- on linux the games dialog
> > doesn't have the icons on the game buttons.
> 
> If I run the following on windows it works, but on linux (I tried
> three different setups) I get the label without the bitmap.

I pushed a fix or this.

Recent versions of Ubuntu are configured disable images that accompany
text labels within buttons. For now, I made `racket/gui' enable button
images globally within the application, although it should really
should be button-specific (and only for buttons, like the ones in "PLT
Games", that are configured to show images independent of the user's
preferences).

_
  For list-related administrative tasks:
  http://lists.racket-lang.org/listinfo/dev