Additional note:  I'm not yet sponsored, although I have been working on the
gallery package in BZ 181599 as well.  I'll need a sponsor for this one if
181599 isn't accepted.

gdm entry missing

I can't seem to find where this GDM entry would be...  any pointers you could
give me?

You need to have a .desktop file in /usr/share/xsessions/ 
(ie, ratpoison.desktop)

Thats what gdm looks at to generate it's list of options for sessions. 
See the gnome/kde/xfce versions for more info. (For Xfce, it's in the xfce-utils
package). 

OK, I see the files you're referring to.  How many languages /translations must
be present in the .desktop file for the package to be acceptable?

Added a .desktop file and installed to /usr/share/xsessions.  Updated packages:

SPEC: http://www.berningeronline.net/ratpoison.spec
SRPM: http://www.berningeronline.net/ratpoison-1.3.0-2.src.rpm

Here's A review:

MUST items:

See below - rpmlint output
OK - Package name.
OK - Spec file name matches.
OK - Package guidelines.
OK - License. (GPL)
OK - License field matches in spec.
OK - License included in files.
OK - Spec in american english.
OK - md5sum of source from upstream
75541248c355a3f1e09e864dd23a43f4  ratpoison-1.3.0.tar.gz
75541248c355a3f1e09e864dd23a43f4  ratpoison-1.3.0.tar.gz.1
OK - Compiles and builds on one arch at least.
OK - No forbidden buildrequires included
OK - Owns all directories it creates.
OK - No duplicate files in %files listing.
OK - Permissions on files correct.
OK - Clean section correct.
OK - Macros consistant.
OK - Code not content.
OK - Doesn't own any files/dirs that are already owned by other packages.

Items needing attention:

1. Not a blocker, but 1.4.0 is out.

2. blocker: BuildRequires: xorg-x11-devel isn't valid in fc5 and
later. Builds ok on fc4.
Suggest: 'BuildRequires: libX11-devel' for devel/fc5.

3. blocker: unpackaged files on build:
warning: Installed (but unpackaged) file(s) found:
   /usr/share/info/dir
Suggest: adding to the end of install:
rm -f ${RPM_BUILD_ROOT}/%{_infodir}/dir

4. Requires: aren't needed at all. Can be removed. rpm will detect them.

5. rpmlint has some output:
W: ratpoison strange-permission ratpoison.desktop 0755
E: ratpoison script-without-shellbang /usr/share/xsessions/ratpoison.desktop

Can be ignored.

E: ratpoison non-executable-script /usr/share/ratpoison/allwindows.sh 0644
E: ratpoison non-executable-script /usr/share/ratpoison/genrpbindings 0644
E: ratpoison non-executable-script /usr/share/ratpoison/rpshowall.sh 0644
E: ratpoison non-executable-script /usr/share/ratpoison/split.sh 0644
E: ratpoison non-executable-script /usr/share/ratpoison/clickframe.pl 0644
E: ratpoison non-executable-script /usr/share/ratpoison/rpws 0644

These should all probibly be 755?

E: ratpoison info-files-without-install-info-postin
/usr/share/info/ratpoison.info.gzE: ratpoison
info-files-without-install-info-postun /usr/share/info/ratpoison.info.gz
Should use install-info in postin, and postun?

Above errors corrected - although I didn't get the rpmlint errors about the
install-info from the old SRPM on my system.  Not sure what my system is doing
to me.  New packages:

SPEC: http://www.berningeronline.net/ratpoison.spec
SRPM: http://www.berningeronline.net/ratpoison-1.4.0-1.src.rpm

The spec in comment #8 seems to be the old spec, but the new one is in the
src.rpm, so no problems. ;) 

fc4 now builds ok, but fc5 and devel fail with: 

gcc  -O2 -g -march=i386 -mcpu=i686 -Wall -O2    -o ratpoison  actions.o bar.o
completions.o communications.o editor.o events.o format.o frame.o getopt.o
getopt1.o globals.o group.o history.o hook.o input.o linkedlist.o main.o
manage.o number.o sbuf.o screen.o split.o window.o xinerama.o  -lXinerama -lXext
-lX11  -lhistory
actions.o: In function `cmd_rathold':
/home/kevin/rpm/BUILD/ratpoison-1.4.0/src/actions.c:3032: undefined reference to
`XTestFakeButtonEvent'
/home/kevin/rpm/BUILD/ratpoison-1.4.0/src/actions.c:3030: undefined reference to
`XTestFakeButtonEvent'
actions.o: In function `cmd_ratclick':
/home/kevin/rpm/BUILD/ratpoison-1.4.0/src/actions.c:3012: undefined reference to
`XTestFakeButtonEvent'
/home/kevin/rpm/BUILD/ratpoison-1.4.0/src/actions.c:3013: undefined reference to
`XTestFakeButtonEvent'
collect2: ld returned 1 exit status
make[2]: *** [ratpoison] Error 1
make[2]: Leaving directory `/home/kevin/rpm/BUILD/ratpoison-1.4.0/src'
make[1]: *** [all] Error 2
make[1]: Leaving directory `/home/kevin/rpm/BUILD/ratpoison-1.4.0/src'
make: *** [all-recursive] Error 1
error: Bad exit status from /var/tmp/rpm-tmp.43773 (%build)

Looks like a missing BuildRequires: libXtst-devel (for fc5, devel only )

The install-info lines aren't quite right, resulting in: 
Preparing...                ########################################### [100%]
   1:ratpoison              ########################################### [100%]
install-info: No dir file specified; try --help for more information.

Please look at the Texinfo section of the scriptlets page: 
http://fedoraproject.org/wiki/ScriptletSnippets

Those are the last two blockers I see. Fix those up and I would be happy to
approve the package. 

Fixed up in new packages:

SPEC: http://www.berningeronline.net/ratpoison.spec
SRPM: http://www.berningeronline.net/ratpoison-1.4.0-2.src.rpm

Looking good. 
I did run one last rpmlint and it outputs: 

W: ratpoison devel-file-in-non-devel-package /usr/share/ratpoison/unrat.c
E: ratpoison script-without-shellbang /usr/share/ratpoison/unrat.c
E: ratpoison script-without-shellbang /usr/share/ratpoison/ratpoison.el
W: ratpoison devel-file-in-non-devel-package /usr/share/ratpoison/sloppy.c
E: ratpoison script-without-shellbang /usr/share/ratpoison/sloppy.c
E: ratpoison script-without-shellbang /usr/share/xsessions/ratpoison.desktop

You might want to chmod 644 those .c/.el files, they shouldn't need to be
executable. 

You can go ahead and do that before you import it... 

Everything looks good aside from that, so this package is approved. 
If you still need a sponsor, I would be happy to sponsor you. 

You should be able to continue the process from the "Get a Fedora Account" step
from the http://fedoraproject.org/wiki/Extras/Contributors page. 

If you have any questions feel free to drop me an email. 

Chmod's done in spec file, waiting for account sponsorship now.  Thanks for the
help!

Imported, plague FC4 builds, FC5 and devel fail on ./configure checking for X
headers.  Working on that now.