From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from resdmta-h1p-028598.sys.comcast.net (resdmta-h1p-028598.sys.comcast.net [IPv6:2001:558:fd02:2446::e]) by sourceware.org (Postfix) with ESMTPS id C77B23858D1E for ; Mon, 10 Oct 2022 00:28:36 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org C77B23858D1E Authentication-Results: sourceware.org; dmarc=fail (p=none dis=none) header.from=acm.org Authentication-Results: sourceware.org; spf=fail smtp.mailfrom=acm.org Received: from resomta-h1p-028509.sys.comcast.net ([96.102.179.194]) by resdmta-h1p-028598.sys.comcast.net with ESMTP id hfwdovFfTYUZIhgepomaFV; Mon, 10 Oct 2022 00:28:35 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=comcastmailservice.net; s=20211018a; t=1665361715; bh=ATgeCEkpgC93po9jJR+YqAZO9m1BmGkjSwAEVdPRNxk=; h=Received:Received:Message-ID:Date:MIME-Version:Subject:To:From: Content-Type; b=sZz/lOzGm786cWNGwce/Z3PjOxsHoPJMGkxFPotmwy+O96LMPRQEMqsMOrmfRtbEV Ze7ADyRME49T8Vd7/pAqK20CsW80hNz/D908YzC4mk5yefrFN4zG7ua7v4zrcUyiUx t+Mb7kfx8dJEhaK8y7ENGmWEUbna7K3dh5YlInXFDfHavdSezxsYiU6NNp0Sj1seX2 1LdgdMF3kNU4JxlhoMuLQRv8pLGYi+Y0wEy54l4jd/gHOt4bqIr+ENYuuq1IUWOnRh Ppqj1cJZ/eUeMqkXTxMqnCARpHNhB/7ak6h20xAx6UcYrZy2SMC4E3tnk55eYMiQxj RQKaNnm0edtTg== Received: from [IPV6:2601:547:c500:dbe:4020:7910:4ac4:da9b] ([IPv6:2601:547:c500:dbe:4020:7910:4ac4:da9b]) by resomta-h1p-028509.sys.comcast.net with ESMTPSA id hgePo0euziRwJhgeRo7j3u; Mon, 10 Oct 2022 00:28:12 +0000 X-Xfinity-VMeta: sc=-100.00;st=legit Message-ID: <3a8aa97d-932a-f0d9-2507-1680d2795b5d@acm.org> Date: Sun, 9 Oct 2022 20:28:10 -0400 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.3.2 Subject: Re: [ITP] rsync 3.2.6 Content-Language: en-US To: Jon Turney , "cygwin-apps@cygwin.com" , Jari Aalto References: <9fbe1bb3-8da9-0b5d-e695-6f9b24a8b8ee@acm.org> From: Chad Dougherty In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-3.3 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,KAM_DMARC_NONE,KAM_NUMSUBJECT,NICE_REPLY_A,SPF_HELO_PASS,SPF_SOFTFAIL,TXREP autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: On 2022-10-09 10:23, Jon Turney wrote: > Thanks for looking into updating this. I'd like to give the existing > maintainer first refusal, though. > Absolutely. It's totally OK with me if Jari would rather still maintain this. I just figured I'd try my hand at the process in the meantime, particularly since it was using the old build method. > Comments on this cygport: > >> REQUIRES="libiconv2 libssl1.1 libxxhash0 libzstd1 liblz4_1" > > You don't (and in fact, shouldn't, because you then need to remember to > manually update them e.g. when soversions change) list here packages > that cygport can automatically detect as dependencies. > I see, thanks. Fixed in my working repo here: https://github.com/crd477/cygports/tree/main/rsync >> # Enable this function for releases that work without autoreconf >> src_compile() > > You should only override the default src_compile if it doesn't work. > > The fact that autoreconf isn't apparently necessary, doesn't mean it > should be omitted, since that means that future fixes to the > autoconfiguration machinery aren't automatically incorporated into the > package, but will only appear when upstream updates the autotools used > to generate the distributed autoconfiguration files. > I see. I tried but the default src_compile() didn't work. According to the INSTALL file, the autoconf should only be done on a git checkout, not the release tarball. The specific comment in the cygport file was actually a holdover from one I was using as an example so I've clarified it. > (I think this topic is touched upon in the cygport reference manual in > the section on cygautoreconf, but perhaps that could be clearer) > I actually hadn't read that section yet. The warning in the reference manual is pretty clear now that I see it though. Might it be a good idea to issue a warning diagnostic at src_compile time if cygautoreconf is skipped but it looks like the source code should be able to handle it? > Comparing the contents of the packages this produced with the current > package, there are various /usr/share/doc/rsync/*.{html,txt} files which > are no longer packaged.  Is this intentional? > It was not really intentional. The only reason they were left out is that they weren't part of the default install rule. I've also addressed this in the working repo mentioned above. Thanks for taking the time to review this. I really appreciate it. rsync is an important tool so I'm a little anxious about screwing it up for some other user of the package. -- -Chad