Re: [Nmh-workers] [nmh-1.2] a serious bug in the nmh configure script

[Igor Sobrado]

In message <address@hidden>, Joel Reicher writes:
> 
> I don't know what you did, but the patch as I posted it can't be applied
> to configure. It's for configure.in and aclocal.m4, and configure needs
> to be regenerated and then run.

D'oh!  You are absolutely right!!!  It was my fault.

I downloaded the right patch to my laptop but *applied* the patch I
wrote yesterday instead (I did "patch < ../configure.patch" instead of
"patch < ../configure.in.patch").  Sorry!!!  This friday is another
perfect monday to me.  :-)

Indeed, your patch applies cleanly to both configure.in and aclocal.m4:

$ patch < ../configure.in.patch                                                
Hmm...  Looks like a unified diff to me...
The text leading up to this was:
--------------------------
|--- configure.in.orig  2005-12-15 11:45:36.000000000 +1100
|+++ configure.in       2005-12-23 09:07:29.000000000 +1100
--------------------------
Patching file configure.in using Plan A...
Hunk #1 succeeded at 603.
Hmm...  The next patch looks like a unified diff to me...
The text leading up to this was:
--------------------------
|--- aclocal.m4.orig    2005-12-15 11:45:36.000000000 +1100
|+++ aclocal.m4 2005-12-23 09:14:06.000000000 +1100
--------------------------
Patching file aclocal.m4 using Plan A...
Hunk #1 succeeded at 44.
Hunk #2 succeeded at 59.
Hunk #3 succeeded at 73.
done

> > ...but the problem remains.  slocal.c is not able to find the right
> > header file yet:
> 
> Either you didn't adapt the patch correctly, or you didn't rerun
> configure.

...well, I applied correctly the *wrong* patch!  :-)

Now it works *just fine* and it builds cleanly:

[...]
sed -f man.sed sortm.man > sortm.1
sed -f man.sed whatnow.man > whatnow.1
sed -f man.sed whom.man > whom.1
sed -f man.sed mh-alias.man > mh-alias.5
sed -f man.sed mh-draft.man > mh-draft.5
sed -f man.sed mh-format.man > mh-format.5
sed -f man.sed mh-mail.man > mh-mail.5
sed -f man.sed mh-profile.man > mh-profile.5
sed -f man.sed mh-sequence.man > mh-sequence.5
sed -f man.sed mh-tailor.man > mh-tailor.5
sed -f man.sed mts.conf.man > mts.conf.5
sed -f man.sed ap.man > ap.8
sed -f man.sed conflict.man > conflict.8
sed -f man.sed dp.man > dp.8
sed -f man.sed fmtdump.man > fmtdump.8
sed -f man.sed mh-mts.man > mh-mts.8
sed -f man.sed post.man > post.8

Thanks a lot!!!  The patch is a fine fix to this problem.

> > If you want access to a NetBSD machine to do some tests let me know.
> 
> I'm using a NetBSD machine.

If I can help you, or any other in this mailing list, in some other way
let me know.  I will be glad to do my best to contribute to this fine
software project.

Best regards,
Igor.