[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] clean up release notes

From: Branko Čibej <brane_at_apache.org>
Date: Mon, 16 May 2016 14:15:06 +0200

On 16.05.2016 14:09, Stefan wrote:
> On 5/16/2016 13:43, Branko Čibej wrote:
>> On 16.05.2016 13:19, Stefan wrote:
>>> Hi,
>>>
>>> following patch fixes some more W3C validation issues inside the release
>>> notes
>>>
>>> [[[
>>> Make the release notes more compliant with the XHTML 1.1 standard.
>>>
>>> * docs/release-notes/1.6.html
>>> (Testsuite fails): add missing closing p-tag
>>>
>>> * docs/release-notes/1.7.html
>>> (svn diff can show git-style diff annotations): close p-tag before ul
>>>
>>> * docs/release-notes/1.8.html
>>> (Skelta style updates are now the default): drop invalid p-section
>> What exactly did you do with the apostrophe in this file? It shows up as
>> an invalid character for me ... note that the files have to be encoded
>> in UTF-8, if your editor changes them to Latin-1, ditch your editor. :)
>>
>> See the first hunk in the diff (which, by the way, is not mentioned here
>> in the log message.
> Ouch. That was certainly unintended. Was caused by an incorrect setting
> in WinMerge on my side (corrected and should no longer occur).
>
> Also I didn't intend to change the absence or presence of the BOM
> (didn't realize my editor is set to drop that for html files).
>
> Both issues fixed with the attached patch.

Looks good, please commit.

-- Brane
Received on 2016-05-16 14:15:12 CEST

This is an archived mail posted to the Subversion Dev mailing list.