unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Patch for Emacs 29
       [not found] <87edorlzdh.fsf.ref@yahoo.com>
@ 2023-04-11  6:14 ` Po Lu
  2023-04-11  7:50   ` Eli Zaretskii
  0 siblings, 1 reply; 3+ messages in thread
From: Po Lu @ 2023-04-11  6:14 UTC (permalink / raw)
  To: emacs-devel

Recent updates to Haiku are unable to build the Emacs 29 pretest:
cfsetspeed is present in termios.h, but is in /system/lib/libbsd.so (and
not the C library), leading to system headers conflicting with the
gnulib replacements.

The solution is to link Emacs with libbsd so that the system's version
can be found by gnulib.  Is this okay for Emacs 29?

diff --git a/configure.ac b/configure.ac
index fc846ce840b..6adc3dbd150 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2249,7 +2249,8 @@ AC_DEFUN
   ## Motif needs -lgen.
   unixware) LIBS_SYSTEM="-lsocket -lnsl -lelf -lgen" ;;
 
-  haiku) LIBS_SYSTEM="-lnetwork" ;;
+  # Recent versions of Haiku need -lbsd for cfsetspeed.
+  haiku) LIBS_SYSTEM="-lnetwork -lbsd" ;;
 esac
 
 AC_SUBST([LIBS_SYSTEM])



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: Patch for Emacs 29
  2023-04-11  6:14 ` Patch for Emacs 29 Po Lu
@ 2023-04-11  7:50   ` Eli Zaretskii
  2023-04-11  8:54     ` Po Lu
  0 siblings, 1 reply; 3+ messages in thread
From: Eli Zaretskii @ 2023-04-11  7:50 UTC (permalink / raw)
  To: Po Lu; +Cc: emacs-devel

> From: Po Lu <luangruo@yahoo.com>
> Date: Tue, 11 Apr 2023 14:14:02 +0800
> 
> Recent updates to Haiku are unable to build the Emacs 29 pretest:
> cfsetspeed is present in termios.h, but is in /system/lib/libbsd.so (and
> not the C library), leading to system headers conflicting with the
> gnulib replacements.
> 
> The solution is to link Emacs with libbsd so that the system's version
> can be found by gnulib.  Is this okay for Emacs 29?

Are you absolutely sure this couldn't do any harm when building Emacs
on older versions of Haiku?  Did you try?

If this is harmless on older versions, then okay.  Otherwise, you will
need an additional check for whether -lbsd is needed.

Thanks.



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Patch for Emacs 29
  2023-04-11  7:50   ` Eli Zaretskii
@ 2023-04-11  8:54     ` Po Lu
  0 siblings, 0 replies; 3+ messages in thread
From: Po Lu @ 2023-04-11  8:54 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: emacs-devel

Eli Zaretskii <eliz@gnu.org> writes:

> Are you absolutely sure this couldn't do any harm when building Emacs
> on older versions of Haiku?  Did you try?

Yes, I did.

> If this is harmless on older versions, then okay.  Otherwise, you will
> need an additional check for whether -lbsd is needed.
>
> Thanks.

OK, I will install this shortly.



^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2023-04-11  8:54 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <87edorlzdh.fsf.ref@yahoo.com>
2023-04-11  6:14 ` Patch for Emacs 29 Po Lu
2023-04-11  7:50   ` Eli Zaretskii
2023-04-11  8:54     ` Po Lu

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).