Page MenuHomeFreeBSD
Differential D17273

net-mgmt/prometheus2: Fix a path in pkg-plist.
ClosedPublic
Actions

Authored by 0mp on Sep 21 2018, 2:57 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Jun 29, 3:47 AM
Unknown Object (File)
Mon, Jun 22, 5:26 PM
Unknown Object (File)
Sat, Jun 20, 12:17 PM
Unknown Object (File)
Sat, Jun 20, 12:17 PM
Unknown Object (File)
Thu, Jun 18, 1:58 PM
Unknown Object (File)
Thu, Jun 18, 11:59 AM
Unknown Object (File)
Wed, Jun 17, 1:35 PM
Unknown Object (File)
Tue, Jun 16, 1:23 PM
View All Files
Subscribers
dor

Details

Reviewers
krion
mat
Commits
rP480310: net-mgmt/prometheus2: Fix a path in pkg-plist.
Summary
net-mgmt/prometheus2: Fix a path in pkg-plist.

Fix the following installation phase error:

    cp: /usr/local/etc/newsyslog.conf.d/prometheus.conf: No such file or directory

It was caused by a relative path being specified in pkg-plist instead of an
absolute one (a leading "/" was missing).

PR:231548
Sponsored by:	Bally Wulff Games & Entertainment GmbH
Test Plan

poudriere testport: ok
portlint: ok

Diff Detail

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

Event Timeline

0mp created this revision.Sep 21 2018, 2:57 PM
Harbormaster completed remote builds in B19726: Diff 48311.Sep 21 2018, 2:58 PM
0mp edited the summary of this revision. (Show Details)Sep 21 2018, 3:02 PM
krion accepted this revision.Sep 21 2018, 3:53 PM
This revision is now accepted and ready to land.Sep 21 2018, 3:53 PM
0mp added a comment.Sep 21 2018, 4:01 PM

@krion, thanks.

I think I'll wait for the maintainer to get their approval as well.

dor added a subscriber: dor.Sep 21 2018, 4:32 PM

Thanks, fixing that path is fine.

This issue was introduced by a FreeBSD port committer adjusting one of my previous patches :(

Closed by commit rP480310: net-mgmt/prometheus2: Fix a path in pkg-plist. (authored by 0mp). · Explain WhySep 21 2018, 9:21 PM
This revision was automatically updated to reflect the committed changes.
mat added inline comments.Sep 24 2018, 11:50 AM
head/net-mgmt/prometheus2/pkg-plist
3–4 ↗(On Diff #48329)

The fix is reversed. It should be etc, /etc is completely.

Revision Contents
Changeset List

PathSize
net-mgmt/
prometheus2/
1 line
2 lines

Diff 48311

View Options

net-mgmt/prometheus2/Makefile

Loading...
View Options

net-mgmt/prometheus2/pkg-plist

Loading...