First look:

There are end-of-line encoding errors on several .htm and .css files, please fix.

There are also .rur and .wld files set executable with no shebang, they should be chmodded -x unless they really need to be executable.

In build and install, it would be better to use a python macro, see the numpy spec, including the first few lines thereof.

In the desktop-file install, don't add the Application category.

Mock build is in progress, but will probably be fine as it's noarch.

Mock build was good.

Fixed.

Spec URL: http://orion.lcg.ufrj.br/RPMS/SPECS/rurple.spec

SRPM URL: http://orion.lcg.ufrj.br/RPMS/src/rurple-1.0-0.2.rc3.fc10.src.rpm

rpmlint is clean now.

The default file save location is the same as the default file open location.  The user won't have write access there, and it fails silently.  Is there a place in .rurple that could be used?  If not, patch to default to the user's home if possible.

Generally, people want to save in the same directory where the file has been read from. If I patch to save in /home, it will always save there, even if the file has been read from a user directory.

I can check if it is the first time, that is, the user has just typed a program (has not read it from anywhere):

The same thing would have to be done for saving the world. 
The file /usr/bin/rur_start.py would have a test like the one below,
which uses a new variable init_dir. What do you think?

def SaveProgramFile(self, dummy):
        if self.isRunning:
            return
        global code
        code = self.ProgramEditor.GetText()
        no_error, mesg = parser.ParseProgram(code)
        if no_error:
            wildcard = _("Program files (*.rur)|*.rur| All files (*.*)|*.*")
            fname = os.path.basename(self.filename)

            if ( self.filename == "" ):
                 init_dir = os.environ.get('HOME')
            else:
                 init_dir = misc.PRGM_DIR

            dlg = wx.FileDialog(self, _("Save new program as"), init_dir,
                               fname, wildcard, wx.SAVE| wx.CHANGE_DIR )

Issues from full review: 
See #4.

License tag should be GPLv2+
Should really be named rur-ple IMHO.  The tarball is rurple, but the project is rur-ple.  It's up to you, just my $0.02.
Otherwise it looks good.

Re: #5

You could always make a copy or the base material to a hidden folder in the user's home, so they'd each have their own copy.  Check out cylindrix in CVS and see what's been done there.

Assuming my reading of your code snippet is correct, you might not even need to do that.

Jon,

I did what you asked: I created an script that copies the data
to .rurple. Same URLs.

Regarding the package name, I do not know. It seems easier for
me to remember "yum install rurple", than having an extra "-" in the name.

But we can choose the name at the end of the revision.

Whoops forgot the flag.

Hmm. Now it won't start:

Line 43, No such file or directory, rur_images

Have you deleted your old ~/.rurple directory?

The first time, we need to create the links.

No, I haven't.  Shouldn't the script check for the links in the .rurple directory, and remove and recreate if they aren't present?

I just copied the cylindrix script, and that was I did not like.
If everything is working for you, I can improve the script, by testing 
each individual link separately.

This is the new /usr/bin/rurple script:

-----------------------------------------------------------------------


#!/bin/bash

#if no .rurple in home
if [ ! -d ~/.rurple ]; then
  #make .rurple in home
  mkdir ~/.rurple || :
fi

#link to data
if [ ! -d ~/.rurple/lessons ]; then
  ln -s /usr/share/rurple/lessons ~/.rurple/lessons || :
fi
if [ ! -d ~/.rurple/python_files ]; then
  ln -s /usr/share/rurple/python_files ~/.rurple/python_files  || :
fi
if [ ! -d ~/.rurple/rur_images ]; then
  ln -s /usr/share/rurple/rur_images ~/.rurple/rur_images || :
fi
if [ ! -d ~/.rurple/rur_locale ]; then
  ln -s /usr/share/rurple/rur_locale ~/.rurple/rur_locale || :
fi

#copy mutable data
if [ ! -d ~/.rurple/rur_programs ]; then
  cp -R -p /usr/share/rurple/rur_programs ~/.rurple/ || :
fi
if [ ! -d ~/.rurple/world_files ]; then
  cp -R -p /usr/share/rurple/world_files ~/.rurple/ || :
fi

cd ~/.rurple
exec /usr/bin/rur_start.py "$@"

Not at the old link yet.  I'd like to test it. :)

http://orion.lcg.ufrj.br/RPMS/src/rurple-1.0-0.3.rc3.fc10.src.rpm

Brilliant!  All of the above fixed, I respect your judgment on the name.

APPROVED.

Thanks Jon, for the fast review and the valuable suggestions.


New Package CVS Request
=======================
Package Name: rurple
Short Description: A Python Learning Environment
Owners: roma
Branches: F-10 F-11 F-12
InitialCC: roma

cvs done.