Page MenuHomeFreeBSD
Differential D12146

devel/pear-SebastianBergmann_Git: Unbreak
ClosedPublic
Actions

Authored by joneum on Aug 28 2017, 6:40 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Aug 10, 8:25 PM
Unknown Object (File)
Mon, Aug 10, 4:49 PM
Unknown Object (File)
Sun, Aug 9, 8:25 AM
Unknown Object (File)
Sun, Aug 9, 5:14 AM
Unknown Object (File)
Sun, Aug 9, 4:15 AM
Unknown Object (File)
Sat, Aug 8, 6:31 PM
Unknown Object (File)
Sat, Aug 8, 2:27 PM
Unknown Object (File)
Sat, Aug 8, 2:26 PM
View All Files
Subscribers
mat
tcberner

Details

Reviewers
tz
rene
miwi
Commits
rP449806: - Unbreak this port
Summary
  • Unbreak this port
  • switch to GitHub

Approved by: xxx (mentor)
#Differential Revision: https://reviews.freebsd.org/Dxxxxx

Test Plan

~...devel/pear-SebastianBergmann_Git: portlint -AC [20:33:11]
WARN: Makefile: for new port, make $FreeBSD$ tag in comment section empty, to make SVN happy.
WARN: Makefile: new ports should not set PORTREVISION.
0 fatal errors and 2 warnings found.

poudriere testport:

10.3-amd64
10.3-i386
11.1-amd64
11.1-i386
12.0-CURRENT r321899 amd64
12.0-CURRENT r321913 i386

Diff Detail

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

Event Timeline

joneum created this revision.Aug 28 2017, 6:40 PM
Herald added a subscriber: mat. · View Herald TranscriptAug 28 2017, 6:40 PM
Harbormaster completed remote builds in B11268: Diff 32450.Aug 28 2017, 6:40 PM
joneum edited the summary of this revision. (Show Details)Aug 28 2017, 6:43 PM
tcberner added a subscriber: tcberner.Aug 28 2017, 6:47 PM

You might reconsider just dropping this port given
https://github.com/sebastianbergmann/git/commit/e92926d76127378fc9f461fa3e6bbbe70a2165b5

mat added inline comments.Aug 28 2017, 8:33 PM
devel/pear-SebastianBergmann_Git/Makefile
8 ↗(On Diff #32450)

I really do not think the package name should be changed.

joneum updated this revision to Diff 32475.Aug 29 2017, 5:00 AM

revert PKGNAMEPREFIX

Harbormaster completed remote builds in B11279: Diff 32475.Aug 29 2017, 5:00 AM
mat added a comment.Aug 29 2017, 8:23 AM

Why remove USES=pear ?

joneum added a comment.Aug 29 2017, 1:39 PM
In D12146#252278, @mat wrote:

Why remove USES=pear ?

There is no package.xml inside the package

miwi accepted this revision.Sep 11 2017, 8:24 AM
This revision is now accepted and ready to land.Sep 11 2017, 8:24 AM
Closed by commit rP449806: - Unbreak this port (authored by joneum). · Explain WhySep 13 2017, 5:00 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
devel/
pear-SebastianBergmann_Git/
25 lines
5 lines

Diff 33032

View Options

head/devel/pear-SebastianBergmann_Git/Makefile

Loading...
View Options

head/devel/pear-SebastianBergmann_Git/distinfo

Loading...