public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@infradead.org>
To: Devin Heitmueller <dheitmueller@kernellabs.com>
Cc: "David Henig" <dhhenig@googlemail.com>,
	"Francis Barber" <fedora@barber-family.id.au>,
	"leandro Costantino" <lcostantino@gmail.com>,
	"Németh Márton" <nm127@freemail.hu>,
	linux-media@vger.kernel.org,
	"Douglas Landgraf" <dougsland@gmail.com>
Subject: Re: Make failed - standard ubuntu 9.10
Date: Fri, 29 Jan 2010 16:32:42 -0200	[thread overview]
Message-ID: <4B6329CA.2030005@infradead.org> (raw)
In-Reply-To: <829197381001290916m4eeb9271x1c858d6a6d0b9b3b@mail.gmail.com>

Devin Heitmueller wrote:
> On Fri, Jan 29, 2010 at 11:02 AM, David Henig <dhhenig@googlemail.com> wrote:
>> Thanks, I appear to have the headers and no longer have to do the symlink,
>> but still getting the same error - any help gratefully received, or do I
>> need to get a vanilla kernel?
> 
> Open up the file v4l/.config and change the line for firedtv from "=m"
> to "=n".  Then run "make".
> 
> This is a known packaging bug in Ubuntu's kernel headers.

This issue is specific to Ubuntu. With Fedora and with upstream kernels, everything compiles fine.

Maybe the better is if one of you that use Ubuntu to write a patch checking for
the affected Ubuntu versions, and automatically disabling the compilation
of this module, or doing some changes on compat.h to properly compile it.

All it is needed is to patch one of some of those files:
	v4l/scripts/make_kconfig.pl 		(for the logic to disable it on Ubuntu)
	/v4l/compat.h				(if is there some compat stuff that can be added)
	/v4l/scripts/make_config_compat.pl 	(for a most sophisticated logic based on some script)


After having the patch done, just submit it to Douglas.

Cheers,
Mauro


      parent reply	other threads:[~2010-01-29 18:32 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-01-28 22:35 Make failed - standard ubuntu 9.10 David Henig
2010-01-29  6:22 ` Németh Márton
2010-01-29  9:24   ` David Henig
2010-01-29 12:30     ` leandro Costantino
2010-01-29 14:27       ` David Henig
2010-01-29 14:52         ` Francis Barber
2010-01-29 16:02           ` David Henig
2010-01-29 17:16             ` Devin Heitmueller
2010-01-29 18:29               ` David Henig
2010-01-29 19:31                 ` Németh Márton
2010-01-29 18:32               ` Mauro Carvalho Chehab [this message]

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=4B6329CA.2030005@infradead.org \
    --to=mchehab@infradead.org \
    --cc=dheitmueller@kernellabs.com \
    --cc=dhhenig@googlemail.com \
    --cc=dougsland@gmail.com \
    --cc=fedora@barber-family.id.au \
    --cc=lcostantino@gmail.com \
    --cc=linux-media@vger.kernel.org \
    --cc=nm127@freemail.hu \
    /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