mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Alexander Aring <alex.aring@gmail.com>
To: ptxdist@pengutronix.de
Subject: Re: [ptxdist] [PATCH 2/3] pciutils: bump version and cleanup
Date: Wed, 2 Apr 2014 12:35:05 +0200	[thread overview]
Message-ID: <20140402103448.GA1597@x61s.campuswlan.hs-rm.de> (raw)
In-Reply-To: <20140402091546.GG31118@pengutronix.de>

On Wed, Apr 02, 2014 at 11:15:46AM +0200, Michael Olbrich wrote:
> On Tue, Apr 01, 2014 at 06:02:34PM +0200, Alexander Aring wrote:
> > This patch bumps the version of pciutils from 3.1.2 to 3.2.1.
> > 
> > Also cleanup the rule files:
> > 
> > - remove of the prepare rule.
> > - use of call ptx/ifdef instead ifdefs.
> > - Adding support for installing libpci.
> >   - Also adds a patch with rename the install dependency from
> >     install-pcilib to install-lib, which also install pkg-config files.
> >     Smells like a mainline bug, I don't know powertop doesn't find the
> >     library otherwise.
> > - Addings support for libkmod feature (sounds cool).
> > - Add install of /usr/bin/update-pciids.
> > - fit the help messages in Kconfig in 80 char width.
> > 
> > Signed-off-by: Alexander Aring <alex.aring@gmail.com>
> > ---
> >  ...iutils-install-lib-dependency-when-SHARED.patch | 31 ++++++++++++++++++++++
> >  patches/pciutils-3.2.1/series                      |  1 +
> >  rules/pciutils.in                                  | 23 ++++++++++++----
> >  rules/pciutils.make                                | 24 ++++++++---------
> >  4 files changed, 61 insertions(+), 18 deletions(-)
> >  create mode 100644 patches/pciutils-3.2.1/0001-pciutils-install-lib-dependency-when-SHARED.patch
> >  create mode 100644 patches/pciutils-3.2.1/series
> > 
> > diff --git a/patches/pciutils-3.2.1/0001-pciutils-install-lib-dependency-when-SHARED.patch b/patches/pciutils-3.2.1/0001-pciutils-install-lib-dependency-when-SHARED.patch
> > new file mode 100644
> > index 0000000..84bc3a5
> > --- /dev/null
> > +++ b/patches/pciutils-3.2.1/0001-pciutils-install-lib-dependency-when-SHARED.patch
> > @@ -0,0 +1,31 @@
> > +From d0d7d419885d98f7d54d0a66b321db08f7974a93 Mon Sep 17 00:00:00 2001
> > +From: Alexander Aring <alex.aring@gmail.com>
> > +Date: Tue, 1 Apr 2014 14:40:33 +0200
> > +Subject: [PATCH] pciutils: install-lib dependency when SHARED
> > +
> > +This patch rename the install rule dependency from install-pcilib
> > +to install-lib, when SHARED=yes. The install-pcilib doesn't install the
> > +libpci.pc files for pkg-config but install-lib does. This is necessary
> > +for packages like powertop which search for the libpci library.
> > +
> > +Signed-off-by: Alexander Aring <alex.aring@gmail.com>
> > +---
> > + Makefile | 2 +-
> > + 1 file changed, 1 insertion(+), 1 deletion(-)
> > +
> > +diff --git a/Makefile b/Makefile
> > +index 0a21ee8..5b91a5f 100644
> > +--- a/Makefile
> > ++++ b/Makefile
> > +@@ -118,7 +118,7 @@ ifeq ($(SHARED),yes)
> > + endif
> > + 
> > + ifeq ($(SHARED),yes)
> > +-install: install-pcilib
> > ++install: install-lib
> > + endif
> 
> 
> We can avoid this patch with:
> 
> PCIUTILS_INSTALL_OPT := install install-lib
> 
> right?
> 

I will test it, but I think it could work with

PCIUTILS_INSTALL_OPT := install-lib

