[procps] Re: procps changes

  • From: Craig Small <csmall-procps@xxxxxxxxxx>
  • To: procps@xxxxxxxxxxxxx
  • Date: Fri, 30 Sep 2011 09:04:58 +1000

On Thu, Sep 29, 2011 at 09:10:26PM +0200, Sami Kerola wrote:
> The following changes since commit f43de0df19567d5d312cb722a83ee709d791bde6:
Is this at the HEAD of your git repository or do I need to specify a
specific version.  I was going to merge this weekend (thankyou for your
instruction on how it is done).

Some comments:
For the paths, don't get too worked up about them in the default install
targets.  It is best to leave them as-is, ie whatever automake uses as a
default.  The reason is that they should go into /usr/local/ by default.

Distributions use tricks like using prefix or --exec-dir type flags
to put the files where they should be.  I just put them all in as
bin_PROGRAMS and don't specify the path.

The oom flag should be a configure option. Either --enable-blah or
--disable-blah depending if it is default on or off.

I can see some little minor things that we can fixup later, not using
sysconfdir for example.  At the very least I'll merge it and then
see if we need to worry about it later.

I thought the library name was going to be libproc-ng so there is no
clash?

 - Craig
-- 
Craig Small VK2XLZ    http://www.enc.com.au/       csmall at : enc.com.au
Debian GNU/Linux      http://www.debian.org/       csmall at : debian.org
old fingerprint:       1C1B D893 1418 2AF4 45EE  95CB C76C E5AC 12CA DFA5
NEW fingerprint:       5D2F B320 B825 D939 04D2  0519 3938 F96B DF50 FEA5

Other related posts: