On 2020/05/28 20:41, Omar Polo wrote: > > Bryan Steele <[email protected]> writes: > > > On Wed, May 27, 2020 at 09:33:57PM +0200, Omar Polo wrote: > > > > > > Hi, > > > > > > This is a port for picom, a compositor for X11. It's an actively > > > developed fork of compton. > > > > > > I've been running for a month now, and it has been rock stable > > > (instead > > > of compton which would occasionally crash.) > > > > > > Build and tested on amd64, passes port-lib-depends-check and > > > portcheck. > > > > > > A couple of notes about the patches: > > > > > > - (AFAIK) the project was recently renamed from "compton" to > > > "picom", > > > that's the reason they install some files that can conflict with > > > x11/compton (like the bin/compton{,-trans} links to > > > bin/picom{,-trans}). I've removed (and/or renamed) them to > > > avoid the > > > conflict, since I wanted to have them both installed > > > side-by-side > > > > > > - moved the manpage from the (port) default share/man/man1 to > > > man/man1 > > > > > > - the patch-src_meson_build is an hack to avoid a meson error I > > > cannot > > > understand > > > > > > Cheers! > > > > > > > I can't comment on the port itself, but picom looks like it needs > > a similar fix for vsync as compton in ports does to use the correct > > drm(4) device node. > > > > -Bryan. > > > > diff --git src/vsync.c src/vsync.c > > index 5980155..8273d28 100644 > > --- src/vsync.c > > +++ src/vsync.c > > @@ -54,7 +54,7 @@ static int vsync_drm_wait(session_t *ps) { > > */ > > static bool vsync_drm_init(session_t *ps) { > > // Should we always open card0? > > - if (ps->drm_fd < 0 && (ps->drm_fd = open("/dev/dri/card0", O_RDWR)) < > > 0) { > > + if (ps->drm_fd < 0 && (ps->drm_fd = open("/dev/drm0", O_RDWR)) < 0) { > > log_error("Failed to open device."); > > return false; > > } > > Thanks for finding this! > > attaching an updated tarball with this and the patch by Jan Beich. >
Here's a diff on top to avoid some duplication, put GH_* in the normal
place, put CONFIGURE_* variables before the targets as is usually done
in ports, and update the license marker (some of the source code files
are MPL only so I think we should just show the most restrictive license).
I updated DESCR a bit too.
The manpage is a bit of a mess but I suppose we can't expect much better
from asciidoc!
OK sthen@ with that on top (new tar with these changes attached).
diff 21bb08e451e5f7126e2ff8ba5827c403e7ceea34 /usr/ports/mystuff
blob - f4a7636c5438fc26342d756bb0ccdc139ea20786
file + x11/picom/Makefile
--- x11/picom/Makefile
+++ x11/picom/Makefile
@@ -2,14 +2,13 @@
COMMENT = lightweight compositor for X11
-V = 8
-PKGNAME = picom-${V}
+GH_ACCOUNT = yshui
+GH_PROJECT = picom
+GH_TAGNAME = v8
CATEGORIES = x11
-HOMEPAGE = https://github.com/yshui/picom
-
-# MPL-2.0 AND MIT
+# MPL 2.0
PERMIT_PACKAGE = Yes
WANTLIB += GL X11 X11-xcb c config dbus-1 ev m pcre pixman-1
@@ -17,10 +16,6 @@ WANTLIB += xcb-composite xcb-damage xcb-glx xcb-image
WANTLIB += xcb-randr xcb-render-util xcb-render xcb-shape xcb-sync
WANTLIB += xcb-xfixes xcb-xinerama xcb
-GH_ACCOUNT = yshui
-GH_PROJECT = picom
-GH_TAGNAME = v8
-
MODULES = devel/meson
BUILD_DEPENDS = devel/uthash \
@@ -34,12 +29,12 @@ LIB_DEPENDS = devel/libconfig \
devel/pcre \
x11/dbus
-pre-patch:
- cd ${WRKSRC}/media && mv compton.svg picom.svg
- cd ${WRKSRC}/media/icons/48x48 && mv compton.png picom.png
-
CONFIGURE_ARGS += -Dwith_docs=true
CONFIGURE_ENV += CPPFLAGS="${CPPFLAGS} -I${LOCALBASE}/include" \
LDFLAGS="${LDFLAGS} -L${LOCALBASE}/lib"
+
+pre-patch:
+ cd ${WRKSRC}/media && mv compton.svg picom.svg
+ cd ${WRKSRC}/media/icons/48x48 && mv compton.png picom.png
.include <bsd.port.mk>
blob - 866af67e0f75de1c6b3f447d09d715cc505836b1
file + x11/picom/pkg/DESCR
--- x11/picom/pkg/DESCR
+++ x11/picom/pkg/DESCR
@@ -1,2 +1,4 @@
-Picom is a lightweight compositor for X11 (previously a compton
-fork).
+Picom is a lightweight, standalone compositor for X11, for use with
+window managers that do not natively provide compositing functionality.
+Features include fading, shadows, transparency/dimming for inactive
+windows, and flexible configuration.
picom.tgz
Description: application/tar-gz
