-
Notifications
You must be signed in to change notification settings - Fork 39
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: OpenBSD #4
base: master
Are you sure you want to change the base?
Conversation
OpenBSD uses a different path for X11 (and related) libs. And the default gcc is like 4.2 (from 2007!) -- egcc is a newer version.
I'm not going to comment on the patch system you're essentially proposing, but regardless, your PR should not include the patch already applied in the makefile. |
Ole Strøhm <notifications@github.com> wrote:
I'm not comment on the patch system you're essentially proposing, but
regardless your PR should not include the patch already applied in the
makefile.
Ah oops. Totally missed that. And I figured a patch would be a good idea
since it keeps the code base untouched -- potentially preserving the
author's ideals. Those who may wish to run herbe on an OpenBSD host can
be directed to applying the patch.
|
Hello. Sorry for the incredible late response. I didn't answer because I knew I wanted to make some changes to the makefile, but I kept delaying it. Sorry. I finally made the changes and your patch is now outdated, but it's still a one-liner. I tested it myself on a virtual machine.
My question is, what do you think about about dylans way of patching? I'm also not quite sure if it's possible to somehow update this PR to the latest version or we have to close it and create a new one? I'm still kind of new to these things. |
Hey @dudik, no worries. Yeah, I quite like Dylan's approach to patching. Works for me. And yes, this PR can be updated (even by you!) by pushing to https://github.com/icyphox/herbe at the |
Description
Allows building on OpenBSD.
Original comment:
OpenBSD uses a different path for X11 (and related) libs. And the default gcc is like 4.2 (from 2007!) -- egcc is a newer version.
Download
https://patch-diff.githubusercontent.com/raw/dudik/herbe/pull/4.diff