On Mon, Jun 20, 2005 at 10:25:38PM -0400, Frank W. Miller wrote:
> Copyright: Free for non-commercial use
Copyright is deprecated -- make this "License".
> Source: %{name}-src-%{version}.tar.gz
It's convenient if this is a full URL to where the source lives. Everything
up to the final / is ignored in functional terms, but still handy.
> BuildRoot: %{_tmppath}
Should be "%{_tmppath}/something". Otherwise, you're dumping everything in a
root of /var/tmp, which could be very bad. I suggest;
BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
> mkdir -p $RPM_BUILD_ROOT/etc/cornfed
> install -m 544 sip/wav/ring.wav $RPM_BUILD_ROOT/etc/cornfed
> install sip/gnome/sip $RPM_BUILD_ROOT/usr/bin/sip
> install sip/cli/sip $RPM_BUILD_ROOT/usr/bin/sip_cli
better to use %{_bindir} for /usr/bin and %{_sysconfdir} for /etc. On the
off chance that those ever change. :) And is "etc" really the best place for
a binary sound file? Maybe /usr/share/cornfed/sounds/ring.wav, and then have
a config file in /etc/cornfed that points to the right file?
> rm: cannot remove `/var/tmp': Permission denied
> error: Bad exit status from /var/tmp/rpm-tmp.9826 (%install)
[...]
> It looks like the install stanza is trying to work with /var/tmp rather
> than $RPM_BUILD_ROOT. How can I get this behavior to change?
See above -- that's what you've defined $RPM_BUILD_ROOT to *be*.
--
Matthew Miller mattdm@xxxxxxxxxx <http://www.mattdm.org/>
Boston University Linux ------> <http://linux.bu.edu/>
Current office temperature: 78 degrees Fahrenheit.
