This is the mail archive of the 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]

[Patch]setup.exe type prefixes for io_stream::mkpath_p and io_stream:open paths

----- Original Message -----
From: "Robert Collins" <>
To: "Michael A Chase" <>; "Jan Nieuwenhuizen"
Cc: <>
Sent: Sunday, February 24, 2002 02:44
Subject: Re: setup.exe /var, /tmp?

> ----- Original Message -----
> From: "Michael A Chase" <>
> > These are probably not the only places where the prefixes may be
> missing.
> > Would it be worthwhile to add a log() call if no "file://" or
> "cygfile://"
> > prefix is found in io_stream::mkpath_p()?
> Possibly in your sandbox. I don't feel the need in CVS though :}.
> > I'm doing a search for all the mkpath_p() calls I can find, but often
> the
> > file or directory name is passed from somewhere else.
> Thank you.

I also looked through the calls to io_stream*::open() and found a couple
places that appeared to be missing the required prefix in


To avoid confusion, perhaps cygpath() should be renamed native_path() or
natpath().  I could make that change fairly easily if you want.

Mac :})
** I normally forward private questions to the appropriate mail list. **
Ask Smarter:
Give a hobbit a fish and he eats fish for a day.
Give a hobbit a ring and he eats fish for an age.


2002-02-24  Michael A Chase <>

    * (make_link): Add "file://" prefix to io_stream::mkpath_p()
    (make_passwd_group): Ditto.
    * (save_local_dir): Ditto.
    * (io_stream_cygfile::mklink): Wrap long lines.
    Add "file://" prefix to io_stream::open() calls.

Attachment: cinstall-mac-020224-1.patch
Description: Binary data

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