From: Dani Moncayo <dmoncayo@gmail.com>
To: 16099@debbugs.gnu.org
Subject: bug#16099: 24.3.50; Build failure, undefined function `cl-member'
Date: Tue, 10 Dec 2013 21:57:04 +0100 [thread overview]
Message-ID: <CAH8Pv0j+p-maZv_JBtX=VS3VsGiKLS5sX-ynxuXRYJ94kcepfA@mail.gmail.com> (raw)
In-Reply-To: <83txegwips.fsf@gnu.org>
>> I think that the function 'unmsys--file-name' is conceptually wrong, because:
>>
>> 1. It assumes that every MSYS path will match the "/c/foo/bar"
>
> It does nothing of the kind. It handles _only_ those file names that
> slip into Emacs in the /c/foo/bar form, which Emacs cannot handle.
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.
>> pattern, which in general is false (as we've already seen).
>
> 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.
>> 2. Some directory "c:/whatever" could be mounted in MSYS as "/c/foo",
>> and therefore "/c/foo/bar" should be translated as "c:/whatever/bar"
>> (not "c:/foo/bar"). Improbable but possible.
>
> 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.
>> Therefore, like I've said before, IMO this is unreliable, and we
>> should translate (or "unmsys") _all_ MSYS paths with the 'msys-to-w32'
>> script.
>
> Which is also unreliable, as we've seen.
>
> There are no bullet-proof solutions with MSYS. Building Posix
> packages on Windows is inherently fragile, and always will be.
> Therefore, the solutions should be the simplest ones we can find that
> do the job. People who do unreasonable things should be told not to.
I agree that the MSYS shell auto-conversion of paths can be tricky,
but we still don't know the origin of this problem.
In any case, the problem I pointed out doesn't seem to be the problem
reported by the OP.
Perhaps it would be interesting to see the file 'src/epaths.h'
produced in the failed build. If some path is wrong there, then
_maybe_ the culprit could be the script 'msys-to-w32'.
--
Dani Moncayo
next prev parent reply other threads:[~2013-12-10 20:57 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 [this message]
2013-12-11 17:12 ` Eli Zaretskii
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
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='CAH8Pv0j+p-maZv_JBtX=VS3VsGiKLS5sX-ynxuXRYJ94kcepfA@mail.gmail.com' \
--to=dmoncayo@gmail.com \
--cc=16099@debbugs.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).