Bug 130069 - PATCH: flow-rptfmt in net-mgmt/flow-tools-ng does not work
Summary: PATCH: flow-rptfmt in net-mgmt/flow-tools-ng does not work
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Dmitry Marakasov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-12-30 23:50 UTC by Michael W Lucas
Modified: 2009-01-12 00:40 UTC (History)
0 users

See Also:


Attachments
file.diff (203 bytes, patch)
2008-12-30 23:50 UTC, Michael W Lucas
no flags Details | Diff
flow-tools-ng.diff (2.98 KB, patch)
2009-01-02 17:13 UTC, c.kworr
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael W Lucas 2008-12-30 23:50:00 UTC
Checked on a recent -current with 16 Dec 2008 ports tree.  No update
for net-mgmt/flow-tools-ng on cvsweb.

The flow-rptfmt utility has the path to env hard-coded to /bin/env.
FreeBSD has /usr/bin/env instead.  flow-rptfmt is the recommended way to
create reports from flow-tools data (even though clearly nobody uses
it except me. :-)

This is probably applicable for net-mgmt/flow-tools/ as well.

FreeBSD is the reference platform for my network analysis book, so I'd really like to see this fixed.

Fix: See the attached (trivial) patch.

Patch attached with submission follows:
How-To-Repeat: Install flow-tools-ng.  Get some data.  Try to use flow-report with a
default stat.cfg, i.e.:

# flow-cat ft-v05.2008-12-01.12* | flow-report -v TYPE=ip-source/destination-port -v SORT=+flows | less

Watch it fail.
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2008-12-30 23:50:14 UTC
Maintainer of net-mgmt/flow-tools-ng,

Please note that PR ports/130069 has just been submitted.

If it contains a patch for an upgrade, an enhancement or a bug fix
you agree on, reply to this email stating that you approve the patch
and a committer will take care of it.

The full text of the PR can be found at:
    http://www.freebsd.org/cgi/query-pr.cgi?pr=ports/130069

-- 
Edwin Groothuis via the GNATS Auto Assign Tool
edwin@FreeBSD.org
Comment 2 Edwin Groothuis freebsd_committer freebsd_triage 2008-12-30 23:50:15 UTC
State Changed
From-To: open->feedback

Awaiting maintainers feedback (via the GNATS Auto Assign Tool)
Comment 3 c.kworr 2009-01-02 17:13:24 UTC
Sorry for late response.

How about patching all other python/perl files? Try the attached patch 
please, however it's really a bit trivial...

-- 
Sphinx of black quartz judge my vow.
Comment 4 Michael W Lucas 2009-01-03 04:44:38 UTC
Good idea.  Seems to work fine.  Thanks!

On Fri, Jan 02, 2009 at 07:13:24PM +0200, Volodymyr Kostyrko wrote:
> Sorry for late response.
> 
> How about patching all other python/perl files? Try the attached patch 
> please, however it's really a bit trivial...
> 
> -- 
> Sphinx of black quartz judge my vow.



-- 
Michael W. Lucas 	mwlucas@BlackHelicopters.org, mwlucas@FreeBSD.org
		http://www.BlackHelicopters.org/~mwlucas/
"My pessimism extends to the point of even suspecting the sincerity of
  the pessimists." -- Jean Rostand, French biologist and philosopher
Comment 5 c.kworr 2009-01-03 08:53:15 UTC
Ok, so I can approve it now.

This patch:

  - fixes invocation of python and perl scripts by correcting path to 
env executable.

-- 
Sphinx of black quartz judge my vow.
Comment 6 Mark Linimon freebsd_committer freebsd_triage 2009-01-03 09:07:28 UTC
State Changed
From-To: feedback->open

Maintainer approved.
Comment 7 Dmitry Marakasov freebsd_committer freebsd_triage 2009-01-05 00:10:44 UTC
Responsible Changed
From-To: freebsd-ports-bugs->amdmi3

I'll take it.
Comment 8 Dmitry Marakasov freebsd_committer freebsd_triage 2009-01-12 00:32:52 UTC
State Changed
From-To: open->closed

Committed, with minor changes. Thanks!
Comment 9 dfilter service freebsd_committer freebsd_triage 2009-01-12 00:32:53 UTC
amdmi3      2009-01-12 00:32:44 UTC

  FreeBSD ports repository

  Modified files:
    net-mgmt/flow-tools-ng Makefile 
  Log:
  - Fix invocation of python and perl scripts by correcting path to env executable
  
  PR:             130069
  Submitted by:   Michael W Lucas <mwlucas at blackhelicopters dot org>
  Approved by:    Volodymyr Kostyrko <c dot kworr at gmail dot com> (maintainer)
  
  Revision  Changes    Path
  1.2       +3 -0      ports/net-mgmt/flow-tools-ng/Makefile
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"