Re: bug#34201: [PATCH] automake: do not require @setfilename in Texinfo files

2019-09-15 Thread Jim Meyering
On Sat, Sep 14, 2019 at 5:35 AM Gavin Smith  wrote:
> On Mon, Sep 2, 2019 at 6:28 PM Jim Meyering  wrote:
> >
> > Gavin Smith proposed a patch for this back in http://bugs.gnu.org/34201
> > Another reference to the problem: http://bugs.gnu.org/36921
> >
> > In the attached (in Gavin's name), I've added a NEWS entry and
> > adjusted the ChangeLog entry. Will push in a day or so if no comment.
>
> Thanks for taking the time to look at this.

Oh! Thanks for the reminder. I've just pushed that.



Re: bug#34201: [PATCH] automake: do not require @setfilename in Texinfo files

2019-09-14 Thread Gavin Smith
On Mon, Sep 2, 2019 at 6:28 PM Jim Meyering  wrote:
>
> Gavin Smith proposed a patch for this back in http://bugs.gnu.org/34201
> Another reference to the problem: http://bugs.gnu.org/36921
>
> In the attached (in Gavin's name), I've added a NEWS entry and
> adjusted the ChangeLog entry. Will push in a day or so if no comment.

Thanks for taking the time to look at this.



Re: [PATCH] automake: do not require @setfilename in Texinfo files

2019-09-03 Thread Mathieu Lirzin
Hello Jim,

Jim Meyering  writes:

> Gavin Smith proposed a patch for this back in http://bugs.gnu.org/34201
> Another reference to the problem: http://bugs.gnu.org/36921
>
> In the attached (in Gavin's name), I've added a NEWS entry and
> adjusted the ChangeLog entry. Will push in a day or so if no comment.

Thanks for taking care of this. :-)

> From 309a6c477eec80b847078699303c65ccd7787eb0 Mon Sep 17 00:00:00 2001
> From: Gavin Smith 
> Date: Sun, 25 Aug 2019 21:07:58 -0700
> Subject: [PATCH] automake: do not require @setfilename in Texinfo files
>
> Texinfo no longer requires a @setfilename directive in each
> .texi file, so automake now also relaxes its restriction.
> * bin/automake.in (scan_texinfo_file): Derive name of info file from
> name of input file if no @setfilename line occurs in the file.
> * t/txinfo-no-setfilename.sh: New test.
> * t/list-of-tests.mk: Add it.
> * NEWS: Mention it.
>
> Fixes automake bugs #36921 and #34201.
> ---

Not really important but HACKING 
recommends a slightly different style.

--8<---cut here---start->8---
  topic: brief description (this is the "summary line")

  

  Here goes a more detailed explanation of why the commit is needed,
  and a general overview of what it does, and how.  This section
  should almost always be provided, possibly only with the expection
  of obvious fixes or very trivial changes.

  And if the detailed explanation is quite long or detailed, you can
  want to break it in more paragraphs.

  Then you can add references to relevant mailing list discussions
  (if any), with proper links.  But don't take this as an excuse for
  writing incomplete commit messages!  The "distilled" conclusions
  reached in such discussions should have been placed in the
  paragraphs above.

  Finally, here you can thank people that motivated or helped the
  change.  So, thanks to John Doe for bringing up the issue, and to
  J. Random Hacker for providing suggestions and testing the patch.

  
--8<---cut here---end--->8---

-- 
Mathieu Lirzin
GPG: F2A3 8D7E EB2B 6640 5761  070D 0ADE E100 9460 4D37



[PATCH] automake: do not require @setfilename in Texinfo files

2019-09-02 Thread Jim Meyering
Gavin Smith proposed a patch for this back in http://bugs.gnu.org/34201
Another reference to the problem: http://bugs.gnu.org/36921

In the attached (in Gavin's name), I've added a NEWS entry and
adjusted the ChangeLog entry. Will push in a day or so if no comment.


automake-relax-setfilename.diff
Description: Binary data