unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Sascha Wilde <wilde@sha-bang.de>
To: "Jan Djärv" <jan.h.d@swipnet.se>
Cc: emacs-devel@gnu.org
Subject: Re: Pathnames with two (or more) slashes in Makefile.c
Date: Mon, 29 Jan 2007 21:21:12 +0100	[thread overview]
Message-ID: <m2odohvb2f.fsf@kenny.sha-bang.de> (raw)
In-Reply-To: <45BDAAFD.6050402@swipnet.se> (Jan Djärv's message of "Mon\, 29 Jan 2007 09\:06\:21 +0100")

[-- Attachment #1: Type: text/plain, Size: 879 bytes --]

Jan Djärv <jan.h.d@swipnet.se> wrote:

> Sascha Wilde skrev:
>> Jan Djärv <jan.h.d@swipnet.se> wrote:
[...]
>> So the problem is, that the double slash is written unquoted to
>> Makefile.c where it gets interpreted as a comment.
>>
>> This is IMO a bug.
>>
>> When generating Makefile.c consecutive slashes hould either be reduced
>> to one or appropriately quoted.
>
> Actually it was a bug in my patch, I should not use #define if a
> double slash may be present.  I briefly checked the other #define:s in
> Makefile.in and mostly the valuse comes from s/*.h or m/*.h, so we
> have control over these.
>
> I've checked in a fix, please try it.

Sorry, maybe I'm seriously missing something here, but I can't see
such a change by you, neither in CVS nor in the ChangeLog.

And, build still fails...  :-(

FWIW: my workaround for now looks like this:


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: emacs-configure-hotfix.patch --]
[-- Type: text/x-patch, Size: 658 bytes --]

--- a/configure.in
+++ b/configure.in
@@ -3256,6 +3256,7 @@
       < Makefile.c > junk1.c
   sed -e '1,/start of cpp stuff/d'\
       -e 's,/\*\*/#\(.*\)$,/* \1 */,' \
+      -e 's,\(#.*\)//\+,\1/,g' \
       < Makefile.c > junk.c
   $CPP $undefs -I. -I$srcdir/src $CPPFLAGS junk.c | \
       sed -e 's/^ /	/' -e '/^#/d' -e '/^[ 	\f]*$/d' > junk2.c
@@ -3272,6 +3273,7 @@
       < Makefile.c > junk1.c
   sed -e '1,/start of cpp stuff/d'\
       -e 's,/\*\*/#\(.*\)$,/* \1 */,' \
+      -e 's,\(#.*\)//\+,\1/,g' \
       < Makefile.c > junk.c
   $CPP $undefs -I. -I$srcdir/src $CPPFLAGS junk.c | \
       sed -e 's/^ /	/' -e '/^#/d' -e '/^[ 	\f]*$/d' > junk2.c

[-- Attachment #3: Type: text/plain, Size: 157 bytes --]


cheers
sascha
-- 
Sascha Wilde   -.-. ..- .-. .. --- ... .. - -.-- 
               -.- .. .-.. .-.. . -.. 
               - .... .
               -.-. .- -

[-- Attachment #4: Type: text/plain, Size: 142 bytes --]

_______________________________________________
Emacs-devel mailing list
Emacs-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-devel

  reply	other threads:[~2007-01-29 20:21 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-01-27 19:54 Latest configure.in change breaks building Sascha Wilde
2007-01-28  9:48 ` Jan Djärv
2007-01-28 14:28   ` Pathnames with two (or more) slashes in Makefile.c (was: Latest configure.in change breaks building) Sascha Wilde
2007-01-29  8:06     ` Pathnames with two (or more) slashes in Makefile.c Jan Djärv
2007-01-29 20:21       ` Sascha Wilde [this message]
2007-01-30  7:08         ` Jan Djärv
2007-01-30  8:53           ` Sascha Wilde
2007-01-30  9:09             ` Sascha Wilde
2007-01-31  9:36               ` Sascha Wilde
2007-01-31 17:42                 ` Jan Djärv
2007-01-31 22:33                   ` Sascha Wilde
2007-01-30  9:14             ` Jan Djärv
2007-01-30  9:47               ` Sascha Wilde
2007-01-30 10:26                 ` Jan Djärv
2007-01-30 11:04                   ` Sascha Wilde

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=m2odohvb2f.fsf@kenny.sha-bang.de \
    --to=wilde@sha-bang.de \
    --cc=emacs-devel@gnu.org \
    --cc=jan.h.d@swipnet.se \
    /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).