From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mailout08.t-online.de (mailout08.t-online.de [194.25.134.20]) by sourceware.org (Postfix) with ESMTPS id B6C2E3857B8A for ; Tue, 28 Jun 2022 21:09:14 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org B6C2E3857B8A Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=t-online.de Authentication-Results: sourceware.org; spf=none smtp.mailfrom=t-online.de Received: from fwd75.dcpf.telekom.de (fwd75.aul.t-online.de [10.223.144.101]) by mailout08.t-online.de (Postfix) with SMTP id A7BECE8D3 for ; Tue, 28 Jun 2022 23:09:13 +0200 (CEST) Received: from [192.168.2.102] ([87.187.34.65]) by fwd75.t-online.de with (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384 encrypted) esmtp id 1o6ISK-4WGxbU0; Tue, 28 Jun 2022 23:09:08 +0200 Subject: Re: [ITP] etckeeper 1.18.17-1 To: cygwin-apps@cygwin.com References: <36cc4e85-d20c-0f52-214d-88262f704efa@t-online.de> <20220628174122.arrqtskqzt6gzhxc@lucy.dinwoodie.org> From: Christian Franke Message-ID: <11b99205-0b36-a700-5a07-0ed35b3df641@t-online.de> Date: Tue, 28 Jun 2022 23:09:06 +0200 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:68.0) Gecko/20100101 SeaMonkey/2.53.12 MIME-Version: 1.0 In-Reply-To: <20220628174122.arrqtskqzt6gzhxc@lucy.dinwoodie.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-TOI-EXPURGATEID: 150726::1656450548-01433199-9D77E06A/0/0 CLEAN NORMAL X-TOI-MSGID: 24b1dc98-4e65-48c3-84c7-2d2c7607a2c4 X-Spam-Status: No, score=-2.6 required=5.0 tests=BAYES_00, FREEMAIL_FROM, KAM_DMARC_STATUS, KAM_LAZY_DOMAIN_SECURITY, KAM_NUMSUBJECT, NICE_REPLY_A, RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL, SPF_HELO_NONE, SPF_NONE, TXREP, T_SCC_BODY_TEXT_LINE autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org X-BeenThere: cygwin-apps@cygwin.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Cygwin package maintainer discussion list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 28 Jun 2022 21:09:18 -0000 Adam Dinwoodie wrote: > On Tue, Jun 28, 2022 at 12:58:23PM +0200, Christian Franke wrote: >> I would like to contribute etckeeper. >> >> https://etckeeper.branchable.com/ >> https://repology.org/project/etckeeper/versions >> >> etckeeper-1.18.17-1.hint: >> category: Utils >> requires: bash coreutils grep sed >> sdesc: "Store /etc in git or mercurial" >> ldesc: "Etckeeper is a tool to let /etc be stored in git or >> mercurial.  It hooks into Cygwin Setup to automatically commit changes >> made to /etc during package upgrades.  It tracks file metadata >> (permissions, owner, group) that version control systems do not >> normally support." >> >> Package for review: >> wget -r -nH --cut-dirs=2 \ >> https://chrfranke.de/cygwin/noarch/etckeeper/etckeeper-1.18.17-1.hint \ >> https://chrfranke.de/cygwin/noarch/etckeeper/etckeeper-1.18.17-1-src.hint \ >> https://chrfranke.de/cygwin/noarch/etckeeper/etckeeper-1.18.17-1-src.tar.xz >> \ >> https://chrfranke.de/cygwin/noarch/etckeeper/etckeeper-1.18.17-1.tar.xz \ >> https://chrfranke.de/cygwin/noarch/etckeeper/etckeeper-1.18.17-1.sha256 >> >> Tested with git. Only a few tests were done with hg. > LGTM! I've not tested the actual function, but the packaging looks > sound, and I trust etckeeper enough that if the packaging is sound I'm > happy the rest will follow :) Thanks for review. I did various test, also with a local build of setup which includes my patch to automatically run 'etckeeper pre-install'. > I do wonder if it's worth trying to submit your patches upstream; they > seem like the sort of thing the upstream project might be interested in > taking, and it minimises the amount of work you have to do as a > maintainer. I definitely will do this. > I'm also vaguely pondering whether it's worth adding git as a > dependency. That's not strictly right, since etckeeper doesn't *need* > git, but it's going to be the use case for 99.9% of users, and in the > absence of Cygwin having a "recommends" style dependency, just adding > git seems like it might be sensible. But I'm far from convinced there. I'm also not sure and decided to add no git dependency. 99.8% of the users considering to install etckeeper may already have git installed :-) The Debian package does not use "rec" but "dep (git or hg or brz or darcs)" which defaults to git. If git is installed, the Debian postinst script runs 'etckeeper init && etckeeper commit' on fresh installs. I decided to leave this to the user. -- Regards, Christian