X-Recipient: archive-cygwin@delorie.com
X-Spam-Check-By: sourceware.org
Message-ID: <42afa7b00802280731u7c4bebccqa5fbfe36bf239618@mail.gmail.com>
Date: Thu, 28 Feb 2008 18:31:44 +0300
From: "Kostya Altukhov" <kacygwinlist@gmail.com>
To: cygwin@cygwin.com
Subject: Re: Why does setup insist on trying to install colordiff?
In-Reply-To: <009d01c87a1d$536d2040$2e08a8c0@CAM.ARTIMI.COM>
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
References: <1ff86f510802280635y216c7d06k5f1b2febd2cf75ea@mail.gmail.com> 	 <009a01c87a18$4c347850$2e08a8c0@CAM.ARTIMI.COM> 	 <1ff86f510802280705q275483bdmad7e5d336c467bc@mail.gmail.com> 	 <009d01c87a1d$536d2040$2e08a8c0@CAM.ARTIMI.COM>
X-IsSubscribed: yes
Mailing-List: contact cygwin-help@cygwin.com; run by ezmlm
Precedence: bulk
List-Id: <cygwin.cygwin.com>
List-Unsubscribe: <mailto:cygwin-unsubscribe-archive-cygwin=delorie.com@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

On Thu, Feb 28, 2008 at 6:19 PM, Dave Korn wrote:
>  > My problem is that colordiff requires Perl, and I build my own version
>  > of Perl and don't want it downloaded via setup.exe.  Hence why I
>  > brought it up.
>
>   Oh, ouch.  Maybe until we get this sorted out we should remove the
>  dependencies.

It is probably better to fix the bug itself than to remove the dependencies :)

I've actually looked in setup.exe source and I found this piece of code:

    if (installed
        || categories.find ("Base") != categories.end ()
        || categories.find ("Misc") != categories.end ())
    {
        desired = default_version;

So by default setup attempts to install all packages in categories
Base and Misc. Why category Misc is treated as required? Maybe it
should be removed from this 'if'. But that requires changing
setup.exe, so the simplest fix would be just to change the category
for colordiff from Misc to Utils (like it was in the ITP).

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

