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-1.src.rpm Description: wmx is a really simple and basic window manager for the X windowing system. This is my first package submission, seeking sponsorship.
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.