From owner-freebsd-alpha Fri May 31 2:23:12 2002 Delivered-To: freebsd-alpha@freebsd.org Received: from ns.mikhailov.org (12-234-224-124.client.attbi.com [12.234.224.124]) by hub.freebsd.org (Postfix) with ESMTP id 1765C37B405 for ; Fri, 31 May 2002 02:23:10 -0700 (PDT) Received: from mvp (mvp.mikhailov.org [10.0.0.2]) by ns.mikhailov.org (Postfix) with SMTP id 6D4DA5D12; Fri, 31 May 2002 02:23:09 -0700 (PDT) Message-ID: <000901c20884$c7e84af0$0200000a@mvp> From: "Vadim Mikhailov" To: "Andrey A. Chernov" , Cc: , , , References: <116727983.20020531125229@nagual.pp.ru> Subject: Re: ports/33170: zip -T [zip file] fails with message about missing end signature Date: Fri, 31 May 2002 02:23:10 -0700 X-Priority: 1 X-MSMail-Priority: High X-Mailer: Microsoft Outlook Express 6.00.2600.0000 X-MimeOLE: Produced By Microsoft MimeOLE V6.00.2600.0000 Sender: owner-freebsd-alpha@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.org Hi, Andrey! I agree that I should have checked for fstat() retcode, but: 1) This check does not really matter - zip was broken anyway, and I use that local fsize() function just once. 2) You could have done it yourself, it's just one more line of code. 3) We need working zip _really_ fast before cut-off date for 4.6-R to compile mozilla with prevents compiling gnome. About checking for __FreeBSD_version - this is not kernel code, it's just damn zip. My patch is just an additional sanity check before calling fseek(), it does not really depend on kernel version. So why should we care? P.S. I'm a little bit tired of this. I knew how to get things work 3 months ago, and whoever submitted/reviewed kern/6184 knew it 4 years ago. All I wanted is to help others. Looks like nobody is interested. Vadim Mikhailov. ----- Original Message ----- From: "Andrey A. Chernov" To: Cc: ; ; ; Sent: Friday, May 31, 2002 1:52 AM Subject: Re: ports/33170: zip -T [zip file] fails with message about missing end signature > > begin 666 patch-negativeseek.dat > > 1) This patch is not needed for FreeBSD-current, so all additional > code must be properly ifdef'ed with __FreeBSD_version check. > > 2) Return code of fstat() is not checked, but must be. > > -- > Andrey A. Chernov > http://ache.pp.ru/ > > To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-alpha" in the body of the message