Bug 203766 - [MAINTAINER] x11-servers/Xfstt: Fix gettext-related build failure on 9.X-RELEASE (pkg-fallout)
Summary: [MAINTAINER] x11-servers/Xfstt: Fix gettext-related build failure on 9.X-RELE...
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Ports Security Team
URL:
Keywords: easy, needs-qa, patch, regression
Depends on:
Blocks:
 
Reported: 2015-10-14 09:28 UTC by A.J. "Fonz" van Werven
Modified: 2018-02-18 11:44 UTC (History)
3 users (show)

See Also:
freebsd: maintainer-feedback+
freebsd: merge-quarterly?


Attachments
New patchfile, to be added to PORTDIR/files/ (308 bytes, patch)
2015-10-14 09:28 UTC, A.J. "Fonz" van Werven
freebsd: maintainer-approval+
Details | Diff
Patch for entire port directory (579 bytes, patch)
2015-10-14 09:31 UTC, A.J. "Fonz" van Werven
freebsd: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description A.J. "Fonz" van Werven 2015-10-14 09:28:11 UTC
Created attachment 162024 [details]
New patchfile, to be added to PORTDIR/files/

This is one of the ports affected by the gettext-related issue that breaks builds on 9.X-RELEASE (but not 10.X), as reported by pkg-fallout.

The attached patchfile (generated with "make makepatch") can be added to the port's files/ directory. If a patch for the entire port directory is preferred (i.e. "cd /usr/ports/x11-servers && diff -ruN Xfstt.orig Xfstt") I can submit that as well.

As far as I can tell, this fix doesn't require PORTREVISION to be set/bumped.
Comment 1 A.J. "Fonz" van Werven 2015-10-14 09:31:44 UTC
Created attachment 162025 [details]
Patch for entire port directory

This patch can be applied against the ports directory, in case that's easier than adding the previous file.

Note that this patch doesn't obsolete the previous patchfile: EITHER add the previous patchfile to the files/ directory OR apply this patch to the port's directory; whichever is easier.
Comment 2 Kubilay Kocak freebsd_committer freebsd_triage 2015-10-14 10:39:34 UTC
@AJ, for future references, single (unified svn diff) patch that is maintainer approved is preferred, with all others (previous, outdated, unapproved, etc) obsoleted

Failing the obsoletion, committers will (or should) normally choose the latest maintainer-approval (+) attachment.

QA results (portlint/poudriere) are still needed for this issue
Comment 3 A.J. "Fonz" van Werven 2015-10-14 17:38:25 UTC
Sorry I forgot the logs!

Build logs ("make stage && make check-plist && make stage-qa && make package") for...
9.X i386: https://www.skysmurf.nl/~fonz/attic/Xfstt_9X_i386.log
(So that's fixed.)
10.X amd64: https://www.skysmurf.nl/~fonz/attic/Xfstt_10X_amd64.log
(So 10.X remains unaffected.)

Portlint output:
# env DEVELOPER=yes portlint -A .
WARN: Makefile: for new port, make $FreeBSD$ tag in comment section empty, to make SVN happy.
0 fatal errors and 1 warning found.

@Koobs: So if I understand correctly the second attachment is the right way?
Comment 4 A.J. "Fonz" van Werven 2015-10-14 17:41:12 UTC
Also: if you don't trust the self-signed certificate use http://... instead of https://...
Comment 5 Mathieu Arnold freebsd_committer freebsd_triage 2015-10-16 11:59:15 UTC
Thanks for the patch. As a side note, you don't need to provide poudriere logs, saying "builds fine on 9 and 10" is enough. (same for portlint)
Comment 6 commit-hook freebsd_committer freebsd_triage 2015-10-16 14:19:14 UTC
A commit references this bug:

Author: mat
Date: Fri Oct 16 14:19:08 UTC 2015
New revision: 399478
URL: https://svnweb.freebsd.org/changeset/ports/399478

Log:
  Fix build on 9.

  PR:		203766
  Submitted by:	maintainer
  Sponsored by:	Absolight

Changes:
  head/x11-servers/Xfstt/files/patch-po_Makefile.in.in
Comment 7 A.J. "Fonz" van Werven 2015-10-25 18:10:42 UTC
Can this please be merged into -quarterly?
Comment 8 Kubilay Kocak freebsd_committer freebsd_triage 2015-10-26 00:59:51 UTC
Re-open for MFH
Comment 9 Walter Schwarzenfeld freebsd_triage 2018-01-12 23:27:36 UTC
Forgotten to close?