pkgsrc-Bugs archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: pkg/43084: Fix devel/scmgit-base on big-endian

The following reply was made to PR pkg/43084; it has been noted by GNATS.

From: Holger Weiss <>
To: GNATS Bugs <>
Subject: Re: pkg/43084: Fix devel/scmgit-base on big-endian
Date: Wed, 31 Mar 2010 01:46:42 +0200

 * Joerg Sonnenberger <> [2010-03-30 23:30]:
 >  On Tue, Mar 30, 2010 at 11:25:00PM +0000, Holger Weiss wrote:
 >  > This happens in Git versions newer than, as compat/bswap.h now
 >  > redefines htonl and ntohl to bswap32 not only on selected little-endian
 >  > platforms (as intended), but also if bswap32 has been defined in the
 >  > included system headers, which is the case at least on NetBSD.  See also
 >  > my posting to the Git list:
 >  > 
 >  >
 >  It should use the system version if present. That often uses inline
 >  assembler for example.
 That may be desirable, but that's a separate issue.¹  My point is that
 Git shouldn't use bswap32 in place of htonl/ntohl on big-endian
 ¹ And that's not what the Git guys intended when they introduced the
   bug I'm talking about:;a=commit;h=0fcabdeb52b79775173d009ccc179db104dfbb66

Home | Main Index | Thread Index | Old Index