only.

> > + 
> > + install-pcilib: lib/$(PCILIB)
> > +-- 
> > +1.9.1
> > +
> > diff --git a/patches/pciutils-3.2.1/series b/patches/pciutils-3.2.1/series
> > new file mode 100644
> > index 0000000..c6edb1c
> > --- /dev/null
> > +++ b/patches/pciutils-3.2.1/series
> > @@ -0,0 +1 @@
> > +0001-pciutils-install-lib-dependency-when-SHARED.patch
> > diff --git a/rules/pciutils.in b/rules/pciutils.in
> > index 1c11d65..d44b158 100644
> > --- a/rules/pciutils.in
> > +++ b/rules/pciutils.in
> > @@ -1,12 +1,14 @@
> >  ## SECTION=shell_and_console
> >  menuconfig PCIUTILS
> > -	select ZLIB if PCIUTILS_COMPRESS
> > +	select ZLIB	if PCIUTILS_COMPRESS
> > +	select LIBKMOD	if PCIUTILS_LIBKMOD
> >  	depends on !BUSYBOX_LSPCI || ALLYES
> 
> With the libpci option, I think it makes sense to add a PCIUTILS_TOOLS
> option (default y). Then we can move the busybox stuff there and you don't
> need to care about it in powertop.
> 
Yes, of course this is a nice idea.

> >  	tristate
> >  	prompt "pciutils                      "
> >  	help
> > -	  The PCI Utilities package contains a library for portable access to PCI bus
> > -	  configuration registers and several utilities based on this library.
> > +	  The PCI Utilities package contains a library for portable access
> > +	  to PCI bus configuration registers and several utilities based on
> > +	  this library.
> >  
> >  if PCIUTILS
> >  
> > @@ -15,9 +17,20 @@ config PCIUTILS_COMPRESS
> >  	default y
> >  	prompt "compress pci_ids"
> >  	help
> > -	  Save storage space by compressing the pci-id database. Access will be a
> > -	  little bit slower then.
> > +	  Save storage space by compressing the pci-id database. Access will
> > +	  be a little bit slower then.
> >  
> > +config PCIUTILS_LIBPCI
> > +	bool
> > +	prompt "install shared libpci library"
> > +	help
> > +	  Generate and install the shared libpci library.
> > +
> > +config PCIUTILS_LIBKMOD
> > +	bool
> > +	prompt "libkmod support"
> > +	help
> > +	  Use libkmod to resolve kernel modules on Linux.
> 
> we usually have an empty line here.
> 
oops, I will remember it. Thanks Michael.

- Alex

-- 
ptxdist mailing list
ptxdist@pengutronix.de

  reply	other threads:[~2014-04-02 10:21 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-01 16:02 [ptxdist] [PATCH 0/3] powertop: add new version with new features Alexander Aring
2014-04-01 16:02 ` [ptxdist] [PATCH 1/3] powertop: remove powertop package Alexander Aring
2014-04-01 16:02 ` [ptxdist] [PATCH 2/3] pciutils: bump version and cleanup Alexander Aring
2014-04-02  9:15   ` Michael Olbrich
2014-04-02 10:35     ` Alexander Aring [this message]
2014-04-03  8:29     ` Alexander Aring
2014-04-03  8:48       ` Michael Olbrich
2014-04-01 16:02 ` [ptxdist] [PATCH 3/3] powertop: add new package Alexander Aring
2014-04-02  9:17   ` Michael Olbrich
2014-04-02 10:38     ` Alexander Aring
2014-04-02 13:45       ` Michael Olbrich
2014-04-02 14:00         ` Alexander Aring
2014-04-07  8:03 ` [ptxdist] [PATCH 0/3] powertop: add new version with new features Juergen Beisert
2014-04-07 14:17   ` Alexander Aring

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20140402103448.GA1597@x61s.campuswlan.hs-rm.de \
    --to=alex.aring@gmail.com \
    --cc=ptxdist@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox