Paths

Table of Contentst

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

	F87403498: D17273.id48311.diff
	Tue, Jul 2, 3:22 PM

	Unknown Object (File)
	Sun, Jun 30, 2:39 AM

	Unknown Object (File)
	Mon, Jun 17, 2:47 PM

	Unknown Object (File)
	Wed, Jun 5, 10:49 PM

	Unknown Object (File)
	Mon, Jun 3, 8:21 PM

	Unknown Object (File)
	Mon, Jun 3, 8:21 PM

	Unknown Object (File)
	Mon, Jun 3, 7:27 PM

	Unknown Object (File)
	May 7 2024, 4:42 PM

View All 30 Files

Subscribers

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

Lint Not Applicable

Unit

Tests Not Applicable

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

@krion, thanks.

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

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	The fix is reversed. It should be `etc`, `/etc` is completely.

Revision Contents
Changeset List

Path

Size

head/

net-mgmt/

prometheus2/

1 line

2 lines

Diff 48329

head/net-mgmt/prometheus2/Makefile

Loading...

head/net-mgmt/prometheus2/pkg-plist

Loading...