On Wed, 2007-05-30 at 09:47 +0300, Ville Skyttä wrote: > On Wednesday 30 May 2007, Tom "spot" Callaway wrote: > > On Wed, 2007-05-30 at 00:26 +0100, Jonathan Underwood wrote: > > > On 27/05/07, Ville Skyttä <ville.skytta@xxxxxx> 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 > > > > Good lord. No. Thou shalt not query rpm inside rpm. > > Disagreed, when done carefully. There is no careful way to do it. It is not safe, it is not predictable, it is not reproducable. emacs --version | head -n1 | sed -e 's/[^0-9.]//g' xemacs -V -no-site-file | cut -d " " -f 2 Not that hard. ~spot -- Fedora-packaging mailing list Fedora-packaging@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/fedora-packaging