X-Recipient:	archive-cygwin AT delorie DOT com
DomainKey-Signature:	a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:date:from:to:subject:message-id:reply-to
	:references:mime-version:content-type:in-reply-to; q=dns; s=
	default; b=Z9RQAHYMPRwXOYqWHA/1xtwGxvOr3ZpObZ84BUnRVbDodtN1RM6Mz
	xW37CuuO4c1TNUf+rvdwu+6L8V8uD5fTtNWlHa2TIh3IRr9Y8Yt3CdQt5wxO/Ybk
	n19x6Yxy4lPug1Rtw98D1KVwJtmeL1cYczeRkVs2OaEXNEdHZzV2zI=
DKIM-Signature:	v=1; a=rsa-sha1; c=relaxed; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:date:from:to:subject:message-id:reply-to
	:references:mime-version:content-type:in-reply-to; s=default;
	bh=c+W9gNldI9iS5kyqSHh9gYXUg64=; b=j6IxAaiiEZuqAIAUxVElhnpf+QNR
	9NTeEk+yRKgwzPIebhY1crKdAk7VsEwAxiPQW2l0anCfzNe1eD9vjj4/e4F/+zee
	yb+R4iapSvnIwbCXRenOaYO3caJb4PF0Nt7WlwFlu9xRa1usVC5L+/XRUB5Ht1te
	M8lYHbczCGZ4KSA=
Mailing-List:	contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Id:	<cygwin.cygwin.com>
List-Subscribe:	<mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive:	<http://sourceware.org/ml/cygwin/>
List-Post:	<mailto:cygwin AT cygwin DOT com>
List-Help:	<mailto:cygwin-help AT cygwin DOT com>, <http://sourceware.org/ml/#faqs>
Sender:	cygwin-owner AT cygwin DOT com
Mail-Followup-To:	cygwin AT cygwin DOT com
Delivered-To:	mailing list cygwin AT cygwin DOT com
Authentication-Results:	sourceware.org; auth=none
X-Virus-Found:	No
X-Spam-SWARE-Status:	No, score=-96.5 required=5.0 tests=AWL,BAYES_00,GOOD_FROM_CORINNA_CYGWIN,KAM_LAZY_DOMAIN_SECURITY,RCVD_IN_PBL,RDNS_DYNAMIC autolearn=ham version=3.3.2 spammy=UD:cygwin-htdocs_bsclifton_https_20160521.patch, cygwin-htdocs_bsclifton_https_20160424, cygwinhtdocs_bsclifton_https_20160424, cygwin-htdocs_bsclifton_https_20160521.patch
X-HELO:	calimero.vinschen.de
Date:	Mon, 23 May 2016 12:35:19 +0200
From:	Corinna Vinschen <corinna-cygwin AT cygwin DOT com>
To:	cygwin AT cygwin DOT com
Subject:	Re: Proposed patch for web site: update most links to HTTPS
Message-ID:	<20160523103519.GA30229@calimero.vinschen.de>
Reply-To:	cygwin AT cygwin DOT com
Mail-Followup-To:	cygwin AT cygwin DOT com
References:	<BL2PR03MB228A4FE7B2CCA51E5FF5163DF610 AT BL2PR03MB228 DOT namprd03 DOT prod DOT outlook DOT com> <20160520163624 DOT GA22065 AT calimero DOT vinschen DOT de> <BL2PR03MB2289CA95EC0CCF0D42AB961DF4D0 AT BL2PR03MB228 DOT namprd03 DOT prod DOT outlook DOT com>
MIME-Version:	1.0
In-Reply-To:	<BL2PR03MB2289CA95EC0CCF0D42AB961DF4D0@BL2PR03MB228.namprd03.prod.outlook.com>
User-Agent:	Mutt/1.6.1 (2016-04-27)

--NzB8fVQJ5HfG6fxh
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On May 22 00:29, Brian Clifton wrote:
> >On Apr 24 17:18, Brian Clifton wrote:
> >> Hi folks,
> >>=20
> >> I have a proposed change for the web site. This patch (see below) will=
 update most of the urls to HTTPS. In many cases there was a redirect; for =
