Page MenuHomeFreeBSD
Differential D21735

www/gohugo: Update for D21730
ClosedPublic
Actions

Authored by dmgk on Sep 21 2019, 12:22 PM.
Tags
None
Referenced Files
Unknown Object (File)
Oct 4 2024, 2:13 PM
Unknown Object (File)
Oct 3 2024, 8:02 AM
Unknown Object (File)
Sep 27 2024, 10:19 PM
Unknown Object (File)
Sep 20 2024, 10:23 AM
Unknown Object (File)
Sep 7 2024, 11:56 PM
Unknown Object (File)
Sep 7 2024, 11:15 AM
Unknown Object (File)
Sep 7 2024, 6:33 AM
Unknown Object (File)
Sep 7 2024, 4:52 AM
View All 74 Files
Subscribers
mat

Details

Reviewers
tz
araujo
tobik
Commits
rP512675: Include bsd.port.pre.mk to get DISTINFO_FILE in scope for the shell command.
Summary

www/gohugo: Update for D21730

Include bsd.port.pre.mk to get DISTINFO_FILE in scope for the shell command.

Test Plan

poudriere testport: OK (120a)

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dmgk created this revision.Sep 21 2019, 12:22 PM
Herald added a subscriber: mat. · View Herald TranscriptSep 21 2019, 12:22 PM
Harbormaster completed remote builds in B26596: Diff 62374.Sep 21 2019, 12:22 PM
dmgk mentioned this in D21730: Mk/Uses/go.mk: Omit symbol table and debug information by default.Sep 21 2019, 12:28 PM
tobik accepted this revision.Sep 23 2019, 5:03 AM
tobik added inline comments.
www/gohugo/Makefile
129–131 ↗(On Diff #62374)

I think this could use a short comment in the Makefile as to why it is after bsd.port.pre.mk.

I am not a big fan of grabbing TIMESTAMP from distinfo for this. I think the port should just set BuildDate to some static date that never changes. Recording the build date does not seem very important for the port and just makes it more complicated.

This revision is now accepted and ready to land.Sep 23 2019, 5:03 AM
dmgk updated this revision to Diff 62462.Sep 23 2019, 1:11 PM

Add Makefile comment on why bsd.port.pre.mk is needed.

This revision now requires review to proceed.Sep 23 2019, 1:11 PM
Harbormaster completed remote builds in B26646: Diff 62462.Sep 23 2019, 1:11 PM
dmgk marked an inline comment as done.Sep 23 2019, 1:11 PM
dmgk updated this revision to Diff 62464.Sep 23 2019, 1:13 PM

Shorten.

Harbormaster completed remote builds in B26647: Diff 62464.Sep 23 2019, 1:13 PM
tobik accepted this revision.Sep 23 2019, 4:01 PM
This revision is now accepted and ready to land.Sep 23 2019, 4:01 PM
araujo accepted this revision.Sep 23 2019, 7:58 PM
Closed by commit rP512675: Include bsd.port.pre.mk to get DISTINFO_FILE in scope for the shell command. (authored by dmgk). · Explain WhySep 23 2019, 8:42 PM
This revision was automatically updated to reflect the committed changes.
dmgk added a commit: rP512675: Include bsd.port.pre.mk to get DISTINFO_FILE in scope for the shell command..

Revision Contents
Changeset List

PathSize
head/
www/
gohugo/
11 lines

Diff 62485

View Options

head/www/gohugo/Makefile

Loading...