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

Add Analog web log tool #15016

Closed
wants to merge 13 commits into
from

Conversation

Projects
None yet
9 participants
Contributor

thaxter commented Sep 20, 2012

Addition of this ever-useful weblog analysis tool to brew. Based roughly on the
FreeBSD port install commands, including the analog.cfg-dist, which should be
fine, since analog doesn't really depend on the config file, but brew-specific
style points would be welcome. All extra and supporting files under
share/analog. Of these, only the lang files are essential.

Add analog, venerable web logfile analyzer.
Addition of this ever-useful weblog analysis tool to brew. Based roughly on the
FreeBSD port install commands, including the analog.cfg-dist, which should be
fine, since analog doesn't really depend on the config file, but brew-specific
style points would be welcome. All extra and supporting files under
share/analog. Of these, only the lang files are essential. This is not a full
GNU make, just a simple make, with an extra definition for the lang files
location, and manual copy of supporting files into place.

Suggestions welcome, I would prefer to use brew at this point (over that other
thing that installs parallel libraries willy-nilly) so let's get this committed.

@jacknagel jacknagel and 1 other commented on an outdated diff Sep 20, 2012

Library/Formula/analog.rb
+ depends_on 'jpeg'
+ depends_on 'libpng'
+
+ def install
+ system "make DEFS='-DLANGDIR=\\\"/usr/local/share/analog/lang/\\\"'"
+ system "mkdir -p /usr/local/Cellar/analog/6.0/bin"
+ system "mkdir -p /usr/local/Cellar/analog/6.0/share/analog/lang"
+ system "mkdir -p /usr/local/Cellar/analog/6.0/share/man"
+ system "mkdir -p /usr/local/Cellar/analog/6.0/share/man/man1"
+ system "mkdir -p /usr/local/Cellar/analog/6.0/etc"
+ system "mkdir -p /usr/local/Cellar/analog/6.0/share/analog"
+ system "cp analog /usr/local/Cellar/analog/6.0/bin"
+ system "cp -R examples how-to images lang /usr/local/Cellar/analog/6.0/share/analog/"
+ system "cp analog.man /usr/local/Cellar/analog/6.0/share/man/man1/analog.1"
+ system "cp analog.cfg /usr/local/Cellar/analog/6.0/etc/analog.cfg-dist"
+ end
@jacknagel

jacknagel Sep 20, 2012

Contributor

Can't assume /usr/local, and you don't have to hardcode paths into the cellar, there are methods for this. Take a look at https://github.com/mxcl/homebrew/wiki/Formula-Cookbook

@thaxter

thaxter Sep 20, 2012

Contributor

Thanks, I fixed that, also changed hah to sha1 and committed.

Was having computer troubles and fat fingered something, I think I got it straightened out.

On Sep 19, 2012, at 23:10, Jack Nagel notifications@github.com wrote:

In Library/Formula/analog.rb:

  • depends_on 'jpeg'
  • depends_on 'libpng'
  • def install
  • system "make DEFS='-DLANGDIR=\"/usr/local/share/analog/lang/\"'"
  • system "mkdir -p /usr/local/Cellar/analog/6.0/bin"
  • system "mkdir -p /usr/local/Cellar/analog/6.0/share/analog/lang"
  • system "mkdir -p /usr/local/Cellar/analog/6.0/share/man"
  • system "mkdir -p /usr/local/Cellar/analog/6.0/share/man/man1"
  • system "mkdir -p /usr/local/Cellar/analog/6.0/etc"
  • system "mkdir -p /usr/local/Cellar/analog/6.0/share/analog"
  • system "cp analog /usr/local/Cellar/analog/6.0/bin"
  • system "cp -R examples how-to images lang /usr/local/Cellar/analog/6.0/share/analog/"
  • system "cp analog.man /usr/local/Cellar/analog/6.0/share/man/man1/analog.1"
  • system "cp analog.cfg /usr/local/Cellar/analog/6.0/etc/analog.cfg-dist"
  • end
    Can't assume /usr/local, and you don't have to hardcode paths into the cellar, there are methods for this. Take a look at https://github.com/mxcl/homebrew/wiki/Formula-Cookbook


Reply to this email directly or view it on GitHub.

yrammos and others added some commits Sep 16, 2012

lilypond: add option to build docs
Closes #14946.

Signed-off-by: Jack Nagel <jacknagel@gmail.com>
ircd-hybrid: Bump up the nickname limit to something airier.
There's no config setting for this, so we need to compile it in to something generous and useful

Closes #15005.

Signed-off-by: Max Howell <mxcl@me.com>
net-snmp: fix compile on ML, add darwin12.h
net-snmp has some Fixes in HEAD for us coming in 5.7.2.
It fails on ML due to the same problem it has with Lion,
a missing header, and that's what's being fixed. It aslo
can't build the perl bindings because it wants to write to
`/Library/Perl`

- Duplicate the Lion path for ML and create `darwin12.h`
- Add `--enable-ipv6`
- Add `--with-logfile=#{var}/log/snmpd.log` fixes `/var/log/snmpd.log`
- Add `--disable-embedded-perl`
- Add `--without-perl-modules`
- Add `--disable-debugging`
- Remove an unused function `header_created?`

Fixes #14990.
Closes #15009.

Signed-off-by: Jack Nagel <jacknagel@gmail.com>
task 2.1.2
Closes #15012.

Signed-off-by: Adam Vandenberg <flangy@gmail.com>
atlassian-plugin-sdk 4.0
Closes #15015.

Signed-off-by: Adam Vandenberg <flangy@gmail.com>
Update Xcode.latest_version for 4.5
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
Jenkins 1.482
Closes #15017.

Signed-off-by: Adam Vandenberg <flangy@gmail.com>
Use built-in command for man page install
use man1.install and simplify 2 lines to 1
Contributor

adamv commented Sep 25, 2012

Please squash to a single commit for review; this will be easier if you make the pull request from a non-master branch.

Contributor

adamv commented Sep 30, 2012

Also, this seems to hard-code the use of gcc.

Contributor

adamv commented Sep 30, 2012

If you make pull requests from branches other than master, then you can git push -f after rebasing to update the pull request without getting merge and other commits.

Contributor

adamv commented Sep 30, 2012

The use of the install method also applies to the things installed to share, too.

@samueljohn samueljohn commented on the diff Oct 22, 2012

Library/Formula/analog.rb
@@ -0,0 +1,27 @@
+require 'formula'
+
+# Jason Thaxter <jason@thaxter.net>
@samueljohn

samueljohn Oct 22, 2012

Contributor

I appreciate your work here, but usually we use git blame and git log to assign credit.

@thaxter

thaxter Oct 22, 2012

Contributor

Must have seen it in another formula. I'll get that when I clean up the other bits.

On Oct 22, 2012, at 3:10, Samuel John notifications@github.com wrote:

In Library/Formula/analog.rb:

@@ -0,0 +1,27 @@
+require 'formula'
+
+# Jason Thaxter jason@thaxter.net
I appreciate your work here, but usually we use git blame and git log to assign credit.


Reply to this email directly or view it on GitHub.

Contributor

adamv commented Oct 27, 2012

This needs to be squashed to a single commit and rebased on master for review. Use a non-master branch for your pull request.

@adamv adamv closed this Oct 27, 2012

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