Page MenuHomeFreeBSD
Differential D15876

[New Port] sysutils/ssd_report
ClosedPublic
Actions

Authored by dteske on Jun 18 2018, 1:18 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jan 4, 4:44 AM
Unknown Object (File)
Nov 8 2024, 2:32 PM
Unknown Object (File)
Nov 4 2024, 1:15 PM
Unknown Object (File)
Oct 7 2024, 10:16 PM
Unknown Object (File)
Oct 4 2024, 10:39 PM
Unknown Object (File)
Oct 4 2024, 6:09 PM
Unknown Object (File)
Oct 2 2024, 1:19 PM
Unknown Object (File)
Oct 2 2024, 7:11 AM
View All Files
Subscribers
None

Details

Reviewers
mat
imp
Group Reviewers
portmgr
Commits
rP472896: [New Port] sysutils/ssd_report
Summary

Report SSD health by looking at SMART data.

Diff Detail

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

Event Timeline

dteske created this revision.Jun 18 2018, 1:18 AM
Harbormaster completed remote builds in B17394: Diff 43982.Jun 18 2018, 1:18 AM
dteske updated this revision to Diff 43988.Jun 18 2018, 6:37 AM

Add support for smart utility by Chuck Tuffli

Harbormaster completed remote builds in B17399: Diff 43988.Jun 18 2018, 6:37 AM
mat added inline comments.Jun 18 2018, 10:49 AM
sysutils/ssd_report/Makefile
16–20 ↗(On Diff #43988)

Use helpers:

smart_RUN_DEPENDS= smart:...
...
32 ↗(On Diff #43988)

Why?

35 ↗(On Diff #43988)

This should not be required.

dteske updated this revision to Diff 44044.Jun 18 2018, 9:34 PM

Address feedback from mat

Harbormaster completed remote builds in B17424: Diff 44044.Jun 18 2018, 9:34 PM
dteske marked 3 inline comments as done.Jun 18 2018, 9:35 PM
mat accepted this revision.Jun 18 2018, 9:55 PM

The pkg-descr file should be longer too. It should always be at least a couple of lines long.

This revision is now accepted and ready to land.Jun 18 2018, 9:55 PM
dteske updated this revision to Diff 44126.Jun 20 2018, 4:53 AM

Embiggen pkg-descr ^_^

This revision now requires review to proceed.Jun 20 2018, 4:53 AM
Harbormaster completed remote builds in B17472: Diff 44126.Jun 20 2018, 4:53 AM
mat accepted this revision.Jun 20 2018, 9:04 AM
This revision is now accepted and ready to land.Jun 20 2018, 9:04 AM
dteske updated this revision to Diff 44171.Jun 20 2018, 6:06 PM

Add sysutils/Makefile modification

This revision now requires review to proceed.Jun 20 2018, 6:06 PM
Harbormaster completed remote builds in B17489: Diff 44171.Jun 20 2018, 6:06 PM
dteske added a reviewer: portmgr.Jun 20 2018, 6:07 PM
dteske added 1 blocking reviewer(s): portmgr.

Blocked by pre-commit hook:

svn: E165001: Commit blocked by pre-commit hook (exit code 1) with output:
Do not commit a port with FLAVORS without first
getting approval from portmgr.
mat accepted this revision as: portmgr.Jun 20 2018, 6:29 PM

seriously?

This revision was not accepted when it landed; it landed in state Needs Review.Jun 20 2018, 6:32 PM
Closed by commit rP472896: [New Port] sysutils/ssd_report (authored by dteske). · Explain Why
This revision was automatically updated to reflect the committed changes.
dteske mentioned this in D15934: Remove unnecessary FLAVOR declaration.Jun 20 2018, 7:17 PM
dteske mentioned this in rP472940: sysutils/ssd_report: Remove unnecessary FLAVOR declaration.Jun 20 2018, 10:29 PM

Revision Contents
Changeset List

PathSize
head/
sysutils/
1 line
ssd_report/
36 lines
3 lines
5 lines

Diff 44173

View Options

head/sysutils/Makefile

Loading...
View Options

head/sysutils/ssd_report/Makefile

Loading...
View Options

head/sysutils/ssd_report/distinfo

Loading...
View Options

head/sysutils/ssd_report/pkg-descr

Loading...