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

gnu make fallout #343

Merged
merged 3 commits into from
Oct 7, 2014
Merged

gnu make fallout #343

merged 3 commits into from
Oct 7, 2014

Conversation

ddpbsd
Copy link
Member

@ddpbsd ddpbsd commented Oct 7, 2014

We cannot specify "make" if we're using gnu make.
I'm not sure which other systems need this fix, but I'm pretty sure this should help the BSDs.

@cgzones
Copy link
Contributor

cgzones commented Oct 7, 2014

Is MAKEBIN somehow set for non-BSD's (default to make?)?

@ddpbsd
Copy link
Member Author

ddpbsd commented Oct 7, 2014

@cgzones Not that I'm aware of. That's why I set it earlier in the file. Should I use a different variable name?

@cgzones
Copy link
Contributor

cgzones commented Oct 7, 2014

That's why I set it earlier in the file.

Sorry if i'm wrong, but i can't find it.

@ddpbsd
Copy link
Member Author

ddpbsd commented Oct 7, 2014

@cgzones Lines 79-91

@cgzones
Copy link
Contributor

cgzones commented Oct 7, 2014

Yeah, for OpenBSD, FreeBSD, NetBSD and DragonflyBSD.
But what is MAKEBIN set to on Linux, Darwin...?

@ddpbsd
Copy link
Member Author

ddpbsd commented Oct 7, 2014

@cgzones Thank you for explaining. You're definitely right. I'm a silly goose.

@ddpbsd
Copy link
Member Author

ddpbsd commented Oct 7, 2014

@cgzones and updated.

@cgzones
Copy link
Contributor

cgzones commented Oct 7, 2014

@ddpbsd thanks

cgzones added a commit that referenced this pull request Oct 7, 2014
@cgzones cgzones merged commit 38a1cff into ossec:master Oct 7, 2014
@jrossi jrossi added this to the ossec-hids-2.9 milestone Oct 7, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants