Bug 201862 - [maintainer] databases/pgpool-II-33: version bump 3.3.3 -> 3.3.7
Summary: [maintainer] databases/pgpool-II-33: version bump 3.3.3 -> 3.3.7
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Jason Unovitch
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-07-24 20:22 UTC by Michelle Sullivan
Modified: 2015-08-12 03:15 UTC (History)
1 user (show)

See Also:
michelle: maintainer-feedback+


Attachments
diff for version bump. (985 bytes, text/plain)
2015-07-24 20:22 UTC, Michelle Sullivan
michelle: maintainer-approval+
Details
Updated patch (1.07 KB, patch)
2015-08-11 00:52 UTC, Jason Unovitch
no flags Details | Diff
Poudriere testport log from 10.1-RELEASE jail (90.19 KB, text/x-log)
2015-08-11 00:53 UTC, Jason Unovitch
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Michelle Sullivan 2015-07-24 20:22:07 UTC
Created attachment 159198 [details]
diff for version bump.

Version bump 3.3.3 -> 3.3.7
Comment 1 Michelle Sullivan 2015-07-26 12:01:24 UTC
====>> Building 1 packages using 1 builders
====>> Starting/Cloning builders
====>> Hit CTRL+t at any time to see build progress and stats
====>> [01] Starting build of databases/pgpool-II-33
====>> [01] Status for build databases/pgpool-II-33: check-sanity
====>> [01] Status for build databases/pgpool-II-33: pkg-depends
====>> [01] Status for build databases/pgpool-II-33: fetch-depends
====>> [01] Status for build databases/pgpool-II-33: fetch
====>> [01] Status for build databases/pgpool-II-33: checksum
====>> [01] Status for build databases/pgpool-II-33: distfiles
====>> [01] Status for build databases/pgpool-II-33: extract-depends
====>> [01] Status for build databases/pgpool-II-33: extract
====>> [01] Status for build databases/pgpool-II-33: patch-depends
====>> [01] Status for build databases/pgpool-II-33: patch
====>> [01] Status for build databases/pgpool-II-33: build-depends
====>> [01] Status for build databases/pgpool-II-33: lib-depends
====>> [01] Status for build databases/pgpool-II-33: configure
====>> [01] Status for build databases/pgpool-II-33: build
====>> [01] Status for build databases/pgpool-II-33: run-depends
====>> [01] Status for build databases/pgpool-II-33: stage
====>> [01] Status for build databases/pgpool-II-33: package
====>> [01] Status for build databases/pgpool-II-33: install-mtree
====>> [01] Status for build databases/pgpool-II-33: install
====>> [01] Finished build of databases/pgpool-II-33: Success
====>> Completed: 1 of 1
====>> Stopping 1 builders
Comment 2 Jason Unovitch freebsd_committer freebsd_triage 2015-08-11 00:52:08 UTC
Created attachment 159748 [details]
Updated patch

Hello,
I had a minor issue with the patch in that "PORTVERSION= 3.3.7" was already set in the patch.  I am thinking you have have a later version in your own internal repo.  This patch is against the current SVN master and also resolves a minor `portlint -ac` nit for consistency with USES options.

WARN: Makefile: [15]: the options to USES are not sorted.  Please consider sorting them.
Comment 3 Jason Unovitch freebsd_committer freebsd_triage 2015-08-11 00:53:46 UTC
Created attachment 159749 [details]
Poudriere testport log from 10.1-RELEASE jail

I tested the patch with pgsql=9.2, pgsql=9.3 (attached), and pgsql=9.4 in my make.conf on 9.3-RELEASE, 10.1-RELEASE, 10.2-RC2, and CURRENT and didn't see any issues.
Comment 4 Michelle Sullivan 2015-08-11 07:58:17 UTC
Odd you should say that:

[michelle@colossus /Storage/source/ports/head/databases]$ portlint -ac pgpool-II-33/
WARN: Makefile: [6]: whitespace before end of line.
WARN: /Storage/source/ports/head/databases/pgpool-II-33/files/patch-pool_hba.c: patch was not generated using ``make makepatch''.  It is recommended to use ``make makepatch'' to ensure proper patch format.
WARN: /Storage/source/ports/head/databases/pgpool-II-33/files/patch-recovery.c: patch was not generated using ``make makepatch''.  It is recommended to use ``make makepatch'' to ensure proper patch format.
WARN: /Storage/source/ports/head/databases/pgpool-II-33/files/patch-watchdog__watchdog.h: patch was not generated using ``make makepatch''.  It is recommended to use ``make makepatch'' to ensure proper patch format.
0 fatal errors and 4 warnings found.

Nothing mentioned about USES...
Comment 5 Jason Unovitch freebsd_committer freebsd_triage 2015-08-11 22:15:12 UTC
(In reply to Michelle Sullivan from comment #4)
The most recent portlint-2.16.6 does this.  It also clarifies only using make makepatch when changing patches to minimize repo churn when the patches haven't changed.

Are you ok with the patch?
Comment 6 Michelle Sullivan 2015-08-11 22:34:57 UTC
Sorry missed you had uploaded a patch (my excuse: I've been having problems with my desktop panicing)

Added maintainer approved flag (as I do approve)
Comment 7 commit-hook freebsd_committer freebsd_triage 2015-08-12 03:11:58 UTC
A commit references this bug:

Author: junovitch
Date: Wed Aug 12 03:11:39 UTC 2015
New revision: 394003
URL: https://svnweb.freebsd.org/changeset/ports/394003

Log:
  databases/pgpool-II-33: update 3.3.3_1 -> 3.3.7

  PR:		201862
  Submitted by:	Michelle Sullivan <michelle@sorbs.net> (maintainer)
  Approved by:	pgollucci (mentor)

Changes:
  head/databases/pgpool-II-33/Makefile
  head/databases/pgpool-II-33/distinfo
Comment 8 Jason Unovitch freebsd_committer freebsd_triage 2015-08-12 03:15:33 UTC
Committed and closing.  Thanks for your patch submission.