Re: [Axiom-developer] make patch 50 work with the guessing package

[Waldek Hebisch]

Martin Rubey wrote:
> Waldek Hebisch <address@hidden> writes:
> 
> > Martin Rubey wrote:
> > > Dear Tim,
> > > 
> > > could you explain the following diff:
> > > 
> > > diff -r /local/scratch/axiom/mnt/linux/doc/hypertex/pages/util.ht 
> > > axiom49/mnt/linux/doc/hypertex/pages/util.ht
> > > 435,439c435,437
> > > < % 20030110000 tpd added a leading open paren
> > > < \newcommand{\spadsyscom}[1]{){\tt #1}}
> > > < % 20030110000 tpd these two commands are never used
> > > < %\newcommand{\spadcmd}[1]{\spadsyscom{#1}}
> > > < %\newcommand{\spadsys}[1]{\spadsyscom{#1}}
> > > ---
> > > > \newcommand{\spadsyscom}[1]{{\tt #1}}
> > > > \newcommand{\spadcmd}[1]{\spadsyscom{#1}}
> > > > \newcommand{\spadsys}[1]{\spadsyscom{#1}}
> > > 
> > > It breaks my guessing package!
> > > 
> > 
> > Hej Martin, could you explan what problem this patch causes? 
> 
> When compiling patch 50, the rootpage cannot be displayed, at least when I add
> my guessing package (i.e., building Axiom with it).
> 
> I'm not sure whether HyperDoc runs on Patch 50 unmodified, i.e., without my
> guessing package, but I think it does not.
> 
> I could check tomorrow.
> 
> 
> 
> Martin
> 
> >  Macros \spadcmd and \spadsys are used on many Hypertex pages, without them
> > Hypertex exits when you try to view such a page. Also, pages alredy contain
> > closing parentheses in system commans so \spadsyscom sould not add the 
> > second
> > one.
> 
> Exactly. That's why I don't understand at all why Tim commented out \spadcmd
> and \spadsys and added the close parenthesis. In any case, using the 
> unmodified
> version, i.e.,
> 
> > > > \newcommand{\spadsyscom}[1]{{\tt #1}}
> > > > \newcommand{\spadcmd}[1]{\spadsyscom{#1}}
> > > > \newcommand{\spadsys}[1]{\spadsyscom{#1}}
> 
> works just perfect.
> 

OK, I see:  The change that commented out the two commands has date
from 2003 and was applied to 'src/share/doc/hypertex/pages/util.ht'
long time ago. AFAICS Tim 'src/hyper/pages/util.ht' does not contain
this change so I thought that Tim reverted the change.  And I consequently
thougt that uncommenting back the commands caused problems for you.

ATM both build-improvements and wh-sandbox picked version were
the two commands are commented out, while in silver2 the two copies
are inconsistent.

In fact, I would like to do the following change to build-improvements
and wh-sandbox, but your message suggested that you have problems with it.

--- silver2/src/share/doc/hypertex/pages/util.ht        2006-10-27 
14:01:37.000000000 +0200
+++ silver2/src/hyper/pages/util.ht     2006-10-27 13:57:59.000000000 +0200
@@ -432,11 +432,9 @@
 % redundant defns for system commands
 \newcommand{\syscom}[1]{\lispdownlink{)#1}{(|syscomPage| '|#1|)}}
 %
-% 20030110000 tpd added a leading open paren
-\newcommand{\spadsyscom}[1]{){\tt #1}}
-% 20030110000 tpd these two commands are never used
-%\newcommand{\spadcmd}[1]{\spadsyscom{#1}}
-%\newcommand{\spadsys}[1]{\spadsyscom{#1}}
+\newcommand{\spadsyscom}[1]{{\tt #1}}
+\newcommand{\spadcmd}[1]{\spadsyscom{#1}}
+\newcommand{\spadsys}[1]{\spadsyscom{#1}}
 
 
 % Following macros should be phased out in favor of ones above:


-- 
                              Waldek Hebisch
address@hidden