New formula: powerdnsauth #14627

Closed
wants to merge 1 commit into
from

Projects

None yet

7 participants

Contributor

PowerDNS Authoritative Server 3.1.

Contributor
adamv commented Sep 2, 2012

Have to think about this one, specifically, how the MySQL or compatible software deps are handled.

Contributor

No worries. There aren't any formulas making use of the MySQL alternatives, so this seemed like the most practical way.

Happy to rework this formula as needed.

Contributor

Took a different approach, and went with PostgreSQL instead. ;)

Contributor

Just squashed this into one commit as well, as that seems to be the preferred way of doing things. :)

I get errors on compile with Mountain Lion. For a log see here: https://gist.github.com/3662592

Contributor

Ouch, that's not good. I've only tested on Lion (not personally being keen to upgrade). I'll report that bug on the upstream developer mailing list, and see if they can work out a proper fix.

Contributor

@BlakeGardner If you have a moment, please try this latest commit to see if it works.

Upstream have responded that the failure only happens with clang. So, that's now disabled for this formula.

Would like your confirmation though, just to be sure, before squashing this commit.

@justinclift I've tested the latest commit and the compile failed again. I have updated the gist which should be more verbose: https://gist.github.com/3662592

Also i'm using apple-gcc42 provided by the homebrew/dupes tap if that helps any.

Contributor
Habbie commented Sep 10, 2012

The formatting and formulation of the errors clearly shows that this is clang too. I'm not sure what apple-gcc42 is, but those errors are from clang.

Contributor
Habbie commented Sep 10, 2012

PowerDNS SVN revision 2708 has a fix by @ahupowerdns for this issue.

Contributor

@BlakeGardner This formula should be good now.

It includes the upstream patch for the clang errors, thanks to @Habbie and Christof Meerwald. :)

@justinclift The formula now compiles and installs on OS X 10.8.1

Contributor

Thanks @BlakeGardner. :)

@adamv All ok now?

Contributor

ping ?

works - is there anything we need to do?

Bert Hubert
Netherlabs Computer Consulting BV - http://www.netherlabs.nl/
Dokter Bakkerenlaan 12, 2631 EV, Nootdorp, The Netherlands
Phone: +31-622-44-0095

On Sep 24, 2012, at 10:08 AM, Justin Clift wrote:

ping ?


Reply to this email directly or view it on GitHub.

Contributor

Nah, just waiting on one of the Homebrew committers to commit it.

Sometimes things fall through the cracks, so need a little nudging. :)

Justin Clift New formula: powerdnsauth
PowerDNS Authoritative Server 3.1.

* Updated to include origin of the patches, as per
  Adam's direction here:

    http://librelist.com/browser//homebrew/2012/9/27/powerdns-formula/#d03071974547b9a63304d59b74a11c5b

* Updated to use Requirements approach for PostgreSQL,
  as per Adam's direction here:

    http://librelist.com/browser//homebrew/2012/9/27/powerdns-formula/#5c0ba45319934ac813cceaaa45091c83
f941900
Contributor
adamv commented Oct 29, 2012

Build error on Lion:

/bin/sh ../libtool --tag=CXX   --mode=link c++ -DSYSCONFDIR=\"/usr/local/homebrew/Cellar/powerdnsauth/3.1/etc\" -DLIBDIR=\"/usr/local/homebrew/Cellar/powerdnsauth/3.1/lib\" -DLOCALSTATEDIR=\"/var/run\" -Ibackends/bind  -I/usr/local/include -I/usr/local/homebrew/Cellar/sqlite/3.7.14/include   -Iext/polarssl-1.1.2/include -pthread -Wall -O2 -rdynamic  -L/usr/local/lib -R/usr/local/lib -pthread  -o dnsreplay dnsreplay.o misc.o qtype.o logger.o statbag.o dnspcap.o dnsparser.o dnsrecords.o rcpgenerator.o base64.o dnswriter.o dnslabeltext.o unix_utility.o sillyrecords.o nsecrecords.o base32.o -lboost_program_options-mt -lz
libtool: link: c++ -DSYSCONFDIR=\"/usr/local/homebrew/Cellar/powerdnsauth/3.1/etc\" -DLIBDIR=\"/usr/local/homebrew/Cellar/powerdnsauth/3.1/lib\" -DLOCALSTATEDIR=\"/var/run\" -Ibackends/bind -I/usr/local/include -I/usr/local/homebrew/Cellar/sqlite/3.7.14/include -Iext/polarssl-1.1.2/include -pthread -Wall -O2 -rdynamic -pthread -o dnsreplay dnsreplay.o misc.o qtype.o logger.o statbag.o dnspcap.o dnsparser.o dnsrecords.o rcpgenerator.o base64.o dnswriter.o dnslabeltext.o unix_utility.o sillyrecords.o nsecrecords.o base32.o -Wl,-bind_at_load  -L/usr/local/lib -lboost_program_options-mt -lz -pthread
brew: superenv removed: -I/usr/local/include -Wall -O2 -L/usr/local/lib
clang: warning: argument unused during compilation: '-pthread'
clang: warning: argument unused during compilation: '-rdynamic'
clang: warning: argument unused during compilation: '-pthread'
clang: warning: argument unused during compilation: '-pthread'
libtool: link: c++ -DSYSCONFDIR=\"/usr/local/homebrew/Cellar/powerdnsauth/3.1/etc\" -DLIBDIR=\"/usr/local/homebrew/Cellar/powerdnsauth/3.1/lib\" -DLOCALSTATEDIR=\"/var/run\" -Ibackends/bind -I/usr/local/include -I/usr/local/homebrew/Cellar/sqlite/3.7.14/include -Iext/polarssl-1.1.2/include -pthread -Wall -O2 ../modules/gsqlite3backend/gsqlite3backend.o -rdynamic -pthread -o pdnssec pdnssec.o dbdnsseckeeper.o dnsparser.o dnsrecords.o dnswriter.o misc.o rcpgenerator.o base64.o unix_utility.o logger.o statbag.o qtype.o sillyrecords.o nsecrecords.o dnssecinfra.o base32.o ueberbackend.o dnsbackend.o arguments.o packetcache.o dnspacket.o bindbackend2.o binddnssec.o bindparser.o bindlexer.o gsqlbackend.o zoneparser-tng.o dynlistener.o dns.o randombackend.o dnssecsigner.o polarrsakeyinfra.o md5.o signingpipe.o dnslabeltext.o ednssubnet.o dnslabel.o ssqlite3.o -Wl,-bind_at_load  -ldl -L/usr/local/lib ext/polarssl-1.1.2/library/libpolarssl.a -lboost_program_options-mt -lboost_serialization-mt -L/usr/local/homebrew/Cellar/sqlite/3.7.14/lib -lsqlite3 -lz -pthread
brew: superenv removed: -I/usr/local/include -Wall -O2 -L/usr/local/lib
clang: warning: argument unused during compilation: '-pthread'
clang: warning: argument unused during compilation: '-rdynamic'
clang: warning: argument unused during compilation: '-pthread'
clang: warning: argument unused during compilation: '-pthread'
Undefined symbols for architecture x86_64:
  "unsigned long const& std::max<unsigned long>(unsigned long const&, unsigned long const&)", referenced from:
      RSADNSCryptoKeyEngine::getPublicKeyString() const in polarrsakeyinfra.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make[4]: *** [pdnssec] Error 1
