[PATCH] Setup.exe: restore commandline localdir option.

[Dave Korn]

    Hi again!

  During the recent rejig of settings handling, the command-line "-l" option
(aka "--local-package-dir") got borked.  This patch restores it, but it might
not be quite what's wanted, because it changes the semantics slightly; that's
just the way I happened to want it to work for the stuff I was developing when
I noticed this bug, so I've sent it that way; it would be equally trivial to
restore the original semantics if that's preferred.

  In the original semantics, -l specifies a local package dir to use,
overriding any existing local package dir stored in the user settings.  In the
new semantics, if there is an existing local package dir on the machine, that
takes priority.  Like I said, that's just the way I wanted it for my purposes,
so I didn't have to bother checking (in my enclosing installer, I'm a 3PP :-O
now!) if there already was one and setting the -l option I invoke setup.exe
with differently.  For upstream setup.exe, we probably want to keep the
original semantics but I thought I'd throw it open.

	* localdir.cc (LocalDirSetting::LocalDirSetting): Restore -l option.

  OK this way?  Or the other way round, with the check for LocalDirOption
moved before the check for last-cache, as it was in the original semantics?

    cheers,
      DaveK

-------------- next part --------------
A non-text attachment was scrubbed...
Name: setup-localdir-restore-option.diff
Type: text/x-c
Size: 565 bytes
Desc: not available
URL: <http://cygwin.com/pipermail/cygwin-apps/attachments/20091104/52646f23/attachment.bin>