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

Please add a formula for weighttp. #16209

Closed
wants to merge 2 commits into from
Closed

Conversation

hnakamur
Copy link
Contributor

No description provided.

@adamv
Copy link
Contributor

adamv commented Nov 23, 2012

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

@hnakamur
Copy link
Contributor Author

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

@hnakamur
Copy link
Contributor Author

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

@hnakamur
Copy link
Contributor Author

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

@adamv
Copy link
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

@adamv
Copy link
Contributor

adamv commented Jan 15, 2013

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

@ghost ghost assigned adamv Jan 15, 2013
@hnakamur
Copy link
Contributor Author

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

None yet

2 participants