make[4]: *** Waiting for unfinished jobs....
mv -f .deps/serialtweaker.Tpo .deps/serialtweaker.Po
make[3]: *** [all-recursive] Error 1
make[2]: *** [all] Error 2
make[1]: *** [all-recursive] Error 1
make: *** [all] Error 2
Error: Failure while executing: make 
Contributor

Damn.

Which version of Xcode are you running Adam?

Contributor
adamv commented Oct 30, 2012

4.3.2 I think. I'll try updating to 4.5.1 at some point.

Contributor
adamv commented Nov 2, 2012

Does this fail or work for anyone else?

Contributor

This builds for me on Xcode 4.5.1. (Edit: Lion 10.7.2.)

Question about the PostgresInstalled requirement. This looks for pg_config, which does come with Lion. My understanding though is that Lion doesn't include a complete postgres installation. Is the existence of pg_config enough to determine whether a useful postgres is present?

Contributor

Well, it's a hard call, depending upon the intended use. :)

Having pg_config around indicates at least the PostgreSQL client is present, which can be good enough if there's a PostgreSQL server available somewhere. ie on another host.

It's arguably more likely that a person installing PowerDNS on OSX would want PostgreSQL locally... but it's definitely not the only option. Worst case scenario, people can install a PostgreSQL server locally (via Homebrew, etc) if needed.

Contributor
adamv commented Jan 7, 2013

So is this a case of needing a newer Xcode and things are OK? Need to merge or reject this.

Contributor

Well, it's definitely working for me with Xcode 4.5.2, on Lion 10.7.5.

It seems like the only person having trouble with it was yourself using Xcode 4.3.2. Have you updated to newer yet?
If you have, and this formula starts working for you, then I think we have an answer. :)

Contributor
adamv commented Jan 7, 2013

I've upgraded Xcode, will give it another try.

Contributor
adamv commented Jan 7, 2013

Question to other maintainers because I forgot: is the use of HOMEBREW_PREFIX here deprecated in favor of deps' opt_prefix?

Owner

Yeh, think so.

@ghost
ghost commented Jan 7, 2013

Is powerdnsauth is really the best name? Typing powerdnsauth in google doesn't give as clear results as pdns.

Contributor
adamv commented Jan 7, 2013

If there's a name issue, let me know fast.

Contributor
adamv commented Jan 7, 2013
Contributor
adamv commented Jan 7, 2013

(We should probably be consistent with Debian.)

@adamv adamv was assigned Jan 8, 2013
Contributor

We already have a "pdnsd" formula, which I felt might be a bit too close naming wise.

If we're ok with "pdns" though, that's definitely the preferred name here. :) I can update the formula pretty easily.

@adamv adamv closed this in d5350f8 Jan 9, 2013
@ghost
ghost commented Jan 9, 2013

So pdns is not preferred name anymore ?

Contributor
adamv commented Jan 9, 2013

Oops.

@dholm dholm added a commit to dholm/homebrew that referenced this pull request Jan 14, 2013
@dholm Justin Clift + dholm PowerDNS Authoritative Server 3.1.
Closes #14627.

Signed-off-by: Adam Vandenberg <flangy@gmail.com>
3b85736
@xu-cheng xu-cheng 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.