From: David Engster <deng@randomsample.de>
To: ding@gnus.org
Subject: Re: A Cleaner Build
Date: Thu, 17 Mar 2011 22:11:18 +0100 [thread overview]
Message-ID: <87sjultpu1.fsf@randomsample.de> (raw)
In-Reply-To: <m3aagt5ufw.fsf@quimbies.gnus.org> (Lars Magne Ingebrigtsen's message of "Thu, 17 Mar 2011 22:05:39 +0100")
Lars Magne Ingebrigtsen writes:
>> Otherwise it looks nice&clean, at least for the warnings enabled in
>> dgnushack (the buildbot does a normal 'make', not 'make warn').
>
> Right. I'll switch on the "fail on warnings" thing in dgnushack. Or
> should we have a special build target for that? Like "make warn"? I
> guess so. I've now added a "warn" target in the makefile, but it just
> does the same as "all" for now. But feel free to switch the target to
> "make warn" on the buildbot, and I'll make the "warn" target do the
> right thing later.
Erm, there is already a warn target in the lisp/Makefile which calls
(dgnushack-compile t). This will also output stuff like 'obsolete' and
'might not be defined at runtime'.
Anyway, I also think having an extra target for 'fail on warning' is the
best choice.
-David
next prev parent reply other threads:[~2011-03-17 21:11 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-03-17 18:42 Lars Magne Ingebrigtsen
2011-03-17 18:50 ` David Engster
2011-03-17 18:58 ` Lars Magne Ingebrigtsen
2011-03-17 19:00 ` Lars Magne Ingebrigtsen
2011-03-17 20:55 ` David Engster
2011-03-17 21:05 ` Lars Magne Ingebrigtsen
2011-03-17 21:11 ` David Engster [this message]
2011-03-17 21:30 ` Lars Magne Ingebrigtsen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87sjultpu1.fsf@randomsample.de \
--to=deng@randomsample.de \
--cc=ding@gnus.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).