Paths

Table of Contentst

Differential D24397

lang/go-devel: Convert to slave of lang/go, apply D24122
ClosedPublic
Actions

Authored by dmgk on Apr 13 2020, 3:13 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Mar 31, 9:52 AM

	Unknown Object (File)
	Thu, Mar 20, 2:55 PM

	Unknown Object (File)
	Fri, Mar 7, 8:24 AM

	Unknown Object (File)
	Mar 1 2025, 8:11 AM

	Unknown Object (File)
	Feb 27 2025, 6:45 PM

	Unknown Object (File)
	Feb 24 2025, 5:57 PM

	Unknown Object (File)
	Feb 5 2025, 8:43 PM

	Unknown Object (File)
	Jan 18 2025, 10:58 PM

Subscribers

Details

Reviewers

mikael
tobik

Commits

rP531626: lang/go-devel: Convert to a slave of lang/go, apply D24122

Summary

- Convert to slave port of lang/go
- Apply D24122 (similar to r530387 for lang/go) to relax module consistency
  checks in go1.14 mode

Test Plan

poudriere testport: OK for both lang/go and lang/go-devel

Diff Detail

Repository

rP FreeBSD ports repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dmgk created this revision.Apr 13 2020, 3:13 PM

Herald added a subscriber: mat. · View Herald TranscriptApr 13 2020, 3:13 PM

Harbormaster completed remote builds in B30456: Diff 70494.Apr 13 2020, 3:13 PM

tobik added inline comments.Apr 13 2020, 3:56 PM

lang/go/Makefile
39–40 ↗	(On Diff #70494)	Unrelated to this review, but these are now flagged by portscan/portclippy. Can you drop them? Without setting `SHEBANG_LANG=sh` they are unused.
42 ↗	(On Diff #70494)	Overwrites the one from go-devel. Use `?=`.

remove sh*CMD
do not overwrite CONFLICTS_INSTALL

Harbormaster completed remote builds in B30462: Diff 70500.Apr 13 2020, 4:14 PM

Thanks. One more nit but otherwise LGTM.

lang/go/Makefile
16 ↗	(On Diff #70500)	Oh, COMMENT also needs `?=`.

This revision is now accepted and ready to land.Apr 13 2020, 4:19 PM

also leave COMMENT alone

This revision now requires review to proceed.Apr 13 2020, 4:28 PM

Harbormaster completed remote builds in B30463: Diff 70502.Apr 13 2020, 4:28 PM

dmgk marked an inline comment as done.Apr 13 2020, 4:29 PM

dmgk added inline comments.

lang/go/Makefile
16 ↗	(On Diff #70500)	Thanks for catching these.

dmgk marked an inline comment as done.Apr 13 2020, 4:31 PM

mikael accepted this revision.Apr 13 2020, 6:53 PM

This revision is now accepted and ready to land.Apr 13 2020, 6:53 PM

Closed by commit rP531626: lang/go-devel: Convert to a slave of lang/go, apply D24122 (authored by dmgk). · Explain WhyApr 13 2020, 7:16 PM

This revision was automatically updated to reflect the committed changes.

dmgk added a commit: rP531626: lang/go-devel: Convert to a slave of lang/go, apply D24122.

Revision Contents
Changeset List

Path

Size

head/

lang/

go-devel/

119 lines

2 lines

files/

patch-src_cmd_go_internal_modload_init.go

12 lines

6 lines

go/

17 lines

Diff 70509

head/lang/go-devel/Makefile

Loading...

head/lang/go-devel/distinfo

Loading...

head/lang/go-devel/files/patch-src_cmd_go_internal_modload_init.go

Loading...

head/lang/go-devel/pkg-descr

Loading...

head/lang/go/Makefile

Loading...