Bug 233666 - use HTTPS in MASTER_SITES for several ports
Summary: use HTTPS in MASTER_SITES for several ports
Status: Closed Not Accepted
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: freebsd-ports-bugs (Nobody)
URL:
Keywords: easy
Depends on:
Blocks:
 
Reported: 2018-11-30 15:40 UTC by Peter Malcom
Modified: 2019-06-05 03:55 UTC (History)
6 users (show)

See Also:

Attachments
patch (2.74 KB, patch)
2018-11-30 15:40 UTC, Peter Malcom 		no flags Details | Diff
patch (3.18 KB, patch)
2018-12-01 01:43 UTC, Peter Malcom 		no flags Details | Diff
patch (3.18 KB, patch)
2018-12-07 20:31 UTC, Peter Malcom 		no flags Details | Diff
patch (2.15 KB, patch)
2018-12-11 19:31 UTC, Peter Malcom 		no flags Details | Diff
patch (3.50 KB, patch)
2019-01-07 15:54 UTC, Peter Malcom 		no flags Details | Diff
Show Obsolete (4) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Peter Malcom 2018-11-30 15:40:03 UTC 
Created attachment 199687 [details]
patch

This diff adds/replaces some mirrors in MASTER_SITES with secure HTTP versions. All ports were tested and still pass make checksum.
Comment 1 Peter Malcom 2018-12-01 01:43:33 UTC 
Created attachment 199700 [details]
patch

2nd version
Comment 2 Kubilay Kocak freebsd_committer freebsd_triage 2018-12-01 04:13:47 UTC 
CC FreeBSD committers that are MAINTAINERS of ports being changed
Comment 3 Peter Malcom 2018-12-07 20:06:29 UTC 
swills' latest changeg to devel/ncurses breaks this patch.
That section now needs to be reworked to the new version number.
Can someone please commit this trivial patch soon?
Comment 4 Peter Malcom 2018-12-07 20:31:08 UTC 
Created attachment 199939 [details]
patch

New one with ncurses fixed
Comment 5 Tatsuki Makino 2018-12-08 01:10:10 UTC 
Excuse me. By the way, do we need to add the following line when using https:?

FETCH_DEPENDS= ca_root_nss>=0:security/ca_root_nss
Comment 6 Peter Malcom 2018-12-11 19:31:19 UTC 
Created attachment 200049 [details]
patch

Another commit by swills broke this patch again.
Here is an updated one.
Comment 7 Peter Malcom 2019-01-07 15:54:26 UTC 
Created attachment 200867 [details]
patch

Updated patch, still waiting for commit.
Comment 8 Yuri Victorovich freebsd_committer freebsd_triage 2019-01-19 19:44:20 UTC 
Please create individual PRs so that their maintainers have a chance to reply.
Comment 9 Dmitry Marakasov freebsd_committer freebsd_triage 2019-04-30 13:19:53 UTC 
(In reply to Tatsuki Makino from comment #5)
> Excuse me. By the way, do we need to add the following line when using
> https:?
> 
> FETCH_DEPENDS= ca_root_nss>=0:security/ca_root_nss

No. Well behaving HTTPS sites provide the full chain of intermediate certificates.
Comment 10 Steve Wills freebsd_committer freebsd_triage 2019-05-26 20:10:07 UTC 
Sorry, but as Yuri said, these should be patches to the individual ports. Also, you should list both https and http, just in case.