Created attachment 198356 [details] Patch sysutils/google-compute-engine-oslogin to version 1.3.1 Update to the latest upstream version (aka 1.3.1) to make new features available for GCE cloud images. The file files/patch-Makefile was removed since this modification was applied by upstream in this latest version.
Created attachment 198357 [details] Poudriere logs
Created attachment 198361 [details] Patch sysutils/google-compute-engine-oslogin to version 1.3.1 Updating patch.
Comment on attachment 198361 [details] Patch sysutils/google-compute-engine-oslogin to version 1.3.1 Thanks for the patch!
Created attachment 198766 [details] Updated patch sysutils/google-compute-engine-oslogin to version 1.3.1 Remove PORTREVISION.
portlint -C reports: WARN: /usr/local/poudriere/ports/default/sysutils/google-compute-engine-oslogin/pkg-descr: includes lines that exceed 80 characters. WARN: Makefile: extra item placed in the USES/USE_x section, for example, "MAKE_ARGS". WARN: /usr/local/poudriere/ports/default/sysutils/google-compute-engine-oslogin/files/patch-utils_oslogin__utils.cc: patch was not generated using ``make makepatch''. It is recommended to use ``make makepatch'' when you need to [re-]generate a patch to ensure proper patch format. 0 fatal errors and 3 warnings found.
Created attachment 199181 [details] Fix portlint warnings Most of the warnings reported by portlint were fixed: $ portlint -C WARN: /usr/home/kanashiro/google-compute-engine-oslogin/pkg-descr: includes lines that exceed 80 characters. 0 fatal errors and 1 warning found. Only the above one was not fixed because it is an URL and does not make sense break it to satisfy portlint.
A commit references this bug: Author: swills Date: Fri Nov 23 19:43:46 UTC 2018 New revision: 485700 URL: https://svnweb.freebsd.org/changeset/ports/485700 Log: sysutils/google-compute-engine-oslogin: update to 1.3.1 PR: 232440 Submitted by: Lucas Kanashiro <lucas.kanashiro@collabora.com> Approved by: Helen Koike <helen.koike@collabora.com> (maintainer, earlier version of patch) Changes: head/sysutils/google-compute-engine-oslogin/Makefile head/sysutils/google-compute-engine-oslogin/distinfo head/sysutils/google-compute-engine-oslogin/files/patch-Makefile head/sysutils/google-compute-engine-oslogin/files/patch-bin_google__oslogin__control head/sysutils/google-compute-engine-oslogin/files/patch-libnss__cache__oslogin_nss__cache__oslogin.c head/sysutils/google-compute-engine-oslogin/files/patch-nss__cache_nss__cache.cc head/sysutils/google-compute-engine-oslogin/files/patch-nss__module_nss__oslogin.cc head/sysutils/google-compute-engine-oslogin/files/patch-pam__module_pam__oslogin__admin.cc head/sysutils/google-compute-engine-oslogin/files/patch-pam__module_pam__oslogin__login.cc head/sysutils/google-compute-engine-oslogin/files/patch-utils_oslogin__utils.cc
(In reply to Lucas Kanashiro from comment #6) Thanks, committed. Made a slight change so that the patch didn't have "/usr/local/usr/local" in it and also so that changing PREFIX works. Sorry it took me so long to get back to this.