Bug 209864

Summary: sysutils/3dm includes build hostname in pkg-message, makes package nonreproducible
Product: Ports & Packages Reporter: Ed Maste <emaste>
Component: Individual Port(s)Assignee: Kurt Jaeger <pi>
Status: Closed FIXED    
Severity: Affects Only Me CC: pi, takefu
Priority: --- Flags: pi: maintainer-feedback+
Version: Latest   
Hardware: Any   
OS: Any   
URL: https://reviews.freebsd.org/D6836
Attachments:
Description Flags
3dm-2.11.00.021_3,1.patch takefu: maintainer-approval+

Description Ed Maste freebsd_committer freebsd_triage 2016-05-30 13:41:07 UTC
The sysutils/3dm package was nonreproducible in my recent test due (only) to differences in the pkg-message. One package reported "Visit https://current-master-job-06:888/" and the other "Visit https://current-master-job-22:888/". In the case of binary packages the build hostname is most likely not meaningful; this should probably be "Visit https://localhost:888/" or use example.org.
Comment 1 Mathieu Arnold freebsd_committer freebsd_triage 2016-05-30 14:20:23 UTC
Mmm, poudriere has a BUILDER_HOSTNAME that sets the hostname to the jail as being the same for all jails, it's what's used to fix other ports that do the same things, sysutils/puppet comes to mind (it puts the hostname in the config file.)
Comment 2 Ed Maste freebsd_committer freebsd_triage 2016-05-30 14:42:33 UTC
At least initially I don't want to set an option like that because I have a goal of determining the current state of package reproducibility.

I also believe it is better to address these at the source rather than coerce system inputs to make builds reproducible; for this case BUILDER_HOSTNAME would address the nonreproducibility, but the message still might not be useful for a user of the package.

3dm is not redistributable so this won't affect official FreeBSD packages but would for someone building their own package set.
Comment 3 Ed Maste freebsd_committer freebsd_triage 2016-06-13 18:20:56 UTC
Review at https://reviews.freebsd.org/D6836
Comment 4 takefu 2016-06-15 05:16:24 UTC
Created attachment 171452 [details]
3dm-2.11.00.021_3,1.patch

Will also no problem to change from the hostname to localhost.
Simply it has inherited from #130295,
Not necessarily that particular stick.

I'm sorry too slow.
Comment 5 commit-hook freebsd_committer freebsd_triage 2016-06-27 19:29:17 UTC
A commit references this bug:

Author: pi
Date: Mon Jun 27 19:29:07 UTC 2016
New revision: 417677
URL: https://svnweb.freebsd.org/changeset/ports/417677

Log:
  sysutils/3dm: fix hostname in pkg-message, make package reproducible

  PR:		209864
  Reported by:	emaste
  Submitted by:	takefu@airport.fm (maintainer)

Changes:
  head/sysutils/3dm/Makefile
  head/sysutils/3dm/distinfo
  head/sysutils/3dm/files/pkg-message.in
  head/sysutils/3dm/pkg-descr
Comment 6 Kurt Jaeger freebsd_committer freebsd_triage 2016-06-27 19:30:29 UTC
Committed, thanks