This repository has been archived by the owner. It is now read-only.

Please add a formula for weighttp. #16209

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants
Contributor

hnakamur commented Nov 23, 2012

No description provided.

Contributor

adamv commented Nov 23, 2012

We're not accepting new head-only formulae into core; is there a stable download for this?

Contributor

hnakamur commented Nov 24, 2012

Modified the weighttp formula to have a stable download url for v0.2.
But v0.2 seems old, and real stable seems to be v0.3.
However there is no tags for v0.3
https://github.com/lighttpd/weighttp/tags

Contributor

hnakamur commented Nov 24, 2012

I made a request for tagging v0.3 at
http://redmine.lighttpd.net/issues/2461

Contributor

hnakamur commented Nov 30, 2012

I updated the stable download URL for v0.3.
I also updated the HEAD url.
Please review this formula.
Thanks.

Contributor

adamv commented Jan 15, 2013

Nothing gets installed:

==> ./waf build
./waf build
Waf: Entering directory `/private/tmp/weighttp-e8hB/weighttp-weighttp-0.3/build'
[1/4] cc: src/client.c -> build/default/src/client_1.o
[2/4] cc: src/weighttp.c -> build/default/src/weighttp_1.o
[3/4] cc: src/worker.c -> build/default/src/worker_1.o
[4/4] cc_link: build/default/src/client_1.o build/default/src/weighttp_1.o build/default/src/worker_1.o -> build/default/weighttp
Waf: Leaving directory `/private/tmp/weighttp-e8hB/weighttp-weighttp-0.3/build'
'build' finished successfully (0.318s)
==> ./waf install
./waf install
Waf: Entering directory `/private/tmp/weighttp-e8hB/weighttp-weighttp-0.3/build'
* installing build/default/weighttp as /usr/local/bin/weighttp
Waf: Leaving directory `/private/tmp/weighttp-e8hB/weighttp-weighttp-0.3/build'
'install' finished successfully (0.009s)
==> Cleaning
==> Finishing up
ln -s ../../Cellar/weighttp/0.3 weighttp
ln -s ../../../Users/adamv/homebrew/Cellar/weighttp/0.3 weighttp
==> Summary

Or at least, no prefix is being specified here, so it is probably installing directly into /usr/local

Contributor

adamv commented Jan 15, 2013

Use bin.install instead of the waf installer to install the binary.

@adamv adamv was assigned Jan 15, 2013

Contributor

hnakamur commented Jan 16, 2013

Hi, I updated my pull request.

  • Use bin.install to install the binary.
  • Explicitly add the version.
  • Rebase on master.

@adamv adamv closed this in 10e4a58 Jan 17, 2013

@norioxkimura norioxkimura added a commit to norioxkimura/homebrew that referenced this pull request Jan 26, 2013

@hnakamur @norioxkimura hnakamur + norioxkimura weighttp 0.3
Closes #16209.

Signed-off-by: Adam Vandenberg <flangy@gmail.com>
cb1362e

@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.