Mail Archives: cygwin-apps/2001/11/06/07:47:50
On Tue, 2001-11-06 at 23:32, Earnie Boyd wrote:
> Robert Collins wrote:
>
> > The setup.ini should now be ok,
> > the source discussion, IMO can be done separately from releasing
> > setup.exe.
>
> If a release includes your changes to the Download Only dialogue
> presenting the setup Cygwin root directory dialogue then I don't think
> it's ready. This shouldn't happen. The mail list will complain. I'm
> only downloading the packages not setting up Cygwin. I don't want the
> registry infected with Cygwin entries by setup.exe if I'm only
> downloading packages.
I'll happily accept a patch to not write the last-cache file for
download only installs. That combined with an alteration to cygpath I've
prepared to return NULL on failure should allow the cache read routine
to fail safely too. (with a check for NULL of course :]).
Rob
This is the patch for mount.cc.
Index: mount.cc
===================================================================
RCS file: /cvs/src/src/winsup/cinstall/mount.cc,v
retrieving revision 2.6
diff -u -p -r2.6 mount.cc
--- mount.cc 2001/10/31 13:15:05 2.6
+++ mount.cc 2001/11/06 12:44:19
@@ -444,8 +444,11 @@ cygpath (const char *s, ...)
match = m;
}
+ if (!match)
+ return NULL;
+
char *native;
- if (max_len == strlen (path))
+ if (max_len == (int) strlen (path))
native = strdup (match->native);
else
native = concat (match->native, "/", path + max_len, NULL);
- Raw text -