Needs some work:

> Source0: %{name}-%{version}.tar.gz

Use only internet-names, like:

 Source0: http://www.all-day-breakfast.com/wmx/wmx-6.tar.gz

I can't find where can I download examples.

> BuildRoot: %{_tmppath}/%{name}-root

Use the following string:

 BuildRoot: 	%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)

> make

It's always a good idea to use the following expression instead of plain "make":

 %{__make} %{?_smp_mflags}

> rm -rf %{buildroot}

Instead, use:

 %{__rm} -rf %{buildroot}

and so on...

You'd better to look at the spec-files which conforms the fedora-extras rules.
You may find 'em  at the:

http://cvs.fedora.redhat.com/viewcvs/devel/?root=extras

Anyway, it's a good idea to include a lightweight wm info FE. 

(In reply to comment #1)
> Needs some work:
> 
> > Source0: %{name}-%{version}.tar.gz
> 
> Use only internet-names, like:
> 
>  Source0: http://www.all-day-breakfast.com/wmx/wmx-6.tar.gz

Oops!

 Source0: http://www.all-day-breakfast.com/%{name}/%{name}-%{version}.tar.gz

(In reply to comment #1)
> > rm -rf %{buildroot}
> 
> Instead, use:
> 
>  %{__rm} -rf %{buildroot}
> 
> and so on...

Strictly speaking, it's completely optional to use the macros instead of the
commands.

(In reply to comment #2)
> >  Source0: http://www.all-day-breakfast.com/wmx/wmx-6.tar.gz
> 
> Oops!
> 
>  Source0: http://www.all-day-breakfast.com/%{name}/%{name}-%{version}.tar.gz

Actually, macros should only be used in the tarball filename, not in the path.

Fixed spec file, bumped release to 2:

Spec Name or Url: http://www.contrib.andrew.cmu.edu/~somlo/wmx.spec
SRPM Name or Url: http://www.contrib.andrew.cmu.edu/~somlo/wmx-6pl1-2.src.rpm

the examples aren't downloaded from anywhere -- I added them to give users a
template on how to build their menu. Anyway, I reworked that: now, menu examples
go into a default systemwide directory (/usr/share/wmx/menu) and the example
xinitrc file goes in %doc.

Thanks for the comments, and let me know what you think of the changes.

(In reply to comment #4)

> Spec Name or Url: http://www.contrib.andrew.cmu.edu/~somlo/wmx.spec
> SRPM Name or Url: http://www.contrib.andrew.cmu.edu/~somlo/wmx-6pl1-2.src.rpm

 %{__install} -s -D -m 755 wmx %{buildroot}%{_prefix}/X11R6/bin/wmx

Devel-branch of FC doesn't have /usr/X11R6 and there isn't a particular reason
to use this path if it exists. Change it to:

 %{__install} -s -D -m 755 wmx %{buildroot}%{_bindir}/wmx

You should change %files section in the similar way:

%files
%defattr(-,root,root)
%doc README* UPDATES ANNOYING-BUGS wmx.xsession
%{_bindir}/wmx
%{_datadir}/%{name}

O/T:

Have you looked at the configure-log carefully?

...
checking whether you're still watching... probably not :-)
...

Got rid of X11R6, using %{_bindir} instead:

Spec Name or Url: http://www.contrib.andrew.cmu.edu/~somlo/wmx.spec
SRPM Name or Url: http://www.contrib.andrew.cmu.edu/~somlo/wmx-6pl1-3.src.rpm

> checking whether you're still watching... probably not :-)

yeah, I always thought that was rather cute... :)

I don't think it's necessary to include the package name in either Summary or
%description.  For example,

Summary: a really simple window manager for X

And %description should begin with a capital:  A really simple...

Next, it would be nice to include the appropriate facilities for this window
manager to show up in the xdm list of available sessions.  This can be done by
adding text files to /etc/X11/[g]dm/Sessions.  If it's gdm, the file would
simply be wmx, but if it's dm, it's wmx.desktop.  I'm looking at WindowMaker as
an example:

$ cat /etc/X11/gdm/Sessions/WindowMaker
#!/bin/sh
exec /etc/X11/xdm/Xsession wmaker
$ cat /etc/X11/dm/Sessions/WindowMaker.desktop
[Desktop Entry]
Encoding=UTF-8
Name=WindowMaker
Comment=Start Window Maker
Exec=wmaker
Type=Application

[Window Manager]
SessionManaged=true

Finally, in %files, I prefer to use %{_bindir}/* instead of naming the binary. 
Since this is only one in this case, it's up to you.  If the package ever grows
to include more binaries, it might be nice just to have the wildcard so there is
less to change in the future.  It's just a suggestion.

Spec Name or Url: http://www.contrib.andrew.cmu.edu/~somlo/wmx.spec
SRPM Name or Url: http://www.contrib.andrew.cmu.edu/~somlo/wmx-6pl1-4.src.rpm

Fixed 'Summary' and '%description'.

Also, added wmx.desktop. gdm has a defaul search path for .desktop files
(/etc/X11/sessions/:/etc/X11/dm/Sessions/:/usr/share/gdm/BuiltInSessions/:/usr/share/xsessions/)
Now, of these, only the latter two (both in /usr/share/) already existed on my
stock FC4 machine. So, I placed wmx.desktop in /usr/share/xsessions/ along with
the kde and gnome .desktop files which were already there. Why would I go put
them in /etc/X11/dm/Sessions/ instead ? Is the /usr/share/xsessions/ location
being deprecated?

Also, looking at WindowMaker's example, I don't get the purpose of the shell
script that goes in /etc/X11/gdm/Sessions/WindowMaker; where would that get
called from ? Is it supposed to be a wrapper for the wmaker binary called
by gdm via the .desktop file, or ???

Thanks,
Gabriel

Spec Name or Url: http://www.contrib.andrew.cmu.edu/~somlo/wmx.spec
SRPM Name or Url: http://www.contrib.andrew.cmu.edu/~somlo/wmx-6pl1-5.src.rpm

If the script /etc/X11/xinit/Xclients.d/Xclients.wmx.sh exists, gdm will use it
to fire up wmx if wmx was selected by the user at the gdm greeting screen. This
latest version supplies such a file, which either finds and fires up a custom set
of apps in the user's home directory, or starts up a few defaults (xterm and
xscreensaver) before launching wmx. Tested it on a stock FC4 box and it looks
like it's working right

Let me know what you all think.

- Patches shouldn't use macros in their naming
- The default for %setup -n already is %{name}-%{version}
- Use install -p for %{SOURCE2} instead of cp
- The summary's first letter should be capitalized
- The changelog version/release does not match the package version/release
(6pl1.5 vs. 6pl1-5)
- The package should not own /etc/X11

Spec Name or Url: http://www.contrib.andrew.cmu.edu/~somlo/wmx.spec
SRPM Name or Url: http://www.contrib.andrew.cmu.edu/~somlo/wmx-6pl1-6.src.rpm

- Missing the appropriate BuildRequires for X

Spec Name or Url: http://www.contrib.andrew.cmu.edu/~somlo/wmx.spec
SRPM Name or Url: http://www.contrib.andrew.cmu.edu/~somlo/wmx-6pl1-7.src.rpm

- Upstream source matches
- Builds clean on mock in FC4
- Runs on FC4
- Everything else looks good

APPROVED

Built under FC4 and devel, but the sponsorship process isn't complete.

sponsorship now complete, closing.