mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Roland Hieber <rhi@pengutronix.de>
To: ptxdist@pengutronix.de
Subject: Re: [ptxdist] [PATCH 4/6] ppp: use HTTPS URL
Date: Fri, 9 Jul 2021 12:50:21 +0200	[thread overview]
Message-ID: <20210709105021.56iwmukpyghnkuyg@pengutronix.de> (raw)
In-Reply-To: <20210709065907.musnv6a4n2w6lhga@falbala.internal.home.lespocky.de>

On Fri, Jul 09, 2021 at 08:59:08AM +0200, Alexander Dahl wrote:
> Hello Roland,
> 
> On Fri, Jul 09, 2021 at 12:02:02AM +0200, Roland Hieber wrote:
> > Signed-off-by: Roland Hieber <rhi@pengutronix.de>
> 
> I miss the reasoning for this change. What's the benefit?

Originally I tried to "fix" it because the nightly urlcheck run failed,
but it turned out to be a temporary failure on the samba.org side.
Nevertheless, HTTPS provides more security, and the HTTP URL redirects
to HTTPS anyway (same for the other patches in this series).

 - Roland

> 
> Greets
> Alex
> 
> > ---
> >  rules/ppp.make | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/rules/ppp.make b/rules/ppp.make
> > index 3c8a57fd9396..e39b87b45dd3 100644
> > --- a/rules/ppp.make
> > +++ b/rules/ppp.make
> > @@ -20,7 +20,7 @@ PPP_VERSION	:= 2.4.9
> >  PPP_MD5		:= c88153ae3d16ae114152cd3c15c7301d
> >  PPP		:= ppp-$(PPP_VERSION)
> >  PPP_SUFFIX	:= tar.gz
> > -PPP_URL		:= http://ftp.samba.org/pub/ppp/$(PPP).$(PPP_SUFFIX)
> > +PPP_URL		:= https://www.samba.org/ftp/pub/ppp/$(PPP).$(PPP_SUFFIX)
> >  PPP_SOURCE	:= $(SRCDIR)/$(PPP).$(PPP_SUFFIX)
> >  PPP_DIR		:= $(BUILDDIR)/$(PPP)
> >  PPP_LICENSE	:= BSD AND GPL-2.0-only
> > -- 
> > 2.30.2

-- 
Roland Hieber, Pengutronix e.K.          | r.hieber@pengutronix.de     |
Steuerwalder Str. 21                     | https://www.pengutronix.de/ |
31137 Hildesheim, Germany                | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686         | Fax:   +49-5121-206917-5555 |

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de


  reply	other threads:[~2021-07-09 10:50 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-07-08 22:01 [ptxdist] [PATCH 1/6] dbench: " Roland Hieber
2021-07-08 22:02 ` [ptxdist] [PATCH 2/6] linuxptp: version bump 2.0 -> 3.1.1 Roland Hieber
2021-07-08 22:11   ` [ptxdist] [PATCH v2] " Roland Hieber
2021-07-20 11:48     ` [ptxdist] [APPLIED] " Michael Olbrich
2021-07-08 22:02 ` [ptxdist] [PATCH 3/6] linuxptp: pin down license files Roland Hieber
2021-07-20 11:48   ` [ptxdist] [APPLIED] " Michael Olbrich
2021-07-08 22:02 ` [ptxdist] [PATCH 4/6] ppp: use HTTPS URL Roland Hieber
2021-07-09  6:59   ` Alexander Dahl
2021-07-09 10:50     ` Roland Hieber [this message]
2021-07-09 11:12       ` Alexander Dahl
2021-07-20 11:48   ` [ptxdist] [APPLIED] " Michael Olbrich
2021-07-08 22:02 ` [ptxdist] [PATCH 5/6] rsync: " Roland Hieber
2021-07-20 11:48   ` [ptxdist] [APPLIED] " Michael Olbrich
2021-07-08 22:02 ` [ptxdist] [PATCH 6/6] samba: use HTTPS URL in menu help text Roland Hieber
2021-07-20 11:48   ` [ptxdist] [APPLIED] " Michael Olbrich
2021-07-20 11:48 ` [ptxdist] [APPLIED] dbench: use HTTPS URL Michael Olbrich

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210709105021.56iwmukpyghnkuyg@pengutronix.de \
    --to=rhi@pengutronix.de \
    --cc=ptxdist@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox