X-Recipient: archive-cygwin@delorie.com
DomainKey-Signature: a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:message-id:from:to:references:in-reply-to
	:subject:date:mime-version:content-type
	:content-transfer-encoding; q=dns; s=default; b=t8/4+w5zhjZjPue/
	vHX5RUXEIMs/075+BxbjhqyNxe+7j6vG9axO/ceZPoHlxiNLkUmCiV2EG3d50yG1
	cOYE2uwCwlLledq5PbgKgEmMC5uTE+G+DV0dQTwdrkN0FvfNMqWYUsLcLlhYLO1a
	BioBqM394RPfe+eDBHAeyCzaUVY=
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:message-id:from:to:references:in-reply-to
	:subject:date:mime-version:content-type
	:content-transfer-encoding; s=default; bh=vetEopP5UHi5hFMfbDyXwk
	gQQAg=; b=FbwLTFNZTkS/UsLZYF4ksRoAWDYXDEzYPI2mUjH+817JeRfBNTU+3b
	EaMNm5rF1a+mDDyvzhrqSF45jO0GzcidbLoXd0vB6F/pAdEX3Csz4nfXodXBD65z
	6cWkDHeD25AyVhF/cIMfU+ZnSC8L3zXPi4Aft23rmj4dV0neAUbtI=
Mailing-List: contact cygwin-help@cygwin.com; run by ezmlm
List-Id: <cygwin.cygwin.com>
List-Subscribe: <mailto:cygwin-subscribe@cygwin.com>
List-Archive: <http://sourceware.org/ml/cygwin/>
List-Post: <mailto:cygwin@cygwin.com>
List-Help: <mailto:cygwin-help@cygwin.com>, <http://sourceware.org/ml/#faqs>
Sender: cygwin-owner@cygwin.com
Mail-Followup-To: cygwin@cygwin.com
Delivered-To: mailing list cygwin@cygwin.com
Authentication-Results: sourceware.org; auth=none
X-Virus-Found: No
X-Spam-SWARE-Status: No, score=-2.6 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2
X-HELO: mail-pb0-f51.google.com
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;        d=1e100.net; s=20130820;        h=x-gm-message-state:message-id:from:to:references:in-reply-to         :subject:date:mime-version:content-type:content-transfer-encoding         :importance;        bh=kSxrAmm1arpOXoJ+XGFZ2+OXMI+447zl91HZpNgYmlA=;        b=M5C7u+IJm45U/Ur7V+n0owSn+83yVwXKM1Om8H5QFDWT3Lwkx1/rcENHjdhAtVqJFr         oclZKyv4gyJfsEhiaZAOchiu9t9qwSo3dOlon8pYFDU+lPasq2ggiHYCSF/iSmRFCSyk         ROCFXiO3/gKlyuXSITqCHHrjOqeThmtfZSG4JS3Hf7GpL25xXC947ZpxZerlCCGS+kq1         92q6FWpNb44C8yEntzQ7tfry/6CnYXBUxLKPtnUwlf+PDllyk2U5+k4PFzHlCs3kU/70         XFC7gMYGh1Jw8ayayDkMhoxYR+SC06rSSXHeuuyX/ZcDnGtdhpCekCWnonVvSTf7jWfg         Jcsg==
X-Gm-Message-State: ALoCoQkm55t/ggapO5ow/8Z1vucJiz/pRD6OsUjeXuAxpANTi9s1bPwfwZpqbLP34e8aEOR6kW/Q
X-Received: by 10.68.133.229 with SMTP id pf5mr10649090pbb.115.1393700063595;        Sat, 01 Mar 2014 10:54:23 -0800 (PST)
Message-ID: <712E4E4FFFEE4E59BB840800AA4CEF2E@TKsamsung>
From: "Tony Kelman" <kelman@berkeley.edu>
To: <cygwin@cygwin.com>
References: <CFC05642E43A4A769893CD12F14EB65B@TKsamsung>
In-Reply-To: <CFC05642E43A4A769893CD12F14EB65B@TKsamsung>
Subject: Re: Unpacking 7z archives on Cygwin64
Date: Sat, 1 Mar 2014 10:54:19 -0800
MIME-Version: 1.0
Content-Type: text/plain;	format=flowed;	charset="iso-8859-1";	reply-type=response
Content-Transfer-Encoding: 7bit

> > There is an x86_64 assembly version of the same
> > file included in the source tarball which could be used instead,
> 
> Which is? May you clarify this point?

Sure. There's an Asm directory in the source tree, under which there
are x86 and x64 subfolders. They both contain versions of the file
7zCrcT8U.asm. I don't know assembly and I'm not familiar enough
with the details of p7zip to know for sure, but I'm guessing these
should do the same thing.

It should be possible to patch makefile.cygwin_asm to use the x64
assembly file instead, it would just be a bit more invasive of a patch.
The p7zip-9.20.1-1.src.patch file already has a section that touches
makefile.cygwin_asm that I could expand upon to do this.

I didn't understand the relationship between the numbered patches listed
in the PATCH_URI section of the .cygport file, and the .src.patch and
.cygwin.patch files, but I at least see the order in which they are
applied now.

Do 32-bit and 64-bit packages use the same build instructions, or would
the 64-bit package be separate? I can add in makefile conditionals to
make the same patch work for both, or use a simpler patch targeting
64-bit specifically.

-Tony


--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple

