Page MenuHomeFreeBSD

sysutils/condy: Add new port
ClosedPublic

Authored by dteske on Tue, Sep 8, 5:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Oct 1, 2:52 AM
Unknown Object (File)
Wed, Sep 30, 7:48 AM
Unknown Object (File)
Wed, Sep 30, 6:34 AM
Unknown Object (File)
Wed, Sep 30, 5:50 AM
Unknown Object (File)
Tue, Sep 29, 11:30 AM
Unknown Object (File)
Tue, Sep 29, 9:04 AM
Unknown Object (File)
Sun, Sep 27, 8:12 AM
Unknown Object (File)
Sun, Sep 27, 3:41 AM
Subscribers
None

Details

Summary

Daemon that starts conky and rotates the desktop wallpaper. Detects
fvwm, bvwm, GNOME, and KDE, and sets wallpaper with hsetroot(1).
Sample conky rcs, wallpapers, and configuration overlay included.

Diff Detail

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

Event Timeline

dteske requested review of this revision.Tue, Sep 8, 5:53 PM
dteske created this revision.

Thank you for your submission.

It would be great to have the configuration files in ${PREFIX}/etc as per the usual conventions. You can use REINPLACE_CMD in a custom do-configure target to do so.

A github WWW is automatically set if WWW is unset and USE_GITHUB is enabled, so you can remove your own setting.

The theme config files mentioned in the script seem to not be present in the repository, is this correct?

Apart from these, the port looks good!

@dteske,

Could you rebase onto a recent main?

% git arc patch -bcs D59509
Switched to a new branch 'gitarc-D59509'
Applying D59509...
Checking patch sysutils/condy/pkg-descr...
Checking patch sysutils/condy/distinfo...
Checking patch sysutils/condy/Makefile...
Checking patch sysutils/Makefile...
error: while searching for:
    SUBDIR += cmospwd
    SUBDIR += colorize
    SUBDIR += conan
    SUBDIR += confctl
    SUBDIR += conky
    SUBDIR += conky-awesome

error: patch failed: sysutils/Makefile:223
Applied patch sysutils/condy/pkg-descr cleanly.
Applied patch sysutils/condy/distinfo cleanly.
Applied patch sysutils/condy/Makefile cleanly.
Applying patch sysutils/Makefile with 1 reject...
Rejected hunk #1.

 Patch Failed!
Usage Exception: Unable to apply patch!
In D59509#1366698, @jrm wrote:

@dteske,

Could you rebase onto a recent main?

% git arc patch -bcs D59509

Works now:

dteske@FreeBSD ports $ git arc patch -cs D59509
Applying D59509...
Checking patch sysutils/condy/pkg-descr...
Checking patch sysutils/condy/distinfo...
Checking patch sysutils/condy/Makefile...
Checking patch sysutils/Makefile...
Applied patch sysutils/condy/pkg-descr cleanly.
Applied patch sysutils/condy/distinfo cleanly.
Applied patch sysutils/condy/Makefile cleanly.
Applied patch sysutils/Makefile cleanly.
 APPLIED  Successfully applied patch.
[gitarc-D59509 759271f54ed6] sysutils/condy: Add new port
 4 files changed, 32 insertions(+)
 create mode 100644 sysutils/condy/Makefile
 create mode 100644 sysutils/condy/distinfo
 create mode 100644 sysutils/condy/pkg-descr

Remove hard-coded WWW (sensible default set by bsd.sites.mk)

LGTM and approved for commit.

Consider patching the script so that it places its configuration files in ${PREFIX}/etc in the future. Perhaps also supply a sample configuration file (see @sample in Porter's Handbook).

Update to 3.2

Add examples, wallpapers, manual, changelog, options

testport ... passed
portclippy ... happy
portfmt -D ... ok
portlint ... fine

Looks good to me.
Approved for commit.

sysutils/condy/Makefile
26

DOCS and EXAMPLES are always on by default and do not need to be set explicitly.

This revision is now accepted and ready to land.Sun, Sep 20, 7:38 PM
This revision was automatically updated to reflect the committed changes.