emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
From: Achim Gratz <Stromeko@nexgo.de>
To: emacs-orgmode@gnu.org
Subject: Re: Org Build System (aka Makefile)
Date: Sun, 12 Aug 2012 22:41:21 +0200	[thread overview]
Message-ID: <87zk5zdery.fsf@Rainer.invalid> (raw)
In-Reply-To: 87k3x4rlb2.fsf@gmx.com

Eric Schulte writes:
> But we certainly shouldn't (and currently aren't?) inhibit the display
> of any warnings when the default make is run.  I was surprised to run
> make compile-source and see additional warnings which weren't shown
> during regular make.

These warnings aren't reliable — the byte compiler doesn't really try to
find and report problems.

> What is the difference between "make" and "make
> compile-source" which results in different warnings?

make -n compile
make -n _COMPILE_=single compile

The difference is starting a single Emacs and then compiling all files
vs. starting a fresh Emacs instance for each file to be compiled.  The
change was originally triggered by some differences to the builds in
package manager (ELPA) and solidified due to the fact that this is the
only method that does function with only Emacs available.  Should have
been discussed around November last year, IIRC.

> After some time digging through the make files, it looks to me like one
> must edit the local.mk file to run these.

You are welcome to dig through whatever files, but maybe you might
consult the documentation first?  As you would read there and can see
above, you can do it all on the command line if you wish.  If you want
to enact that change permanently, you should edit local.mk — that's the
only reason it exists.

> I'd propose that they are added as a separate Makefile target
> (mentioned by "make help") so that they can be easily run.

If you want additional make targets you can also implement those in
local.mk; run `make helpall´ some time and ask yourself if you really
need more.

> Very few people (users or developers) are willing to edit make
> configuration files.

Those same people that have no problem to edit the sources?  Come on,
you can't be serious.

> Perhaps these elint build options should be used to build when "make
> check" is run.  If a user is willing to run the test suite they should
> be willing to endure a slower build for more thorough warnings.

If they want to, they can edit local.mk.  But since it is not necessary
for the build and there won't be any warnings to see if the developers
do a good job, it's not a useful default.  It is maybe useful as an
additional configuration for release tests (just as it is useful to have
multiple configurations to be able to test different versions of Emacs).


Regards,
Achim.
-- 
+<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+

SD adaptations for Waldorf Q V3.00R3 and Q+ V3.54R2:
http://Synth.Stromeko.net/Downloads.html#WaldorfSDada

  reply	other threads:[~2012-08-12 20:41 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-15 20:37 Org Build System (aka Makefile) Achim Gratz
2012-07-15 21:38 ` Bastien
2012-08-09 17:03 ` Achim Gratz
2012-08-10  7:17   ` Bastien
2012-08-12 13:56     ` Achim Gratz
2012-08-12 18:56       ` Eric Schulte
2012-08-12 20:41         ` Achim Gratz [this message]
2012-08-13 13:16           ` Eric Schulte
2012-08-13 13:45             ` Bastien
2012-08-13 19:27               ` Achim Gratz
2012-08-13 22:43                 ` Eric Schulte
2012-08-14  6:13                   ` Achim Gratz
2012-08-14 12:46                     ` Eric Schulte
2012-08-14 22:06                     ` Bastien
2012-08-15 16:35                       ` Achim Gratz
2012-08-14 22:45                 ` Bastien
2012-08-15 17:55                   ` Achim Gratz
2012-08-15 18:56                     ` Bastien
2012-08-13 19:47             ` Achim Gratz
2012-08-14 22:07               ` Bastien
2012-08-12 22:27       ` Bastien
2012-08-13  6:11         ` Achim Gratz
2012-08-13  7:40           ` Bastien
2012-08-13 11:42             ` Achim Gratz
2012-08-13 13:13               ` Bastien
2012-08-13 14:17                 ` Achim Gratz
2012-08-13 14:48                   ` Bastien
2012-08-13 18:56                     ` Achim Gratz
2012-08-13  5:34       ` Bastien
2012-08-12 16:58     ` Samuel Wales

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

  List information: https://www.orgmode.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87zk5zdery.fsf@Rainer.invalid \
    --to=stromeko@nexgo.de \
    --cc=emacs-orgmode@gnu.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.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

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 read-only IMAP folder(s) and NNTP newsgroup(s).