Re: [PATCH] Update FAQs which are out of date on the details of setup UI

[Jon Turney <jon.turney@dronecode.org.uk>]

On 23/07/2022 21:46, Brian Inglis wrote:
> On 2022-07-11 01:45, Corinna Vinschen wrote:
>> On Jul  7 12:44, Jon Turney wrote:
>>> ---
>>>   winsup/doc/faq-setup.xml | 11 ++++++-----
>>>   winsup/doc/faq-using.xml | 14 +++++++-------
>>>   2 files changed, 13 insertions(+), 12 deletions(-)
>> LGTM
> 
> [original did not make it to me; caught up on archive and noticed]
> 
> URL duplicates .html:
> 
>      <ulink url="https://cygwin.com/package-server.html.html">
> 
> should perhaps also have the self-closing tag delimiter "/>":
> 
>      <ulink url="https://cygwin.com/package-server.html" />
> 
> where the extra space ensures it is also valid XHTML/XML so it can be 
> checked or processed with better tools that can catch issues ;^>
> 
> [attachment extract]
> 
> diff --git a/winsup/doc/faq-setup.xml b/winsup/doc/faq-setup.xml
> index ce1069616..da9fce534 100644
> --- a/winsup/doc/faq-setup.xml
> +++ b/winsup/doc/faq-setup.xml
> ...
> @@ -688,7 +689,7 @@ files, reinstall the "<literal>cygwin</literal>" 
> package using the Cygwin Setup
>   this purpose.  See <ulink url="http://rsync.samba.org/"/>,
>   <ulink url="http://www.gnu.org/software/wget/"/> for utilities that 
> can do this for you.
>   For more information on setting up a custom Cygwin package server, see
> -the <ulink url="https://sourceware.org/cygwin-apps/setup.html">Cygwin 
> Setup program page</ulink>.
> +the <ulink url="https://cygwin.com/package-server.html.html">Cygwin 
> Package Server page</ulink>.

I am confused.

The <ulink> tag is closed by the </ulink> tag later on the same line, 
after enclosing the link text 'Cygwin Package Server page'

> 
>   </para>
>   </answer></qandaentry>
> ...