On 30/05/07, Jonathan Underwood jonathan.underwood@gmail.com wrote:
On 27/05/07, Ville Skyttä ville.skytta@iki.fi wrote:
Regarding the current Emacsen add-on draft,
- Added some more info about requiring a version of (X)Emacs newer than or
equal to the one used to compile the *.elcs, and how to find that version out dynamically during package build.
Actually, I was just testing this, and it doesn't work.
You added these macros: %define emacsversion v=$(rpm -q --qf=%%{VERSION} emacs) ; case $v in 2*) echo $v ;; *) echo 0 ;; esac
%define xemacsversion v=$(rpm -q --qf=%%{VERSION} xemacs) ; case $v in 2*) echo $v ;; *) echo 0 ;; esac
And so whenthere is a Requires: emacs-common >= %{emacsversion} you get an error like this:
error: line 21: Dependency tokens must begin with alpha-numeric, '_' or '/': Requires: emacs-common >= v=$(rpm -q --qf=%{VERSION} emacs) ; case $v in 2*) echo $v ;; *) echo 0 ;; esac
Also, this wouldn't work as intended on a system where more than one arch of the emacs package is installed - eg. on my system i seem to have emacs.x86_64 and emacs.i386 installed, and so
rpm -q --qf=%{VERSION} emacs
gives 22.0.9922.0.99.
Granted, that wouldn't happen in a mock chroot buildroot, but still, I think we should do a bit better than that :) Can't see how though :(
J.
J.