dwm
ClosedPublic
Actions

Authored by 0mp on Jul 20 2018, 12:40 AM.

Details

Reviewers

krion
mat

Commits

rP475000: x11-wm/dwm: New option, which adds st and dmenu to runtime dependencies

Summary

x11-wm/dwm is a simple window manager, which is configured by modifying the source code and recompiling it. The default configuration present in the config.h file shipped within the distribution depends on the presence of a dmenu dynamic menu and a st terminal.

At the moment the user is informed about the need to install st and dmenu manually. It turns out to be somewhat confusing to some people (https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=227966).

This patch adds an option to the dwm port, which makes dmenu and st be runtime dependencies of dwm. The option is enabled by default so that it's useful to users depending on pkg.

We may use xterm or uxterm instead of st.

While here:

Remove unsused PATCH_SITES.
Silence STRIP_CMD.
Remove outdated pkg-message.

Test Plan

portlint: ok
building and packaging: ok

Diff Detail

Repository

rP FreeBSD ports repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

0mp created this revision.Jul 20 2018, 12:40 AM

Harbormaster completed remote builds in B18183: Diff 45568.Jul 20 2018, 12:40 AM

0mp retitled this revision from Add an option to make x11/st and x11/dmenu be runtime dependencies for x11-wm/dwm to Add an option to make x11/sterm and x11/dmenu be runtime dependencies for x11-wm/dwm.Jul 20 2018, 12:42 AM

krion accepted this revision.Jul 20 2018, 6:44 AM

This revision is now accepted and ready to land.Jul 20 2018, 6:44 AM

Closed by commit rP475000: x11-wm/dwm: New option, which adds st and dmenu to runtime dependencies (authored by 0mp). · Explain WhyJul 20 2018, 9:40 AM

This revision was automatically updated to reflect the committed changes.