All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mimi Zohar <zohar@linux.ibm.com>
To: Petr Vorel <pvorel@suse.cz>,
	linux-integrity@vger.kernel.org,
	Mimi Zohar <zohar@linux.vnet.ibm.com>,
	Vitaly Chikunov <vt@altlinux.org>
Subject: Re: [PATCH ima-evm-utils] Drop empty NEWS file
Date: Thu, 23 Jul 2020 07:18:18 -0400	[thread overview]
Message-ID: <1595503098.5211.85.camel@linux.ibm.com> (raw)
In-Reply-To: <1595431869.5311.102.camel@linux.ibm.com>

On Wed, 2020-07-22 at 11:31 -0400, Mimi Zohar wrote:
> On Wed, 2020-07-22 at 16:50 +0200, Petr Vorel wrote:
> > Hi,
> > 
> > ...
> > > > Really annoying having to include an empty file, but without it,
> > > > autoconf fails with:
> > 
> > > > Makefile.am: error: required file './NEWS' not found
> > > > src/Makefile.am: installing './depcomp'
> > > > autoreconf: automake failed with exit status: 1
> > 
> > > Maybe current ChangeLog should be renamed to NEWS?
> > 
> > > git log is better changelog, and ChangeLog is really condensed and
> > > suitable to be NEWS.
> > +1, good idea.
> 
> Sure, thank you for the suggestion.  I'll queue this change and any
> other change(s) in the "next" branch for a minor release.

After renaming "ChangeLog" to "NEWS" as discussed, autotools now
complains about the missing "ChangeLog" file.

automake has a number of flavors[1][2].  The "foreign" flavor, which
can be supplied as an option to AM_INIT_AUTOMAKE, doesn't require the
NEWS, COPYING, AUTHORS, ChangeLog, or README files to exist.

Although renaming the files didn't help, it makes sense.  The patch
does both - renames ChangeLog and uses the "foreign" option.

Mimi

[1] https://www.gnu.org/software/automake/manual/automake.html#index-a
utomake-options
[2] https://autotools.io/automake/options.html

  reply	other threads:[~2020-07-23 11:18 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-07-22 11:47 [PATCH ima-evm-utils] Drop empty NEWS file Petr Vorel
2020-07-22 14:27 ` Mimi Zohar
2020-07-22 14:45   ` Vitaly Chikunov
2020-07-22 14:50     ` Petr Vorel
2020-07-22 15:31       ` Mimi Zohar
2020-07-23 11:18         ` Mimi Zohar [this message]
2020-07-23 11:54           ` Petr Vorel
2020-07-23 12:07             ` Mimi Zohar
2020-07-23 12:29               ` Petr Vorel
2020-07-23 14:04                 ` Vitaly Chikunov
2020-07-23 14:30                   ` [PATCH ima-evm-utils] release feature discussion [renamed from Drop empty NEWS file] Mimi Zohar
2020-07-23 17:05                     ` Petr Vorel
2020-07-23 18:04                       ` Mimi Zohar
2020-07-24  5:28                   ` [PATCH ima-evm-utils] Drop empty NEWS file Petr Vorel
2020-07-22 14:45   ` Petr Vorel

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=1595503098.5211.85.camel@linux.ibm.com \
    --to=zohar@linux.ibm.com \
    --cc=linux-integrity@vger.kernel.org \
    --cc=pvorel@suse.cz \
    --cc=vt@altlinux.org \
    --cc=zohar@linux.vnet.ibm.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.