diff options
author | Arch Librarian <arch@canonical.com> | 2004-09-20 16:57:12 +0000 |
---|---|---|
committer | Arch Librarian <arch@canonical.com> | 2004-09-20 16:57:12 +0000 |
commit | 678bc33eaf95356d1b63090d220b65162478b89a (patch) | |
tree | 8e5ab6fadf5d8d94aa16fbb2f05feb001ca0b841 | |
parent | f3bab4fd4525bf5f7401a261ce24dbea70a6c479 (diff) |
BSD testing fixes
Author: jgg
Date: 2001-03-11 05:30:20 GMT
BSD testing fixes
-rw-r--r-- | COMPILING | 30 | ||||
-rw-r--r-- | apt-pkg/contrib/hashes.h | 7 | ||||
-rw-r--r-- | apt-pkg/contrib/sha1.cc | 3 | ||||
-rw-r--r-- | methods/gzip.cc | 11 |
4 files changed, 34 insertions, 17 deletions
@@ -8,10 +8,19 @@ To compile this you need a couple things - A working ANSI C++ compiler, this is not g++ 2.7.* g++ 2.8 works OK and newer egcs work well also. Nobody has tried it on other compilers :< You will need a properly working STL as well. + g++ 3 does not presently work because they made the STL headers + use namespaces. - A C library with the usual POSIX functions and a BSD socket layer. - If you OS conforms to the Single User Spec then you are fine: + If you OS conforms to the Single Unix Spec then you are fine: http://www.opengroup.org/onlinepubs/7908799/index.html +** NOTICE ** +The C++ global constructors do not link correctly when using non-shared +libaries. This is probably the correct behavior of the linker, but I have +not yet had time to devise a work around for it. The correct thing to +do is add a reference to debSystem in apt-pkg/init.cc, +assert(&debSystem == 0) would be fine for instance. + Guidelines ~~~~~~~~~~ I am not interested in making 'ultra portable code'. I will accept patches @@ -27,12 +36,12 @@ functionality. Patches to make autoconf detect these cases and generate the required shims are OK. Current shims: - * C9x integer types 'inttypes.h' - * sys/statvfs.h to convert from BSD/Linux statfs to SUS statvfs + * C99 integer types 'inttypes.h' + * sys/statvfs.h to convert from BSD/old-glibc statfs to SUS statvfs * rfc2553 hostname resolution (methods/rfc*), shims to normal gethostbyname. The more adventerous could steal the KAME IPv6 enabled resolvers for those OS's with IPv6 support but no rfc2553 (why?) - * define _XOPEN_EXTENDES_SOURCE to bring in h_errno on HP-UX + * define _XOPEN_EXTENDED_SOURCE to bring in h_errno on HP-UX * socklen_t shim in netdb.h if the OS does not have socklen_t The only completely non-shimed OS is Linux with glibc2.1, glibc2.0 requires @@ -42,24 +51,29 @@ Platform Notes ~~~~~~~~~~~~~~ Debian GNU Linux 2.1 'slink' Debian GNU Linux 'potato' +Debian GNU Linux 'woody' * All Archs - Works flawlessly - - You will want to have debiandoc-sgml and yodl installed to get + - You will want to have debiandoc-sgml and docbook2man installed to get best results. - No IPv6 Support in glibc's < 2.1. Sun Solaris SunOS cab101 5.7 Generic_106541-04 sun4u sparc + SunOS csu201 5.8 Generic_108528-04 sun4u sparc - Works fine - Note, no IPv6 Support, OS lacks RFC 2553 hostname resolution OpenBSD OpenBSD gsb086 2.5 CMPUT#0 i386 unknown - - Works fine + OpenBSD csu101 2.7 CMPUT#1 i386 unknown - OS needs 'ranlib' to generate the symbol table after 'ar'.. (not using GNU ar with the gnu tool chain :<) - - Note, no IPv6 Support, OS lacks RFC 2553 hostname resolution - + - '2.5' does not have RFC 2553 hostname resolution, but '2.7' does + - Testing on '2.7' suggests the OS has a bug in its handling of + ftruncate on files that have been written via mmap. It fills the page + that crosses the truncation boundary with 0's. + HP-UX HP-UX nyquist B.10.20 C 9000/780 2016574337 32-user license - Evil OS, does not conform very well to SUS diff --git a/apt-pkg/contrib/hashes.h b/apt-pkg/contrib/hashes.h index e00eaeb2d..481ea88f8 100644 --- a/apt-pkg/contrib/hashes.h +++ b/apt-pkg/contrib/hashes.h @@ -1,6 +1,6 @@ // -*- mode: cpp; mode: fold -*- // Description /*{{{*/ -// $Id: hashes.h,v 1.1 2001/03/06 07:15:29 jgg Exp $ +// $Id: hashes.h,v 1.2 2001/03/11 05:30:20 jgg Exp $ /* ###################################################################### Hashes - Simple wrapper around the hash functions @@ -14,7 +14,7 @@ #define APTPKG_HASHES_H #ifdef __GNUG__ -#pragma interface "apt-pkg/hashesh.h" +#pragma interface "apt-pkg/hashes.h" #endif #include <apt-pkg/md5.h> @@ -29,8 +29,7 @@ class Hashes inline bool Add(const unsigned char *Data,unsigned long Size) { - MD5.Add(Data,Size); - SHA1.Add(Data,Size); + return MD5.Add(Data,Size) && SHA1.Add(Data,Size); }; inline bool Add(const char *Data) {return Add((unsigned char *)Data,strlen(Data));}; bool AddFD(int Fd,unsigned long Size); diff --git a/apt-pkg/contrib/sha1.cc b/apt-pkg/contrib/sha1.cc index 1f3927c1c..b1c80925f 100644 --- a/apt-pkg/contrib/sha1.cc +++ b/apt-pkg/contrib/sha1.cc @@ -1,6 +1,6 @@ // -*- mode: cpp; mode: fold -*- // Description /*{{{*/ -// $Id: sha1.cc,v 1.1 2001/03/06 05:03:49 jgg Exp $ +// $Id: sha1.cc,v 1.2 2001/03/11 05:30:20 jgg Exp $ /* ###################################################################### SHA1 - SHA-1 Secure Hash Algorithm. @@ -269,7 +269,6 @@ SHA1SumValue SHA1Summation::Result() { uint32_t *state = (uint32_t *)State; uint32_t *count = (uint32_t *)Count; - uint8_t *buffer = (uint8_t *)Buffer; // Apply the padding if (Done == false) diff --git a/methods/gzip.cc b/methods/gzip.cc index 4a52c7a0c..c0092c755 100644 --- a/methods/gzip.cc +++ b/methods/gzip.cc @@ -1,6 +1,6 @@ // -*- mode: cpp; mode: fold -*- // Description /*{{{*/ -// $Id: gzip.cc,v 1.12 2001/03/06 07:15:29 jgg Exp $ +// $Id: gzip.cc,v 1.13 2001/03/11 05:30:20 jgg Exp $ /* ###################################################################### GZip method - Take a file URI in and decompress it into the target @@ -34,7 +34,8 @@ class GzipMethod : public pkgAcqMethod // GzipMethod::Fetch - Decompress the passed URI /*{{{*/ // --------------------------------------------------------------------- -/* */ +/* This forks gzip and hashes the resulting decompressed output as it + flows to the destination file */ bool GzipMethod::Fetch(FetchItem *Itm) { URI Get = Itm->Uri; @@ -102,7 +103,11 @@ bool GzipMethod::Fetch(FetchItem *Itm) break; Hash.Add(Buffer,Count); - To.Write(Buffer,Count); + if (To.Write(Buffer,Count) == false) + { + Failed = true; + break; + } } // Wait for gzip to finish |