nmh-workers
[Top] [All Lists]

Re: [Nmh-workers] Having configure Check for cc's -pedantic -ansi.

2016-10-24 03:54:42
Hi David,

This now works:

  wget http://git.savannah.gnu.org/cgit/nmh.git/plain/docs/contrib/build_nmh 
&&
  sh build_nmh

Good enough?

Worked well on a machine where I hadn't built it before, but was likely
to have some of the optional development packages installed.  I went
with the defaults for all but the first prompt.

    $ sh build_nmh
    Install prefix [/usr]: /home/tmp/1477265170.143862724
    Locking type (dot|fcntl|flock|lockf) [determined by configure]: 
    MTS (smtp|sendmail/smtp|sendmail/pipe) [smtp]: 
    SMTP server(s), space separated [localhost]: 
    Cyrus SASL support [y]: 
    TLS support [y]: 
    OAuth2 support [n]: 
    $

I then had to poke around for the log file.  (With my "just paste the
commands I was told" hat on.)  Perhaps its path can be printed at the
end?

I didn't add an option to email the log file, that's easy enough with
mhmail...

I think that's fine.  They might not be able to email from that machine,
etc., so let them handle mailing it.

I wouldn't want to email the one here.  tools/showbuildenv captures the
environment and that has personal information in it, e.g. paths can
contain real-life data.  I'm wondering how useful env[] is, other than
for the uname values stuffed in there by showbuildenv.

Should it be moved out of contrib?  I was thinking of moving it to
tools, but that directory isn't included in the distribution.  Include
it?  Then build_nmh could use it (it does now only if it exists).  Or
etc?

It's a peer to configure, Makefile, etc?  So have it at the top level?
etc seems wrong for an executable script.

With flex 2.6.1-1 here, the generated C caused

    sbr/dtimep.c:1137:18: warning: comparison between signed and unsigned 
integer expressions [-Wsign-compare]
       for ( n = 0; n < max_size && \
                      ^
    sbr/dtimep.c:1837:3: note: in expansion of macro ‘YY_INPUT’
       YY_INPUT( (&YY_CURRENT_BUFFER_LVALUE->yy_ch_buf[number_to_move]),
       ^~~~~~~~
    sbr/dtimep.c: In function ‘yy_scan_bytes’:
    sbr/dtimep.c:2383:17: warning: comparison between signed and unsigned 
integer expressions [-Wsign-compare]
      for ( i = 0; i < _yybytes_len; ++i )
                     ^

And ditto for sbr/icalendar.c.  Summarised at the end of the log file as

    sbr/dtimep.c:1137:18: warning: comparison between signed and unsigned 
integer expressions [-Wsign-compare]
    sbr/dtimep.c:2383:17: warning: comparison between signed and unsigned 
integer expressions [-Wsign-compare]
    sbr/icalendar.c:766:18: warning: comparison between signed and unsigned 
integer expressions [-Wsign-compare]
    sbr/icalendar.c:1940:17: warning: comparison between signed and unsigned 
integer expressions [-Wsign-compare]

The `tail -4' output near the end, that has the ===..., has escape codes
for colour BTW.

-- 
Cheers, Ralph.
https://plus.google.com/+RalphCorderoy

_______________________________________________
Nmh-workers mailing list
Nmh-workers(_at_)nongnu(_dot_)org
https://lists.nongnu.org/mailman/listinfo/nmh-workers

<Prev in Thread] Current Thread [Next in Thread>