Page MenuHomeFreeBSD

New port: www/go-appengine-sdk: App Engine SDK for Go
ClosedPublic

Authored by yuri on Feb 11 2018, 3:12 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Mar 1, 8:29 PM
Unknown Object (File)
Thu, Feb 29, 2:04 PM
Unknown Object (File)
Feb 5 2024, 12:30 AM
Unknown Object (File)
Jan 9 2024, 6:54 AM
Unknown Object (File)
Dec 23 2023, 3:25 AM
Unknown Object (File)
Dec 22 2023, 10:19 AM
Unknown Object (File)
Dec 21 2023, 11:45 PM
Unknown Object (File)
Nov 14 2023, 1:35 PM

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 14958
Build 15070: arc lint + arc unit

Event Timeline

www/go-appengine-sdk/Makefile
44–50
GOARCH=  ${ARCH:S/i386/386}
52–54

GO387_VARS= GO386=387

www/go-appengine-sdk/Makefile
44–50

Thanks. I just realised that there's no need for IGNORE because of ONLY_FOR_ARCHS above.

Simplify GOARCH and GO386 options.

dmgk marked 2 inline comments as done.Feb 12 2018, 3:34 PM
sunpoet added inline comments.
www/go-appengine-sdk/Makefile
8

Use DISTNAME and USES=zip.

13–14

Please update LICENSE and add LICENSE_FILE.

Since you did not install the whole zip file, you do not need to list all licenses mentioned in ${WRKSRC}/LICENSE.
Please set LICENSE to the licenses of installed items.

www/go-appengine-sdk/Makefile
74

I am not sure what is being CP'ed here, but never use cp to install files, use COPYTREE_SHARE/BIN or one of the INSTALL_* macros.

dmgk marked 2 inline comments as done.

Address review feedback.

www/go-appengine-sdk/Makefile
13–14

We do install the while zip file (apart from a few Linux binary artifacts and PHP stuff that doesn't have a separate license) so all licenses listed in LICENSE_FILE apply.

74

This ports rebuilds two Go distributions (1.6 and 1.8) included in the upstream zip to get native FreeBSD binaries and installs resulting GOROOTs with ${CP}, I stole this CP line from lang/go. While it probably would be possible to get the same result with a combination of COPYTREE_SHARE/BIN, it would make do-install much more complex because there's a bunch of executables and .pl/.sh scripts scattered through GOROOT subfolders.

www/go-appengine-sdk/Makefile
21

Maybe we need something like LICENSE_FILE_combo for cases when all licenses are lumped together in a single file.

yuri added a reviewer: dmgk.
yuri retitled this revision from www/go-appengine-sdk: App Engine SDK for Go to New port: www/go-appengine-sdk: App Engine SDK for Go.Feb 17 2018, 3:42 AM
yuri edited reviewers, added: tcberner, adamw; removed: dmgk.
dmgk added inline comments.
www/go-appengine-sdk/Makefile
83

That would remove executable bit from all binaries, no?

From my side it looks good.

This revision is now accepted and ready to land.Feb 17 2018, 9:54 PM
This revision was automatically updated to reflect the committed changes.
head/www/go-appengine-sdk/Makefile
12–20 ↗(On Diff #39446)

(1)
For me, I would not set LICENSE_FILE for a license without full text in the file.

Though 7 licenses are mentioned in ${WRKSRC}/LICENSE, it only has full text of 2 licenses (BSD3CLAUSE and MIT).

That means I would only set LICNESE_FILE_BSD3CLAUSE and LICENSE_FILE_MIT.

If you think ${WRKSRC}/LICENSE contains 7 licenses, then LICENSE_FILE=${WRKSRC}/LICENSE is fine (support added in r451432).

(2)
Regarding GPLv2, LGPL21 and MPL11, I guess you add them because of cacerts.
GPLv2 and LGPL21 should be change to GPLv2+ and LGPL21+.
The correct license for this port is "APACHE20 and BSD3CLAUSE and MIT and PSFL and (GPLv2+ or LGPL21+ or MPL11)".
Since our LICENSE framework does not support the case of "A and B and C and D and (E or F or G)", it will be better to add a comment.

from LICENSE:

cacerts

Version: MPL 1.1/GPL 2.0/LGPL 2.1
<snip>
Alternatively, the contents of this file may be used under the terms of
either the GNU General Public License Version 2 or later (the "GPL"), or
the GNU Lesser General Public License Version 2.1 or later (the "LGPL"),

70 ↗(On Diff #39446)

Use ${PYTHON_CMD} instead of "{LOCALBASE}/bin/python2.7".

head/www/go-appengine-sdk/Makefile
12–20 ↗(On Diff #39446)

Sunpoet,

Thank you for this information.
I will update the port.

Yuri