Re: PATCH doc/refcard.tex

Re: PATCH doc/refcard.tex

[Eli Zaretskii]

> From: Adam Mirowski <mir@sunchorus.France.Sun.COM>
> Date: Tue, 26 Dec 2000 11:41:10 +0-100
> >
> >This works for me with Bash.  Hmm.  Does it work if you replace
> >"$(REFEDITS)" with $$REFEDITS ?  (You might need to "export REFEDITS"
> >somewhere in the Makefile.)
> 
> There are no quotes around the refedits variable, that
> is the problem.

Does it work in /bin/sh if you add an extra pair of quotes?  If that
helps, I will commit the change.
From pedwards@disaster.jaj.com Tue Dec 26 11:56:00 2000
From: Phil Edwards <pedwards@disaster.jaj.com>
To: Eli Zaretskii <eliz@is.elta.co.il>
Cc: mir@sunchorus.France.Sun.COM, gdb-patches@sources.redhat.com
Subject: Re: PATCH doc/refcard.tex
Date: Tue, 26 Dec 2000 11:56:00 -0000
Message-id: <20001226150540.A7688@disaster.jaj.com>
References: <200012261042.LAA19117@sunchorus.France.Sun.COM> <2110-Tue26Dec2000184743+0200-eliz@is.elta.co.il>
X-SW-Source: 2000-12/msg00259.html
Content-length: 1024

On Tue, Dec 26, 2000 at 06:47:44PM +0200, Eli Zaretskii wrote:
> > From: Adam Mirowski <mir@sunchorus.France.Sun.COM>
> > Date: Tue, 26 Dec 2000 11:41:10 +0-100
> > >
> > >This works for me with Bash.  Hmm.  Does it work if you replace
> > >"$(REFEDITS)" with $$REFEDITS ?  (You might need to "export REFEDITS"
> > >somewhere in the Makefile.)

Even using the latest version of bash 2.x that failed for me.  And since
REFEDITS is a makefile variable, it should be replaced before the shell
is called; exporting isn't necessary.


> > There are no quotes around the refedits variable, that
> > is the problem.
> 
> Does it work in /bin/sh if you add an extra pair of quotes?  If that
> helps, I will commit the change.

It helps for me, but I don't know what that will do to other shells.

Phil

-- 
pedwards at disaster dot jaj dot com  |  pme at sources dot redhat dot com
devphil at several other less interesting addresses in various dot domains
The gods do not protect fools.  Fools are protected by more capable fools.
From eliz@delorie.com Wed Dec 27 00:25:00 2000
From: Eli Zaretskii <eliz@delorie.com>
To: mir@sunchorus.France.Sun.COM
Cc: pedwards@disaster.jaj.com, gdb-patches@sources.redhat.com
Subject: Re: PATCH doc/refcard.tex
Date: Wed, 27 Dec 2000 00:25:00 -0000
Message-id: <200012270825.DAA02213@indy.delorie.com>
References: <200012261042.LAA19117@sunchorus.France.Sun.COM>
X-SW-Source: 2000-12/msg00260.html
Content-length: 781

> 
> There are no quotes around the refedits variable, that
> is the problem.

Sorry, I came to a conclusion that I don't understand the problem.
Here's the relevant fragment from doc/Makefile.in:

    # GDB QUICK REFERENCE (dvi output)
    refcard.dvi : refcard.tex $(REFEDITS)
	    if [ -z "$(REFEDITS)" ]; then \
		    cp $(srcdir)/refcard.tex sedref.tex ; \
	    else \
		    echo > tmp.sed ; \
		    for f in "$(REFEDITS)" ; do \
			    cat $(srcdir)/$$f >>tmp.sed ; done ; \
		    sed -f tmp.sed $(srcdir)/refcard.tex >sedref.tex ; \
	    fi
	    $(SET_TEXINPUTS) $(TEX) sedref.tex
	    mv sedref.dvi refcard.dvi
	    rm -f sedref.log sedref.tex tmp.sed

As you see, $(REFEDITS) _is_ wrapped in quotes, and the shell should
see ``for f in "" ; do''.  Am I missing something?
From eliz@delorie.com Wed Dec 27 00:26:00 2000
From: Eli Zaretskii <eliz@delorie.com>
To: pedwards@disaster.jaj.com
Cc: mir@sunchorus.France.Sun.COM, gdb-patches@sources.redhat.com
Subject: Re: PATCH doc/refcard.tex
Date: Wed, 27 Dec 2000 00:26:00 -0000
Message-id: <200012270826.DAA02223@indy.delorie.com>
References: <200012261042.LAA19117@sunchorus.France.Sun.COM> <2110-Tue26Dec2000184743+0200-eliz@is.elta.co.il> <20001226150540.A7688@disaster.jaj.com>
X-SW-Source: 2000-12/msg00261.html
Content-length: 567

> Date: Tue, 26 Dec 2000 15:05:40 -0500
> From: Phil Edwards <pedwards@disaster.jaj.com>
> 
> > > There are no quotes around the refedits variable, that
> > > is the problem.
> > 
> > Does it work in /bin/sh if you add an extra pair of quotes?  If that
> > helps, I will commit the change.
> 
> It helps for me, but I don't know what that will do to other shells.

Could you post diffs for a change that corrects the problem for you?
As I wrote in another message today, I don't really understand why are
you missing quotes, so I'd like to see an actual patch.

TIA

Re: PATCH doc/refcard.tex