those I captured the new canonical address.
> >>=20
> >> (Per the https://cygwin.com/contrib.html, this is *not* a change to=20
> >> anything in the winsup directory, which is why I chose the=20
> >> cygwin AT cygwin distro)
> >>=20
> >> The patch can be found here:
> >> https://patch.clifton.io/cygwin/cygwin-htdocs_bsclifton_https_20160424
> >> .patch
> >
> >Thanks for the patch, but, here's a question:
> >
> >Since Cygwin.com redirects http requests to https anyway, all links to c=
ygwin.com (or, FWIW, sourceware.org) will end up as https requests anyway.
> >
> >Having said that, wouldn't it make sense then to avoid absolute links to=
 cygwin.com and rather convert them to relative links, i.e.:
> >
>   >-<a href=3D"http://cygwin.com/ml/cygwin/2003-03/msg02182.html">Inspire=
d</a> by
>   >+<a href=3D"/ml/cygwin/2003-03/msg02182.html">Inspired</a> by
> >
> >?
> >
> >
> >Thanks,
> >Corinna
>=20
> Good points Corinna; I updated the patch to use relative URLs (attached).=
 If the attachment does not work for some reason, it can also be downloaded=
 from here:
> https://patch.clifton.io/cygwin/cygwin-htdocs_bsclifton_https_20160521.pa=
tch

Thanks for doing that, but this patch can't apply.  Please note that the
cygwin-api, cygwin-ug-net, and faq subdirs are maintained as part of the
cygwin repo itself and are in XML, not in HTML.  Only the generic stuff
is maintained in the htdocs repo.

On second thought, using relative paths inside the Cygwin docs
(cygwin-api, cygwin-ug-net, and faq) is not such a bright idea, because
the docs should ideally work even if not on the cygwin.com website.
Sorry about that, but I didn't notice that your patch also changes
the Cygwin docs :}

Can you please split up the patch in two chunks?

The generic website patches should be based on the Website git repo at

  git://sourceware.org/git/cygwin-htdocs.git

the Cygwin docs patches on the Cygwin git repo at

  git://sourceware.org/git/newlib-cygwin.git.

The docs XML files are in the winsup/doc subdir.


Thanks,
Corinna

--=20
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat

--NzB8fVQJ5HfG6fxh
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iQIcBAEBCAAGBQJXQtznAAoJEPU2Bp2uRE+g0s8P/RKnwe6DY8GOXDq2ZkkcaB2u
r4775uhY/jzUZbPN+yOhf7wk4rtb0AARzduxoAbsmY7ozRrJFDYx22fSVV5l53Ew
0SV/yEvMuz+wBLzWxVxup/KRn3Al2bXlYACaIwt0xiWnFp3WEbHRUrNs3eqikecq
a3F8xxgA2WAZtC8yBEHqPOxbmkqvxtipnvNGthcknk7+xf3ZiHZxBLLwjklFl+v9
gF1HtZSc216U3MVIeFCtTamSQqrOYkWVioGn9TJgOf42L9p2/7OwfZwDekir7MqT
2wIrch0E38aGFSVmTpiH1cejJEI3eWLkyyuyiCO0mnW1vCTNTbZM9Gu5fBecwsM9
kQx2qmxPs5/oQYLOioHxWCgXp+38HzHkVlbnSFr2O6fD4l4ssthdgNTB5Fo4cfG5
s6Sn6dmxVQUJn+s+ZV+Zdfsxp/nPDDFBrg295yRrZ8qPndvWpqz81vE2QxPhvwKG
PV56RRLbiTQcYkGqwmd6kl4/DBc6v4Bxe6CZf6jL8njz+ShHB0pB6VPhE7djs8Yf
25d29YJKl3wA/v1X9GCEcVgm7RH6PX+P1aeQZRmRZaVAqALvbGLNl8DbEtiKJ9Q/
jE+Pmd37mvVlnCJ0VJo5AULGYeYPUA8zuMXSOjisWtwSoxvHJ8T7J/PRjXwNpbYW
PZNIylALuWay2knE1Yij
=yHS7
-----END PGP SIGNATURE-----

--NzB8fVQJ5HfG6fxh--

- Raw text -