Mellanox / sockperf

Network Benchmarking Utility
Other
594 stars 119 forks source link

README points to irrelevant file autogen.sh #52

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. just unpack the tar.gz
2. ./autogen.sh

What is the expected output? What do you see instead?

What version of the product are you using? On what operating system?
241

Please provide any additional information below.

Original issue reported on code.google.com by dlin...@gmail.com on 19 Dec 2013 at 2:09

GoogleCodeExporter commented 9 years ago
./autogen.sh is not expected to be in tar.gz; it is only in svn/git.
in tar.gz you expect to find configure.ac and to start with ./configure.
when taking from svn/git you run ./autogen.sh for creating configure.ac.

Original comment by avne...@gmail.com on 24 Feb 2014 at 6:24

GoogleCodeExporter commented 9 years ago
In the readme it will lead to wrong steps, maybe it is require more words on 
readme:

* How to install:

  The sockperf package uses the GNU autotools compilation and installation
  framework.

    $ ./autogen.sh
    $ ./configure --prefix=<path to install>
    $ make
    $ make install

Original comment by dlin...@gmail.com on 25 Feb 2014 at 12:46

GoogleCodeExporter commented 9 years ago
Thanks dlin.tw.
Or, please handle README when you have time

Original comment by avne...@gmail.com on 25 Feb 2014 at 9:48

GoogleCodeExporter commented 9 years ago
fixed:
https://code.google.com/p/sockperf/source/detail?r=242

Original comment by orkmella...@gmail.com on 25 Feb 2014 at 10:49

GoogleCodeExporter commented 9 years ago

Original comment by orkmella...@gmail.com on 25 Feb 2014 at 10:50