Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

Zabbix 2.0.2 #14587

Closed
wants to merge 1 commit into from
Closed

Zabbix 2.0.2 #14587

wants to merge 1 commit into from

Conversation

bjoernalbers
Copy link
Contributor

New formula for "The Enterprise-class Monitoring Solution for Everyone".

@bjoernalbers
Copy link
Contributor Author

But Zabbix requires the MySQL libs for compilation and of course the DB itself for storage. Should I really remove this dependency?

@jacknagel
Copy link
Contributor

We allow non-Homebrew mysql installations, so we don't let things depend on it directly. We can use a Requirement to ensure that it is installed. grep the formula directory for "MysqlInstalled" for examples.

@bjoernalbers
Copy link
Contributor Author

@jacknagel ok, makes sense. Requirement is now included.

@adamv
Copy link
Contributor

adamv commented Sep 1, 2012

Does this need a full MySQL server, or can it compile against the mysql-connector-c (ie, for use against an off-box server)?

@bjoernalbers
Copy link
Contributor Author

@adamv The mysql-connector-c works too (just tested this). I'll update the caveats...

@bjoernalbers
Copy link
Contributor Author

Hmm, configure breaks on 10.8.2 with superenv, even when I hard-code mysql_config.
This was spit out:

==> ./configure --prefix=/usr/local/Cellar/zabbix/2.0.2 --enable-server --enable-agent --with-mysql='/usr/local/bin/mysql_config' --enable-ipv6 --with-net-snmp --with-libcurl --with-ssh2
checking for file /proc/net/dev... no
checking for long long format... no
checking for -rdynamic linking option... yes
checking for mysql_config... no
configure: error: MySQL library not found

Error: zabbix did not build
Logs: /Users/bjoern/Library/Logs/Homebrew/zabbix/
Help: https://github.com/mxcl/homebrew/wiki/troubleshooting

Same behavior when I run --with-mysql without an argument or with `which 'mysql_config' (which nils anyway).
Any ideas from the superenv S.W.A.T.? Thanks ahead!

@bjoernalbers
Copy link
Contributor Author

Ok, i got it fixed. Just for documentation:

# These do not work:                                                                                                                                                                                        
"--with-mysql='/usr/local/bin/mysql_config'",     # configure pukes on single quotes?! :-(                                                                                                                                                         
"--with-mysql=#{which 'mysql_config'}",                                                                                                                                                                    
"--with-mysql=#{which 'mysql_config' || "#{HOMEBREW_PREFIX}/bin/mysql_config"}",                                                                                                             

# ...and these do work:                                                                                                                                                                                        
"--with-mysql=/usr/local/bin/mysql_config",
"--with-mysql=#{which('mysql_config') || "#{HOMEBREW_PREFIX}/bin/mysql_config"}",

@bjoernalbers
Copy link
Contributor Author

A patch was delivered (tested successfully on 10.7.4 and 10.8.2). Thanks in advance for pulling this one in!

New formula for "The Enterprise-class Monitoring Solution for Everyone".
@adamv adamv closed this in 9bf9bab Sep 30, 2012
allenhuang pushed a commit to allenhuang/homebrew that referenced this pull request Oct 11, 2012
Closes Homebrew#14587.

Signed-off-by: Adam Vandenberg <flangy@gmail.com>
snakeyroc3 pushed a commit to snakeyroc3/homebrew that referenced this pull request Dec 17, 2012
Closes Homebrew#14587.

Signed-off-by: Adam Vandenberg <flangy@gmail.com>
@Homebrew Homebrew locked and limited conversation to collaborators Feb 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants