diameter aaa protocol server: http://www.freediameter.net/trac/
It has been less than a day, please have patience.
@humdedum Be polite and patient. We're busy people with jobs who work on Homebrew in our spare time.
:recommended indicates that a dependency is not required for the formula to function but is recommended for usual functionality. An example would by PNG support in an image viewer; it's something that most users will make use of.
These flags are not currently used as the functionality has not been completed but we try and keep them accurate when possible so we can make use of them in future.
You still don't seem to have learned how to be polite. Homebrew is open-source so it's perfectly possible for you to have found all this out yourself. We have hundreds of other issues to deal with, many of which are more important than this. I'm going to just delete any comments you post that are rude.
Has the gnutls update been tested against stuff in brew uses gnutls? It already exists in homebrew-versions because I didn't want to test it against all of its dependent formulae.
brew uses gnutls
See as nothing actually requires version 3.x, I'd rather be conservative and make sure everything works. So basically everything that has a direct or transitive dependency on it (brew uses -r gnutls) should be tested.
brew uses -r gnutls
It has DTLS support, which will most probably be need in freeDiameter in few days.
brew install aqbanking bitlbee finch glib-networking gobby gst-plugins-good gwenhywfar ideviceinstaller ifuse inspircd kbtin lftp libimobiledevice libinfinity libpurple libsoup libvirt libxmlsec1 loudmouth mcabber minbif net6 nzbget obby pan pianobar qemu tiger-vnc weechat wireshark
libinfinity, gobby, libpar2 and nzbget required --use-llvm
gobby, libpurple brew failed to link but that does not look like it has anything to do with gnutls. You can see conflicting files in gist above for these two if you search for brew link.
qemu site was offline (tested from mirror: works)
tiger-vnc is only one that is dependent on 2x which I am looking right now if there is workaround.
CPU: dual-core 64-bit penryn
OS X: 10.7.5-i386
LLVM-GCC: build 2336
Clang: 4.1 build 421
X11: 2.6.4 => /usr/X11
System Ruby: 1.8.7-358
Ruby: /usr/bin/ruby => /System/Library/Frameworks/Ruby.framework/Versions/1.8/usr/bin/ruby
*just edited --config above, I accidentally run it on my mac mini and not on the machine I was testing this.
Cool, thanks for the diligence on this.
I added tiger-vnc gnutls 3x patch. Got qemu tarball from their official github mirror, since their site is still offline (I cannot add it as mirror because they do not provide same compression type on mirror) - it works.
Does github send email everytime I update comment ?
Ok this #14456 finally explained me what => :recommended does. That should be put in wiki once it's done.
So I will remove => :recommended as I cannot separate them and then use with_option or however you decide to name it once it's done.
freeDiameter formula now uses :postgresql and :mysql.
Don't worry about marking things as recommended or optional, especially because neither does anything at the moment. Jack's PR to enable those might land in a month. In the meantime you only need to mark things as => :build or nothing at all. Thanks for working on these.
@jacknagel enabling :postgresql and :mysql should not conflict with anything ? to revert it or not ?
ping @jacknagel I helped this guy make freeDiameter portable on mac and made a mistake telling him it will be available on homebrew as granted thinking: cool you want new formulas, it would be my first one :-) yay. We put homebrew method in mac os installation instruction. I tried to change everything that you have asked me to change.
If you do not want it, just tell me so I can go on with my life.
If you do not want it, just tell me so I can go on with my life.
If you need me to make a decision right now, then no. If you are willing to be patient and lose the attitude, I'll be more than happy to review this, when I have time to review it. Please understand that turnaround time > 3 days is not at all unusual, especially for new formulae as they require more scrutiny.
Unless you can magically add hours to the day or wish to hire me to work on Homebrew full-time, repeatedly pinging me isn't going to increase the amount of time I can spend reviewing contributions.
Let's make sure you're clear @humdedum your options while this works its way along. Feel free to ask questions like the one you did about :postgresql and whatnot. Because you directed it at Jack, I didn't answer it, but I can see your motivated. So I can probably help with the details, and one of the admins like Jack can do the final review.
After you're happy with the formula and it's still pending, you can still have your users/friends use it by installing via url:
brew install https://raw.github.com/humdedum/homebrew/master/Library/Formula/freediameter-server.rb
I just wanted to mention that so you don't feel like your work has hit a roadblock. Now let me make some points on how the workflow usually goes around here. Just like GitHub Help's "Bootcamp" explains, people fork mxcl/Homebrew and then create a new branch, perhaps newFreediameterServer, then they make one new thing happen in that branch, in this case the new formula. Then they submit that as a pull request. What you did was different in two respects. You submitted this off master, and you submitted four changes. That complicates things, and may not be acceptable to the admins for gory reasons that are better left unexplained.
Assuming you had four branches and four pull requests submitted, the simpler ones often get pulled first. So some progress gets made. I've had new formulas take weeks and even months for complex ones to be accepted. Once you've made your case and fixed all the issues, it shouldn't take long.
I'll make some points inline in the code, but I can also say that there is no :mysql syntax in Homebrew. The easy way to check things like that is with
brew grep :mysql
Then you can see how that dependency is crafted. You can do the same with postgresql if you use the right case:
brew grep PostgresqlInstalled
where you will see how we craft PGS requirements. Please bear with us if we seem old, as we don't answer things around here in real time like ppl do with SMS or email these days. It's because my Apple II is slow.
@2bits I was inquiring if this was ready to be used #15878
You can use :mysql and :postgresql now, I just pushed my patch.
Would it be OK if I closed this PR and open one from non-master branch ?