[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: More build.sh ctf fallout on a linux host
On 06/12/2018 08:50 PM, Chuck Zmudzinski wrote:
You are right - ulonglong_t is not necessary and I successfully built
tools on Debian 8 after undoing your previous fix and replacing with
your new patch of external/cddl/osnet/dist/uts/common/rpc/types.h.
I wonder if it is safe to remove u_longlong_t from <sys/types.h>.
Linux does not have it, BSD does not need it. Does any modern software
package use it?
Actually it appears u_longlong_t is needed in some of the cddl software,
but since now it can build on Linux systems that do not define that type
in the system <sys/types.h> file, I wonder if it is necessary to have
that type defined in NetBSD's system <sys/types.h> file, which is
src/sys/sys/types.h in the source tree or /usr/include/sys/types.h in an
installed system. In the build of tools on a Linux build host, it
appears the u_longlong_t is defined from the header files in
src/external/cddl/osnet/sys/sys and does not depend on u_longlong_t
being defined in /usr/include/sys/types.h.
On 06/12/2018 06:01 PM, maya%netbsd.org@localhost wrote:
blah, oops. Given the comment I'm not sure ulonglong_t is even
necessary, it doesn't appear anywhere.
Does reverting the previous and adding this (which upstream might
accept, they seem to just forget to remove it) help?
Works for building tools under netbsd.
RCS file: /cvsroot/src/external/cddl/osnet/dist/uts/common/rpc/types.h,v
retrieving revision 1.2
diff -u -r1.2 types.h
--- ./osnet/dist/uts/common/rpc/types.h 10 Apr 2015 22:44:20
+++ ./osnet/dist/uts/common/rpc/types.h 12 Jun 2018 21:55:19 -0000
@@ -49,13 +49,6 @@
typedef int bool_t;
typedef int enum_t;
- * The ulonglong_t type was introduced to workaround an rpcgen bug
- * that has been fixed, this next typedef will be removed in a
- * Do *NOT* use!
-typedef u_longlong_t ulonglong_t;
#if defined(_LP64) || defined(_I32LPx)
typedef uint32_t rpcprog_t;
typedef uint32_t rpcvers_t;
Main Index |
Thread Index |