[m-rev.] for review: convert NEWS file to markdown and begin restructuring it
Julien Fischer
jfischer at opturion.com
Tue Dec 31 04:29:39 AEDT 2019
On Tue, 31 Dec 2019, Zoltan Somogyi wrote:
> On Mon, 30 Dec 2019 23:56:51 +1100 (AEDT), Julien Fischer <jfischer at opturion.com> wrote:
>> In addition to the diff, I have attached the full NEWS file and also the
>> HTML version generated by pandoc. Reviews wishing to generate their own
>> HTML versions using pandoc, should use the command:
>>
>> $ pandoc -s -o news.html NEWS
>>
>> You should also note that ancient versions of pandoc will produce very
>> strange results. I found this out the hard way :-(
>
> Which version is too ancient, and which works?
In my case 1.12 is the ancient version -- it's what ships on CentOS 7
systems; the working version is 2.2, which is also quite old as it
happens.
>> I haven't really edited the content of the entries beyond removing some
>> redundancies and shifting some things that make break compatibility with
>> exisiting code into the appropriate section.
>
> I had a look at the initial part of the diff, and it looks fine, though I have to assume
> that the quotations are right, since I have no idea what markdown rules the NEWS
> file now follows.
They are correct. The rules we are following are here: https://commonmark.org
> The part about the individual library modules is not effectively reviewable due to
> all the reordering, but then again I don't think it needs review that badly; I, at least,
> trust you to get it right.
>
>> I will deal with the sections of the document separately.
>
> What does this mean?
I meant to say the *other* sections of the document, mainly the "Changes
to the Mercury compiler" section.
>> I would also note that there several fairly major changes that currently
>> have no entry, notably:
>>
>> * the library license change.
>> * the changes to the random module.
>
> I just noticed that there is another change that maybe should be mentioned:
> the --show-definitions compiler option.
Yes, that should be mentioned; I'll do it when I clean up the compiler
changes section.
Julien.
More information about the reviews
mailing list