all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Dani Moncayo <dmoncayo@gmail.com>
Cc: 16099@debbugs.gnu.org
Subject: bug#16099: 24.3.50; Build failure, undefined function `cl-member'
Date: Wed, 11 Dec 2013 19:12:02 +0200	[thread overview]
Message-ID: <83iouvwbvh.fsf@gnu.org> (raw)
In-Reply-To: <CAH8Pv0j+p-maZv_JBtX=VS3VsGiKLS5sX-ynxuXRYJ94kcepfA@mail.gmail.com>

> Date: Tue, 10 Dec 2013 21:57:04 +0100
> From: Dani Moncayo <dmoncayo@gmail.com>
> 
> I think that it would be possible that the path to "unmsys" had the
> form "/foo/bar".  For example if someone has the source code tree
> under his MSYS tree and invokes the configure script with an absolute
> MSYS path (e.g. "/home/user/emacs/trunk/configure").
> 
> In that case, 'unmsys--file-name' will not translate the MSYS path
> ("/home/user/...") as expected.

How is this different from your use case, which is already handled?

> > Not sure what you meant here.  If you mean your use case of building
> > inside the MSYS tree, then that one should be (and was) handled by
> > different means.
> 
> It was handled in one place (for generating the native paths in
> 'src/epaths.h'), but it seems that there are more places where a
> translation to native w32 format is performed, and it would be nice if
> that translation was as reliable as possible.

Does it make any trouble?  If so, please report the details (and I
still don't understand how it works for you, since that's exactly what
you do).

If we have no specific problems, let's leave this alone, since it is
not broken.  Our aim is not to translate file names, or aim is to
build Emacs reliably and correctly.

> > People also shoot themselves in the foot, but why should we cater to
> > suicidal ones?  "If it hurts, don't do that."  MSYS is a tool to build
> > Posix packages, it has no purpose other than that.  So it makes very
> > little sense to configure MSYS in a way that interferes with its main
> > purpose.  People could do that by mistake, of course, but then the
> > solution is to recognize the mistake and correct it.
> 
> Well yes, this second problem is minor, but we could fix it with the
> same effort.

It is not a problem at all, and therefore isn't worth wasting time and
energy.

> I agree that the MSYS shell auto-conversion of paths can be tricky,
> but we still don't know the origin of this problem.

What problem are you talking about?  The lack of auto-conversion?  Its
origin is well known: MSYS simply tries to play it safe, and doesn't
auto-convert unless it is absolutely sure it sees a file name.





  reply	other threads:[~2013-12-11 17:12 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-12-10 12:13 bug#16099: 24.3.50; Build failure, undefined function `cl-member' Richard Copley
2013-12-10 16:46 ` Eli Zaretskii
2013-12-10 16:53   ` Richard Copley
2013-12-10 17:09     ` Glenn Morris
2013-12-10 18:05     ` Eli Zaretskii
2013-12-10 18:13       ` Richard Copley
2013-12-10 18:16         ` Glenn Morris
2013-12-10 18:35         ` Eli Zaretskii
     [not found] ` <handler.16099.D16099.138669538824350.notifdone@debbugs.gnu.org>
2013-12-10 17:42   ` bug#16099: closed (Re: bug#16099: 24.3.50; Build failure, undefined function `cl-member') Richard Copley
2013-12-10 17:55     ` Richard Copley
2013-12-10 18:12       ` bug#16099: 24.3.50; Build failure, undefined function `cl-member' Glenn Morris
2013-12-10 18:25         ` Richard Copley
2013-12-10 18:34           ` Eli Zaretskii
2013-12-10 18:37             ` Richard Copley
2013-12-10 18:59               ` Eli Zaretskii
2013-12-10 20:47                 ` Richard Copley
2013-12-10 20:58                   ` Eli Zaretskii
2013-12-10 21:08                     ` Glenn Morris
2013-12-11 17:07                       ` Eli Zaretskii
2013-12-10 18:42             ` Glenn Morris
2013-12-10 18:48               ` Glenn Morris
2013-12-10 18:58                 ` Eli Zaretskii
2013-12-10 20:09                 ` Dani Moncayo
2013-12-10 20:31                   ` Eli Zaretskii
2013-12-10 20:57                     ` Dani Moncayo
2013-12-11 17:12                       ` Eli Zaretskii [this message]
2013-12-12 14:08                         ` Dani Moncayo
2013-12-12 16:33                           ` Eli Zaretskii
2013-12-12 19:53                             ` Dani Moncayo
2013-12-14  9:33                               ` Dani Moncayo
2013-12-14  9:56                                 ` Eli Zaretskii
2013-12-14 20:08                                 ` Glenn Morris
2013-12-14 21:32                                   ` Dani Moncayo
2013-12-14 21:43                                     ` Glenn Morris
2013-12-14 21:56                                       ` Dani Moncayo
2013-12-10 18:58               ` Eli Zaretskii
2013-12-10 18:13     ` bug#16099: closed (Re: bug#16099: 24.3.50; Build failure, undefined function `cl-member') Eli Zaretskii

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

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

  git send-email \
    --in-reply-to=83iouvwbvh.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=16099@debbugs.gnu.org \
    --cc=dmoncayo@gmail.com \
    /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 external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.