Packaing question: need some includes from kernel source

2013-11-19 Thread Chris Adams
I'm looking at packaing mpt-status (get status info from LSI MPT SAS
RAID controllers) for Fedora and EPEL.  However, to build, it needs some
include files from the kernel source, specifically from the
drivers/message/fusion directory.  I've built an RPM using an extra
tarball of all the .h files under that directory (using the 3.11.7
kernel source), and it works (and it also works on an older kernel under
RHEL 6).

Is this allowed under the packaging guidelines, or is there a better
way?
-- 
Chris Adams li...@cmadams.net
-- 
devel mailing list
devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/devel
Fedora Code of Conduct: http://fedoraproject.org/code-of-conduct

Re: Packaing question: need some includes from kernel source

2013-11-19 Thread Dridi Boukelmoune
Hi,

Is kernel-headers enough ?
https://apps.fedoraproject.org/packages/kernel-headers

Dridi

On Tue, Nov 19, 2013 at 5:47 PM, Chris Adams li...@cmadams.net wrote:
 I'm looking at packaing mpt-status (get status info from LSI MPT SAS
 RAID controllers) for Fedora and EPEL.  However, to build, it needs some
 include files from the kernel source, specifically from the
 drivers/message/fusion directory.  I've built an RPM using an extra
 tarball of all the .h files under that directory (using the 3.11.7
 kernel source), and it works (and it also works on an older kernel under
 RHEL 6).

 Is this allowed under the packaging guidelines, or is there a better
 way?
 --
 Chris Adams li...@cmadams.net
 --
 devel mailing list
 devel@lists.fedoraproject.org
 https://admin.fedoraproject.org/mailman/listinfo/devel
 Fedora Code of Conduct: http://fedoraproject.org/code-of-conduct
-- 
devel mailing list
devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/devel
Fedora Code of Conduct: http://fedoraproject.org/code-of-conduct

Re: Packaing question: need some includes from kernel source

2013-11-19 Thread Chris Adams
Once upon a time, Dridi Boukelmoune dridi.boukelmo...@gmail.com said:
 Hi,
 
 Is kernel-headers enough ?
 https://apps.fedoraproject.org/packages/kernel-headers

I should have mentioned that neither kernel-headers nor kernel-devel has
the necessary headers.

-- 
Chris Adams li...@cmadams.net
-- 
devel mailing list
devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/devel
Fedora Code of Conduct: http://fedoraproject.org/code-of-conduct

Re: Packaing question: need some includes from kernel source

2013-11-19 Thread Dridi Boukelmoune
Maybe this should be added to an existing package. I forgot to mention
that during a review I was told not to have multiple sources.

On Tue, Nov 19, 2013 at 5:59 PM, Chris Adams li...@cmadams.net wrote:
 Once upon a time, Dridi Boukelmoune dridi.boukelmo...@gmail.com said:
 Hi,

 Is kernel-headers enough ?
 https://apps.fedoraproject.org/packages/kernel-headers

 I should have mentioned that neither kernel-headers nor kernel-devel has
 the necessary headers.

 --
 Chris Adams li...@cmadams.net
 --
 devel mailing list
 devel@lists.fedoraproject.org
 https://admin.fedoraproject.org/mailman/listinfo/devel
 Fedora Code of Conduct: http://fedoraproject.org/code-of-conduct
-- 
devel mailing list
devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/devel
Fedora Code of Conduct: http://fedoraproject.org/code-of-conduct

Re: Packaing question: need some includes from kernel source

2013-11-19 Thread Matthew Garrett
On Tue, Nov 19, 2013 at 10:47:12AM -0600, Chris Adams wrote:
 I'm looking at packaing mpt-status (get status info from LSI MPT SAS
 RAID controllers) for Fedora and EPEL.  However, to build, it needs some
 include files from the kernel source, specifically from the
 drivers/message/fusion directory.  I've built an RPM using an extra
 tarball of all the .h files under that directory (using the 3.11.7
 kernel source), and it works (and it also works on an older kernel under
 RHEL 6).

If the headers describe a stable interface that should be used by 
userland then it's a kernel bug that they're not being exported. If they 
don't, you shouldn't use them.

-- 
Matthew Garrett | mj...@srcf.ucam.org
-- 
devel mailing list
devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/devel
Fedora Code of Conduct: http://fedoraproject.org/code-of-conduct

Re: Packaing question: need some includes from kernel source

2013-11-19 Thread Chris Adams
Once upon a time, Matthew Garrett mj...@srcf.ucam.org said:
 On Tue, Nov 19, 2013 at 10:47:12AM -0600, Chris Adams wrote:
  I'm looking at packaing mpt-status (get status info from LSI MPT SAS
  RAID controllers) for Fedora and EPEL.  However, to build, it needs some
  include files from the kernel source, specifically from the
  drivers/message/fusion directory.  I've built an RPM using an extra
  tarball of all the .h files under that directory (using the 3.11.7
  kernel source), and it works (and it also works on an older kernel under
  RHEL 6).
 
 If the headers describe a stable interface that should be used by 
 userland then it's a kernel bug that they're not being exported. If they 
 don't, you shouldn't use them.

I believe the headers describe what is intended to be a stable
interface.  Would that be something I should bring up on linux-kernel?

If the interface is supposed to be stable, asking upstream kernel to
change isn't going to help make a package any time soon (and likely
never for EPEL).  Is it permissible to carry the necessary headers in
the SRPM until they are no longer needed?
-- 
Chris Adams li...@cmadams.net
-- 
devel mailing list
devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/devel
Fedora Code of Conduct: http://fedoraproject.org/code-of-conduct

Re: Packaing question: need some includes from kernel source

2013-11-19 Thread Matthew Garrett
On Tue, Nov 19, 2013 at 01:31:20PM -0600, Chris Adams wrote:
 Once upon a time, Matthew Garrett mj...@srcf.ucam.org said:
  If the headers describe a stable interface that should be used by 
  userland then it's a kernel bug that they're not being exported. If they 
  don't, you shouldn't use them.
 
 I believe the headers describe what is intended to be a stable
 interface.  Would that be something I should bring up on linux-kernel?

Probably better to look the driver up in MAINTAINERS and contact the 
people there.

 If the interface is supposed to be stable, asking upstream kernel to
 change isn't going to help make a package any time soon (and likely
 never for EPEL).  Is it permissible to carry the necessary headers in
 the SRPM until they are no longer needed?

From a technical perspective? Sure. Better to do that than add a 
gratuitous additional package.

-- 
Matthew Garrett | mj...@srcf.ucam.org
-- 
devel mailing list
devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/devel
Fedora Code of Conduct: http://fedoraproject.org/code-of-conduct

Re: Packaing question: need some includes from kernel source

2013-11-19 Thread Toshio Kuratomi
On Tue, Nov 19, 2013 at 06:05:20PM +0100, Dridi Boukelmoune wrote:
 Maybe this should be added to an existing package. I forgot to mention
 that during a review I was told not to have multiple sources.
 
nod  The main reason is so that the two packages can evolve independent of
each other.  You don't want to have to update your package because the other
source changed.  That would seem to apply here as well.

-Toshio


pgpINYcX0Jklg.pgp
Description: PGP signature
-- 
devel mailing list
devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/devel
Fedora Code of Conduct: http://fedoraproject.org/code-of-conduct