Page MenuHomeFreeBSD
Differential D11888

databases/pgrouting: Update to 2.4.1
ClosedPublic
Actions

Authored by ultima on Aug 5 2017, 10:29 PM.
Tags
None
Referenced Files
F137789728: D11888.id31726.diff
Tue, Nov 25, 9:09 PM
F137752392: D11888.diff
Tue, Nov 25, 12:18 PM
F137752388: D11888.diff
Tue, Nov 25, 12:18 PM
Unknown Object (File)
Fri, Nov 21, 5:03 PM
Unknown Object (File)
Wed, Nov 19, 8:00 AM
Unknown Object (File)
Fri, Nov 14, 7:59 PM
Unknown Object (File)
Thu, Nov 13, 10:02 AM
Unknown Object (File)
Tue, Nov 11, 1:27 AM
View All Files
Subscribers
mat

Details

Reviewers
lifanov
matthew
Commits
rP447552: * Updated to 2.4.1
Summary
  • Updated to 2.4.1

PR\: 221092
Submitted by\: lbartoletti <lbartoletti@tuxfamily.org> (maintainer)
Reviewed by\: lifanov (mentor), matthew (mentor)
Approved by\: lifanov (mentor), matthew (mentor)
Differential Revision\: https://reviews.freebsd.org/DXXXXX

Test Plan

portlint:
WARN: Makefile: for new port, make $FreeBSD$ tag in comment section empty, to make SVN happy.
0 fatal errors and 1 warning found.

poudriere:
103i386
103amd64
110i386
110amd64
12i386
12amd64

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 10932
Build 11320: arc lint + arc unit

Event Timeline

ultima created this revision.Aug 5 2017, 10:29 PM
Herald added a subscriber: mat. · View Herald TranscriptAug 5 2017, 10:29 PM
Harbormaster completed remote builds in B10879: Diff 31631.Aug 5 2017, 10:29 PM
mat added a comment.Aug 7 2017, 12:13 PM

Instead of adding a patch, why not use USES=cmake:outsource ?

matthew requested changes to this revision.Aug 7 2017, 8:31 PM

Yeah. USES=cmake:outsource rather than adding a patch is the way to go here. Aside from that, looks fine.

This revision now requires changes to proceed.Aug 7 2017, 8:31 PM
ultima updated this revision to Diff 31726.Aug 7 2017, 10:06 PM
ultima edited edge metadata.

Removed patch and added cmake:outsource

Harbormaster completed remote builds in B10932: Diff 31726.Aug 7 2017, 10:06 PM
ultima added a comment.Aug 7 2017, 10:07 PM

Yeah, I didn't know about that added feature to cmake.

matthew accepted this revision.Aug 8 2017, 2:25 PM

lgtm

This revision is now accepted and ready to land.Aug 8 2017, 2:25 PM
Closed by commit rP447552: * Updated to 2.4.1 (authored by ultima). · Explain WhyAug 8 2017, 5:55 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
databases/
pgrouting/
5 lines
6 lines
26 lines

Diff 31726

View Options

databases/pgrouting/Makefile

Loading...
View Options

databases/pgrouting/distinfo

Loading...
View Options

databases/pgrouting/pkg-plist

Loading...