Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

patch: FreeBSD #16

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

patch: FreeBSD #16

wants to merge 2 commits into from

Conversation

dudik
Copy link
Owner

@dudik dudik commented Sep 16, 2020

Description

Allows building on FreeBSD.

Download

https://patch-diff.githubusercontent.com/raw/dudik/herbe/pull/16.diff

@dudik dudik added the patch This pull request is a patch label Sep 16, 2020
@dudik dudik mentioned this pull request Sep 16, 2020
@vermaden
Copy link

Just made a short 'howto' for FreeBSD here:
https://vermaden.wordpress.com/2021/04/18/freebsd-desktop-part-23-configuration-herbe-notifications/

@dudik
Copy link
Owner Author

dudik commented Apr 19, 2021

@vermaden It feels great reading a post about my little program. It made my day. Thank you!

@vermaden
Copy link

@vermaden It feels great reading a post about my little program. It made my day. Thank you!

Thanks for writing such nice and minimalistic notifications app :)

Have a great day.

@0mp
Copy link

0mp commented Apr 19, 2021

Heads-up: herbe has landed in the FreeBSD Ports Collection: https://cgit.freebsd.org/ports/commit/?id=1b6314a7773b47e18f5da903035bca74a235f35c

@vermaden
Copy link

Good to hear that.

IMHO patches for Vertical Stacking and Xresources should be used as default for the FreeBSD port ... just my $0.02.

@N-R-K
Copy link

N-R-K commented Mar 12, 2022

IMO, it doesn't make much sense to leave this and #4 as a patch. Most suckless programs simply include it in their Makefile/config.mk and comment it out, here's dwm and st for example including OpenBSD flags as comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch This pull request is a patch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants