Bug 9522

*** This bug has been marked as a duplicate of 5825 ***

Hi,

I do not think that lisp-startup-localealias.patch is the most correct solution.
But let me says that this is the bug in GNU Emacs, because you can not use the
version which is beeing built on server with local.alias from X11 on the server
without this file. This variable should be checked on startup of Emacs not on
compilation time. The most correct solution is probably to patch some startup
file to correctly set the variable on runtime.

Your patch only removes other locale aliases from the possible completition. It
is wrong. Try to evaluate this in Emacs:

(let ((files '("/usr/lib/X11/locale/locale.alias" ; e.g. X11R6.4
                 "/usr/X11R6/lib/X11/locale/locale.alias" ; e.g. RedHat 4.2
                 "/usr/openwin/lib/locale/locale.alias" ; e.g. Solaris 2.6
                 ;;
                 ;; The following name appears after the X-related names above,
                 ;; since the X-related names are what X actually uses.
                 "/usr/share/locale/locale.alias" ; GNU/Linux sans X
                 )))
    (while (and files (not (file-exists-p (car files))))
      (setq files (cdr files)))
    (car files))

This elisp code will return the first (car) member of the list (files) which
does exist (i.e. file-exists-p). If the file does not exist it can not be
returned. The behaviour of the Emacs binary is then dependant of the machine
which the Emacs is built on which is still: try to eval source-directory in
*scratch* :-(

I think that the patch should remove the whole while cycle and use simply this
in nox:

(defvar locale-translation-file-name
  "/usr/share/locale/locale.alias" ; GNU/Linux sans X
  "*File name for the system's file of locale-name aliases, or nil if none.")

But it is a matter of beeing correct...

Another thing is that the spec file contains useless stripping:

LDFLAGS=-s
....
strip $RPM_BUILD_ROOT/usr/lib/emacs/$RPM_PACKAGE_VERSION/$ARCHDIR/$I||:

Do you still think that there are some not-stripped binaries? I do not think so.

Other changes seem to be promising - keep up the good work. The only one thing
that denies the Emacs loers to use precompiled RPM of GNU Emacs is an old
version of Gnus distributed in GNU Emacs... But it is not your fault... But you
can change it :-)

Hello,

etc/DOC-*20.5.1 files should be with the binary - I do not want etc/DOC-X-20.5.1
when I do not have emacs-X11 installed, it is almos 1.5 MB :-(