unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Lennart Borgman <lennart.borgman@gmail.com>
To: Stefan Monnier <monnier@iro.umontreal.ca>
Cc: Eli Zaretskii <eliz@gnu.org>, sds@gnu.org, emacs-devel@gnu.org
Subject: Re: convert-standard-filename
Date: Mon, 8 Aug 2011 00:05:05 +0200	[thread overview]
Message-ID: <CANbX367CFgQS_=znD_9UpLFLtU8CMNbR-YZ4Byy9FhT+ve_z=w@mail.gmail.com> (raw)
In-Reply-To: <jwv39hdm2ki.fsf-monnier+emacs@gnu.org>

On Sun, Aug 7, 2011 at 21:43, Stefan Monnier <monnier@iro.umontreal.ca> wrote:
>>>>> But how should it decide what is relative and what is not, e.g. in the
>>>>> case of "c:/foo" (or worse "c:foo") mentioned in the docstring?
>>>> Why by file-name-absolute-p, of course ;-)
>>> This would mean that the input is interpreted in an OS-dependent way.
>>> It would seem to make more sense to say that the arg to
>>> convert-filename-argument (or its new replacement) should be
>>> a Unix-style filename, i.e. "C:<foo>" is always interpreted as
>>> a relative file name, even under Window or DOS.
>> If we allow OS-dependent file names then does it not make more sense
>> to interpret them in the OS-dependent way? At least that seems much
>> more likely to make users surprised.
>
> The whole point of convert-standard-filename was to provide an
> OS-dependent file name from an OS-independent one (typically from
> a constant filename in a package).  So interpreting the input to this
> function in an OS-dependent way is wrong.

I would rather have it return a file name that fits the OS, regardless
of the file name that it is given. I.e. it should work both if the
argument is an Emacs standard file name or if it is an OS style file
name.

That does not fit the current doc string but the description in info:

 -- Function: convert-standard-filename filename
     This function alters the file name FILENAME to fit the conventions
     of the operating system in use, and returns the result as a new
     string.

If it does not work this way then file names read in some way must be
tested first to see if which type they are.



  reply	other threads:[~2011-08-07 22:05 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <E1QjaPL-0003HL-TM@colonialone.fsf.org>
2011-08-02  3:39 ` [Emacs-diffs] /srv/bzr/emacs/trunk r105295: * lisp/progmodes/etags.el (etags-file-of-tag, etags-tags-table-files) Stefan Monnier
2011-08-02  4:34   ` Sam Steingold
2011-08-02  4:51   ` Eli Zaretskii
2011-08-02 18:57     ` Stefan Monnier
2011-08-02 19:12       ` Eli Zaretskii
2011-08-02 20:24         ` Stefan Monnier
2011-08-03  9:05           ` Eli Zaretskii
2011-08-05 17:18             ` convert-standard-filename (was: [Emacs-diffs] /srv/bzr/emacs/trunk r105295: * lisp/progmodes/etags.el (etags-file-of-tag, etags-tags-table-files)) Stefan Monnier
2011-08-06  7:20               ` Eli Zaretskii
2011-08-07 15:33                 ` convert-standard-filename Stefan Monnier
2011-08-07 15:43                   ` convert-standard-filename Lennart Borgman
2011-08-07 19:43                     ` convert-standard-filename Stefan Monnier
2011-08-07 22:05                       ` Lennart Borgman [this message]
2011-08-08  2:54                         ` convert-standard-filename Eli Zaretskii
2011-08-08  3:23                           ` convert-standard-filename Lennart Borgman
2011-08-08  4:19                             ` convert-standard-filename Eli Zaretskii
2011-08-08  4:26                               ` convert-standard-filename Lennart Borgman
2011-08-08  4:38                                 ` convert-standard-filename Eli Zaretskii
2011-08-08 10:37                                   ` convert-standard-filename Lennart Borgman
2011-08-08 11:31                                     ` convert-standard-filename Eli Zaretskii
2011-08-08 17:15                                       ` convert-standard-filename Lennart Borgman
2011-08-07 16:18                   ` convert-standard-filename Eli Zaretskii
2011-08-07 16:22                     ` convert-standard-filename Lennart Borgman
2011-08-07 20:19                     ` convert-standard-filename Stefan Monnier
2011-08-07 20:35                       ` convert-standard-filename Eli Zaretskii
2011-08-07 21:20                         ` convert-standard-filename Sam Steingold
2011-08-08  2:53                           ` convert-standard-filename Eli Zaretskii
2011-08-08  3:24                           ` convert-standard-filename Stefan Monnier
2011-08-08  4:14                             ` convert-standard-filename Eli Zaretskii
2011-08-08 18:20                               ` convert-standard-filename Stefan Monnier
2011-08-08 19:14                                 ` convert-standard-filename Eli Zaretskii
2011-08-08 21:24                                   ` convert-standard-filename Stefan Monnier
2011-08-09  5:19                                     ` convert-standard-filename Eli Zaretskii
2011-08-09 18:33                                       ` convert-standard-filename Stefan Monnier
2011-08-09 18:36                                       ` convert-standard-filename Stefan Monnier
2011-08-08  7:33                       ` convert-standard-filename Michael Albinus
2011-08-02  8:11   ` [Emacs-diffs] /srv/bzr/emacs/trunk r105295: * lisp/progmodes/etags.el (etags-file-of-tag, etags-tags-table-files) Francesco Potortì
2011-08-02 15:33     ` Sam Steingold
2011-08-02 18:55       ` Stefan Monnier

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CANbX367CFgQS_=znD_9UpLFLtU8CMNbR-YZ4Byy9FhT+ve_z=w@mail.gmail.com' \
    --to=lennart.borgman@gmail.com \
    --cc=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=monnier@iro.umontreal.ca \
    --cc=sds@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).