[Bug 282954] comms/syncterm: Update to 1.3

From: <bugzilla-noreply_at_freebsd.org>
Date: Sun, 01 Dec 2024 21:56:56 UTC
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=282954

Robert Clausecker <fuz@FreeBSD.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Assignee|ports-bugs@FreeBSD.org      |fuz@FreeBSD.org
             Status|New                         |In Progress

--- Comment #22 from Robert Clausecker <fuz@FreeBSD.org> ---
Thanks.  This looks great.
On commit, I'll use DISTVERSION over PORTVERSION as per policy.

The port also builds fine on aarch64, so I'll add that to the list of supported
architectures.  Why do you believe it should not build on architectures other
than those you have listed?  On armv7 it fails as follows:

cc -c -D__UNIX__ -DNDEBUG -I. -DDATA_LITTLEENDIAN -DFIXED_SEED=0xdb933068
-DHAS_DEVCRYPTO -fstack-protector-strong -D_FORTIFY_SOURCE=2 -Wno-pointer-sign
-Wno-switch -fwrapv -fno-delete-null-pointer-checks -fPIC -DHAS_RECURSIVE_MUTEX
-DHAS_ROBUST_MUTEX -DOSVERSION=14 -fomit-frame-pointer -pthread -o
./static-obj/int_err.o misc/int_err.c
misc/int_err.c:44:14: error: invalid operands to binary expression ('va_list'
(aka '__builtin_va_list') and 'void *')
   44 |         REQUIRES_B( verifyVAList( argPtr ) );
      |         ~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~
./misc/os_spec.h:628:37: note: expanded from macro 'verifyVAList'
  628 |   #define verifyVAList( x ) ( ( x ) != NULL )
      |                                     ^  ~~~~
./misc/safety.h:26:33: note: expanded from macro 'REQUIRES_B'
   26 | #define REQUIRES_B( x )         if( !( x ) ) retIntError_Boolean()
      |                                        ^


This seems like a wrong assertion; va_list is not guaranteed to be a pointer
type.

Could I interest you in maintainership over this port?

-- 
You are receiving this mail because:
You are the assignee for the bug.