[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Nmh-workers] Having configure Check for cc's -pedantic -ansi.
From: |
Ralph Corderoy |
Subject: |
Re: [Nmh-workers] Having configure Check for cc's -pedantic -ansi. |
Date: |
Mon, 24 Oct 2016 09:54:22 +0100 |
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] Having configure Check for cc's -pedantic -ansi., Ralph Corderoy, 2016/10/21
- Re: [Nmh-workers] Having configure Check for cc's -pedantic -ansi., David Levine, 2016/10/21
- Re: [Nmh-workers] Having configure Check for cc's -pedantic -ansi., Tom Lane, 2016/10/21
- Re: [Nmh-workers] Having configure Check for cc's -pedantic -ansi., David Levine, 2016/10/23
- Re: [Nmh-workers] Having configure Check for cc's -pedantic -ansi., David Levine, 2016/10/24
- Re: [Nmh-workers] Having configure Check for cc's -pedantic -ansi., Ralph Corderoy, 2016/10/24
- Re: [Nmh-workers] Having configure Check for cc's -pedantic -ansi., David Levine, 2016/10/24
- Re: [Nmh-workers] Having configure Check for cc's -pedantic -ansi., Ralph Corderoy, 2016/10/29
- Re: [Nmh-workers] Having configure Check for cc's -pedantic -ansi., David Levine, 2016/10/29
- Re: [Nmh-workers] Having configure Check for cc's -pedantic -ansi., Ralph Corderoy, 2016/10/29
- Re: [Nmh-workers] Having configure Check for cc's -pedantic -ansi., David Levine, 2016/10/29
- Re: [Nmh-workers] Having configure Check for cc's -pedantic -ansi., Ralph Corderoy, 2016/10/29