Patch for 0.87 Spec file

[Kenneth P. <sh...@we...>]

Had to tweak the spec file for 0.87.

The first tweak is to get rid of the redundant variable definitions at
the top. These are already handled by the equivalent directives. For
example, you don't need "%define version" if you have "Version:".

The second tweak is that the macros need to be all lower-case, not
leading capitals. %{BuildRoot} doesn't work. The "BuildRoot:" directive
will automatically define %{buildroot} and this is what should get used
everywhere. (The directive will NOT redefine the macro if the macro is
already defined, such as in the packager's private macros file. This is
the desired behavior, so the packager can override settings in the spec
file.)

Tweak 3 is the addition of "BuildArchitectures: noarch". Since webmin
is all scripts, no binaries, it makes no sense to make it a i386
package.

I'm attaching a diff of the resulting spec file against the original.

BTW, if you want to check your RPM macros, just add this to the top of
your %prep section:

%dump
exit 1

This will dump all your macro settings and immediately exit without
building anything.

Ken
mailto:sh...@we...
http://www.sewingwitch.com/ken/
[If answering a mailing list posting, please don't cc me your reply. I'll take my answer on the list.]