X-Recipient: archive-cygwin AT delorie DOT 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:from:subject:to:references:message-id:date
	:mime-version:in-reply-to:content-type
	:content-transfer-encoding; q=dns; s=default; b=ZtFkmaYDZ1iMtIEI
	8SjPw1qVurxcT1GZARnUR2S9k9kPgrS/VKmGfgvaWNNKejQb9TvJrTKlDW98MVOO
	Z2DgH8Gn5LDjP3JE9Yf5z9M65GACRVw9JynL5piEWdoz8bydOk14rCFq1yhDO1VJ
	kzFrqJUtS9X7ovlG025sUGvLN9U=
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:from:subject:to:references:message-id:date
	:mime-version:in-reply-to:content-type
	:content-transfer-encoding; s=default; bh=HhfyH7oxft5HDJPlwWO5yl
	MNyhE=; b=OaL7pDdPmT6i94dqBHYHMKjgov2Arvk3D9OPc6kuBk8BG5U5zI3E/r
	6QJ7hfrsWEOplJb0vGx4TemV6gmYgazRvMJMSQNoQ9d5u+iEr+sQ9F4Y2vWgJcWg
	OuEg0w2xCdLld7+XSizk+ixlnAZVu7ArvXozE792QMf7UKA1OD71M=
Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Id: <cygwin.cygwin.com>
List-Subscribe: <mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive: <http://sourceware.org/ml/cygwin/>
List-Post: <mailto:cygwin AT cygwin DOT com>
List-Help: <mailto:cygwin-help AT cygwin DOT com>, <http://sourceware.org/ml/#faqs>
Sender: cygwin-owner AT cygwin DOT com
Mail-Followup-To: cygwin AT cygwin DOT com
Delivered-To: mailing list cygwin AT cygwin DOT com
Authentication-Results: sourceware.org; auth=none
X-Virus-Found: No
X-Spam-SWARE-Status: No, score=-1.0 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_LOW autolearn=ham version=3.3.2
X-HELO: out1-smtp.messagingengine.com
From: Jon Turney <jon DOT turney AT dronecode DOT org DOT uk>
Subject: Re: [PATCH setup 0/3] Setup replacement for incver_ifdep
To: Achim Gratz <Stromeko-i47jiTeKxPI AT public DOT gmane DOT org>, cygwin AT cygwin DOT com
References: <1442937170-17580-1-git-send-email-jon DOT turney AT dronecode DOT org DOT uk> <87mvwegy0r DOT fsf AT Rainer DOT invalid>
Message-ID: <5602A5F4.3060505@dronecode.org.uk>
Date: Wed, 23 Sep 2015 14:15:32 +0100
User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0
MIME-Version: 1.0
In-Reply-To: <87mvwegy0r.fsf@Rainer.invalid>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 7bit

On 22/09/2015 18:32, Achim Gratz wrote:
> Jon Turney writes:
>> Since we now have scripts which run on every setup run, a package which requires
>> another package to do some work after it is installed or uninstalled can create
>> a file to act as a trigger for that to happen.
>
> There aren't any stratified scripts for pre-remove, so a lot of the
> things that you might wish for don't work anyway.  So again it would
> need to be fixed without getting a trigger or setup would need to be
> made a lot smarter than it is today.

I thought that permanent postinstall scripts run even when no packages 
are being installed, or only packages are being removed, so they they 
are effectively run every time setup is?

>> Unfortunately, it's not very practical to change to doing that for the all
>> packages which contain info files, so I am suggesting this approach.
>
> So just do it unconditionally; I was planning to change _update-info-dir
> accordingly, but haven't found the time yet.  I do the same with mandb
> locally already and I haven't looked back.

Tempting, but I don't believe that is a good solution, since it adds the 
time it takes to rebuild the info directory to every setup run.

> $ time /etc/postinstall/update-info-dir.sh.done
>
> real    0m18.169s
> user    0m3.261s
> sys     0m5.703s

I also don't think it sets a good example.  Any other packages which 
require update scripts to run (which may take even longer) when packages 
are installed/removed, should not be doing that every time setup is run, 
if avoidable.

--
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