[Phil Edwards]

On Wed, Dec 27, 2000 at 03:25:34AM -0500, Eli Zaretskii wrote:
> 
> Am I missing something?

Updated sources?  :-)

As I write this, current CVS contains no quotes.  Quotes /should/ be there,
but they don't appear to be as of...

    % cvs status Makefile.in 
    ===================================================================
    File: Makefile.in       Status: Up-to-date

       Working revision:    1.11
       Repository revision: 1.11    /cvs/src/src/gdb/doc/Makefile.in,v
       Sticky Tag:          (none)
       Sticky Date:         (none)
       Sticky Options:      -ko
    %

...revision 11.

-- 
pedwards at disaster dot jaj dot com  |  pme at sources dot redhat dot com
devphil at several other less interesting addresses in various dot domains
The gods do not protect fools.  Fools are protected by more capable fools.
From pedwards@disaster.jaj.com Wed Dec 27 00:35:00 2000
From: Phil Edwards <pedwards@disaster.jaj.com>
To: Eli Zaretskii <eliz@is.elta.co.il>
Cc: mir@sunchorus.France.Sun.COM, gdb-patches@sources.redhat.com
Subject: Re: PATCH doc/refcard.tex
Date: Wed, 27 Dec 2000 00:35:00 -0000
Message-id: <20001227034423.A10106@disaster.jaj.com>
References: <200012261042.LAA19117@sunchorus.France.Sun.COM> <2110-Tue26Dec2000184743+0200-eliz@is.elta.co.il> <20001226150540.A7688@disaster.jaj.com> <200012270826.DAA02223@indy.delorie.com>
X-SW-Source: 2000-12/msg00263.html
Content-length: 810

On Wed, Dec 27, 2000 at 03:26:11AM -0500, Eli Zaretskii wrote:
> 
> Could you post diffs for a change that corrects the problem for you?
> As I wrote in another message today, I don't really understand why are
> you missing quotes, so I'd like to see an actual patch.

No problem.


Index: Makefile.in
===================================================================
RCS file: /cvs/src/src/gdb/doc/Makefile.in,v
retrieving revision 1.11
diff -u -3 -r1.11 Makefile.in
--- Makefile.in	2000/07/09 05:41:21	1.11
+++ Makefile.in	2000/12/27 08:33:35
@@ -203,7 +203,7 @@
 		cp $(srcdir)/refcard.tex sedref.tex ; \
 	else \
 		echo > tmp.sed ; \
-		for f in $(REFEDITS) ; do \
+		for f in "$(REFEDITS)" ; do \
 			cat $(srcdir)/$$f >>tmp.sed ; done ; \
 		sed -f tmp.sed $(srcdir)/refcard.tex >sedref.tex ; \
 	fi

Re: PATCH doc/refcard.tex

[Eli Zaretskii]

> Date: Sun, 24 Dec 2000 23:50:38 -0500
> From: Phil Edwards <pedwards@disaster.jaj.com>
> 
> While trying to recall some of the more esoteric GDB commands (using
> current CVS version to try to debug some C++, sigh), I tried to print
> the reference card.  While futzing with paper sizes, I noticed that the
> version and copyright information is old.  Fixed thusly.

Thanks.  However, I think the FSF doesn't want us to use year ranges
(as in "1998--2000"), so I used explicit years instead.

> (Incidentally, the doc/Makefile.in commands for refcard.dvi won't work.
> When REFEDITS is empty, the command sequence includes "for f in ; do some
> commands ; done" and every Bourne-family shell I tried pukes on the early
> ';'.  The irony is that this is in a branch which knows that REFEDITS is
> not empty, but is joined together with the lines executed when it is empty,
> into one long shell statement.)

This works for me with Bash.  Hmm.  Does it work if you replace
"$(REFEDITS)" with $$REFEDITS ?  (You might need to "export REFEDITS"
somewhere in the Makefile.)
From mir@sunchorus.France.Sun.COM Tue Dec 26 02:42:00 2000
From: Adam Mirowski <mir@sunchorus.France.Sun.COM>
To: "Eli Zaretskii" <eliz@is.elta.co.il>, <pedwards@disaster.jaj.com>
Cc: <gdb-patches@sources.redhat.com>
Subject: Re: PATCH doc/refcard.tex
Date: Tue, 26 Dec 2000 02:42:00 -0000
Message-id: <200012261042.LAA19117@sunchorus.France.Sun.COM>
X-SW-Source: 2000-12/msg00257.html
Content-length: 988

>> (Incidentally, the doc/Makefile.in commands for refcard.dvi won't work.
>> When REFEDITS is empty, the command sequence includes "for f in ; do some
>> commands ; done" and every Bourne-family shell I tried pukes on the early
>> ';'.  The irony is that this is in a branch which knows that REFEDITS is
>> not empty, but is joined together with the lines executed when it is empty,
>> into one long shell statement.)
>
>This works for me with Bash.  Hmm.  Does it work if you replace
>"$(REFEDITS)" with $$REFEDITS ?  (You might need to "export REFEDITS"
>somewhere in the Makefile.)

There are no quotes around the refedits variable, that
is the problem. Yes, it seems that bash does not need
them in a for/in/do/done construct after "in" when the
variable passed is empty, unlike the original Bourne shell.
Looks like almost everybody compiling gdb docs must be
using bash; I was quite suprized some time ago to see
that a simple "make ps" did not work in the docs directory.

-Adam