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

apib 21d336c #23438

Closed
wants to merge 1 commit into from
Closed

apib 21d336c #23438

wants to merge 1 commit into from

Conversation

DominikTo
Copy link
Contributor

A simple, fast HTTP performance-testing tool

"apib is a tool that makes it easy to test API servers. It is supported for Unix systems that include the Apache Portable Runtime and OpenSSL. That includes implementations of Linux, Windows under Cygwin, and Apple OS X."

@MikeMcQuaid
Copy link
Member

Please rebase this on master.

homepage 'https://github.com/apigee/apib'
url 'https://github.com/apigee/apib/archive/aced0748c055c53ef6407ed1b13a465e121d336c.zip'
sha1 '92c1021d3ef4ad883fac8d46c7973a155fa39d5e'
version '21d336c'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you ask upstream for a tag?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reported here: apigee/apib#8

@DominikTo
Copy link
Contributor Author

Sorry! Also just noticed it. Already done. :)

"--disable-silent-rules",
"--prefix=#{prefix}"
system "make"
bin.install('apib')
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove parens, and install both in the same call: bin.install 'apib', 'apibmon'

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Updated.

@DominikTo
Copy link
Contributor Author

Tag has been added upstream and formula in this PR is updated.

@adamv
Copy link
Contributor

adamv commented Nov 19, 2013

@BrewTestBot test this please

2 similar comments
@MikeMcQuaid
Copy link
Member

@BrewTestBot test this please

@adamv
Copy link
Contributor

adamv commented Nov 20, 2013

@BrewTestBot test this please

@adamv
Copy link
Contributor

adamv commented Nov 23, 2013

Passing brew-bot now.

@adamv
Copy link
Contributor

adamv commented Nov 23, 2013

configure: WARNING: unrecognized options: --disable-debug, --disable-dependency-tracking, --disable-silent-rules

It's always a good idea to brew install -v the formula and visually inspect the output, some of the default options we set aren't used.

@adamv
Copy link
Contributor

adamv commented Nov 23, 2013

Fails to build for me here:

clang -pedantic -Wall -Wno-deprecated-declarations -o ../apib  apib_cpu.o apib_iothread.o apib_lines.o apib_main.o apib_oauth.o apib_reporting.o apib_priorityq.o apib_url.o -laprutil-1 -lapr-1 -lssl -lcrypto 
Undefined symbols for architecture x86_64:
  "_apr_file_info_get$INODE64", referenced from:
      _main in apib_main.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[1]: *** [../apib] Error 1

Update: nevermind, I had httpd installed for testing and that brings in APR.

@adamv adamv closed this in 498ae0c Nov 23, 2013
yriveiro pushed a commit to yriveiro/homebrew that referenced this pull request Nov 26, 2013
Closes Homebrew#23438.

Signed-off-by: Adam Vandenberg <flangy@gmail.com>
ehershey pushed a commit to ehershey/homebrew that referenced this pull request Apr 4, 2014
Closes Homebrew#23438.

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

Successfully merging this pull request may close these issues.

None yet

3 participants