Re: Unbelievably obscure setup bug

2010-11-22 Thread Corinna Vinschen
On Nov 20 12:41, David Sastre wrote: On Sat, Nov 20, 2010 at 11:04:02AM +0100, Corinna Vinschen wrote: David, would you like to take over base-files? It's still officially orphaned... I'll pack a release during next week. Thanks for being patience, though (not much spare time these

Re: Unbelievably obscure setup bug

2010-11-22 Thread Corinna Vinschen
On Nov 21 19:25, Christopher Faylor wrote: On Sat, Nov 20, 2010 at 01:42:07PM +, Jon TURNEY wrote: s/you perhaps might/I should/ :-) I don't understand why we need to document this in setup sources. If we can get the right behavior with no special-case comments then why not just do

Re: Unbelievably obscure setup bug

2010-11-21 Thread Christopher Faylor
On Sat, Nov 20, 2010 at 01:42:07PM +, Jon TURNEY wrote: On 19/11/2010 14:20, Jon TURNEY wrote: On 19/11/2010 13:36, Corinna Vinschen wrote: In theory base-cygwin has no dependecies. Here are the setup.hint snippets: base-cygwin: category: Base base-passwd: requires:

Re: Unbelievably obscure setup bug

2010-11-20 Thread Corinna Vinschen
On Nov 19 19:13, David Sastre wrote: Is there a chance that this bug provides an opportunity to merge base-files and base-passwd? I was told it was difficult (or at least not recommended) because of a dependency chain issue. Please excuse me if this is non-sense. It's not. But it would

Re: Unbelievably obscure setup bug

2010-11-20 Thread David Sastre
On Sat, Nov 20, 2010 at 11:04:02AM +0100, Corinna Vinschen wrote: On Nov 19 19:13, David Sastre wrote: Is there a chance that this bug provides an opportunity to merge base-files and base-passwd? I was told it was difficult (or at least not recommended) because of a dependency chain issue.

Re: Unbelievably obscure setup bug

2010-11-20 Thread Jon TURNEY
On 19/11/2010 14:20, Jon TURNEY wrote: On 19/11/2010 13:36, Corinna Vinschen wrote: In theory base-cygwin has no dependecies. Here are the setup.hint snippets: base-cygwin: category: Base base-passwd: requires: base-cygwin category: Base noautodep: cygwin

Unbelievably obscure setup bug

2010-11-19 Thread Jon TURNEY
Whilst trying to write some other modifications to setup.exe, I've found what looks like an obscure bug introduced in the following change: CVSROOT: /sourceware/projects/cygwin-apps-home/cvsfiles Module name: setup Changes by: cori...@sourceware.org 2009-05-07 14:14:59 Modified

Re: Unbelievably obscure setup bug

2010-11-19 Thread Corinna Vinschen
On Nov 19 11:52, Jon TURNEY wrote: Whilst trying to write some other modifications to setup.exe, I've found what looks like an obscure bug introduced in the following change: CVSROOT:/sourceware/projects/cygwin-apps-home/cvsfiles Module name:setup Changes by:

Re: Unbelievably obscure setup bug

2010-11-19 Thread Jon TURNEY
On 19/11/2010 13:13, Corinna Vinschen wrote: The code in question was supposed to make sure that the order is always base-cygwin base-passwd [...] and that was the case so far. Of course, given the obvious mishandling of the casecompare return value it's not clear why this ever worked. Even

Re: Unbelievably obscure setup bug

2010-11-19 Thread Corinna Vinschen
On Nov 19 13:23, Jon TURNEY wrote: On 19/11/2010 13:13, Corinna Vinschen wrote: The code in question was supposed to make sure that the order is always base-cygwin base-passwd [...] and that was the case so far. Of course, given the obvious mishandling of the casecompare return value

Re: Unbelievably obscure setup bug

2010-11-19 Thread Jon TURNEY
On 19/11/2010 13:36, Corinna Vinschen wrote: On Nov 19 13:23, Jon TURNEY wrote: On 19/11/2010 13:13, Corinna Vinschen wrote: The code in question was supposed to make sure that the order is always base-cygwin base-passwd [...] and that was the case so far. Of course, given the obvious

Re: Unbelievably obscure setup bug

2010-11-19 Thread Christopher Faylor
On Fri, Nov 19, 2010 at 02:20:13PM +, Jon TURNEY wrote: On 19/11/2010 13:36, Corinna Vinschen wrote: On Nov 19 13:23, Jon TURNEY wrote: On 19/11/2010 13:13, Corinna Vinschen wrote: The code in question was supposed to make sure that the order is always base-cygwin base-passwd [...] and

Re: Unbelievably obscure setup bug

2010-11-19 Thread Corinna Vinschen
On Nov 19 10:15, Christopher Faylor wrote: On Fri, Nov 19, 2010 at 02:20:13PM +, Jon TURNEY wrote: On 19/11/2010 13:36, Corinna Vinschen wrote: On Nov 19 13:23, Jon TURNEY wrote: On 19/11/2010 13:13, Corinna Vinschen wrote: The code in question was supposed to make sure that the order

Re: Unbelievably obscure setup bug

2010-11-19 Thread David Sastre
On Fri, Nov 19, 2010 at 06:11:57PM +0100, Corinna Vinschen wrote: On Nov 19 10:15, Christopher Faylor wrote: On Fri, Nov 19, 2010 at 02:20:13PM +, Jon TURNEY wrote: On 19/11/2010 13:36, Corinna Vinschen wrote: On Nov 19 13:23, Jon TURNEY wrote: On 19/11/2010 13:13, Corinna Vinschen