This is the mail archive of the cygwin-apps mailing list for the Cygwin project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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

On Wed, Nov 04, 2009 at 05:47:21PM +0000, Dave Korn wrote:
>    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.
>	* (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?

I think the original semantics are quite a bit less surprising and the option
should be checked first.


>RCS file: /cvs/cygwin-apps/setup/,v
>retrieving revision 2.29
>diff -p -u -r2.29
>---	4 Nov 2009 15:14:51 -0000	2.29
>+++	4 Nov 2009 16:32:42 -0000
>@@ -64,6 +64,8 @@ LocalDirSetting::LocalDirSetting ()
>   const char *fg_ret;
>   if ((fg_ret = UserSettings::instance().get ("last-cache")))
>     local_dir = std::string (fg_ret);
>+  else if (std::string (LocalDirOption).size ())
>+    local_dir = std::string (LocalDirOption);
> }
> void

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]