X-Recipient: archive-cygwin@delorie.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@cygwin.com; run by ezmlm
List-Id: <cygwin.cygwin.com>
List-Subscribe: <mailto:cygwin-subscribe@cygwin.com>
List-Archive: <http://sourceware.org/ml/cygwin/>
List-Post: <mailto:cygwin@cygwin.com>
List-Help: <mailto:cygwin-help@cygwin.com>, <http://sourceware.org/ml/#faqs>
Sender: cygwin-owner@cygwin.com
Mail-Followup-To: cygwin@cygwin.com
Delivered-To: mailing list cygwin@cygwin.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@cygwin.com>
To: cygwin@cygwin.com
Subject: Re: Proposed patch for web site: update most links to HTTPS
Message-ID: <20160523103519.GA30229@calimero.vinschen.de>
Reply-To: cygwin@cygwin.com
Mail-Followup-To: cygwin@cygwin.com
References: <BL2PR03MB228A4FE7B2CCA51E5FF5163DF610@BL2PR03MB228.namprd03.prod.outlook.com> <20160520163624.GA22065@calimero.vinschen.de> <BL2PR03MB2289CA95EC0CCF0D42AB961DF4D0@BL2PR03MB228.namprd03.prod.outlook.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;	protocol="application/pgp-signature"; boundary="NzB8fVQJ5HfG6fxh"
Content-Disposition: inline
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@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--