[archive removal candidate] compiling the ov511 driver fails

Bug #106934 reported by Peter Würtz
6
Affects Status Importance Assigned to Milestone
ov511 (Ubuntu)
Invalid
Undecided
Unassigned

Bug Description

Binary package hint: ov511-source

the code includes "linux/config.h" multiple times, which is not correct anymore. the compiler will quit because the file does not exist.

all includes of "linux/config.h" must be changed to "linux/autoconf.h"

in the current state, compiling the ov511 driver does not work

Revision history for this message
mczero (timlindner-t-online) wrote :

I can confirm this. Compilation with changed "linux/autoconf.h" compiles and runs just fine.

Revision history for this message
Thiago Teixeira (tvst) wrote :

i tried to run:

cd /usr/src/modules/ov511
sudo sed 's_linux/config.h_linux/autoconf.h_' *.c
sudo sed 's_linux/config.h_linux/autoconf.h_' *.h

but, then, module-assistant still gives me an error about there not being a 'config.h'!! and the source files are restored to their original form, with "config.h" instead of "autoconf.h".

what's going on? is this a problem with module-assistant? or should i modify the source files in some other folder?

Revision history for this message
Thiago Teixeira (tvst) wrote :

i tried to run:

cd /usr/src/modules/ov511
sudo sed -i 's_linux/config.h_linux/autoconf.h_' *.c
sudo sed -i 's_linux/config.h_linux/autoconf.h_' *.h

but, then, module-assistant still gives me an error about there not being a 'config.h'!! and the source files are restored to their original form, with "config.h" instead of "autoconf.h".

what's going on? is this a problem with module-assistant? or should i modify the source files in some other folder?

Revision history for this message
Thiago Teixeira (tvst) wrote :

oh, i just noticed that there's an archive called 'ov511.tar.bz2' at /usr/src. i deleted it then re-ran the module-assistant (after doing the replacements as shown on the post above) and it built correctly

Revision history for this message
pierrelux (pierrelucbacon) wrote :

A patch is attached to the message.

Otherwise, you can also run:
for i in `find . -type f`; do sed -i -e 's/<linux\/config.h>/<linux\/autoconf.h>/g' $i; done

from the /usr/src/modules/ov511 directory

Be careful though with "sudo module-assistant auto-install ov511" because it will uncompress ov511.tar.bz2 from /usr/src to /usr/src/modules/ and overwrite ov511/ (patched).

Revision history for this message
Jani Monoses (jani) wrote :

is this package still needed since the ov511 driver seems to be provided in the linux-ubuntu-modules-2.6.24-16-generic package?

Revision history for this message
Karel Marik (kaja-marik) wrote :

It seems there is faulty ov51x-jpeg-source version (1.5.4-1) in Hardy repository. Change log of next version 1.5.5 announces "Fixed compilation with 2.6.24 kernels". Could you please update repo?

(source - http://www.rastageeks.org/ov51x-jpeg/index.php/Changelog)

Revision history for this message
James Westby (james-w) wrote :

Hi,

I'm closing this report as it is about the ov511 driver which
is now included in the mainline kernel, and so this doesn't
need to be fixed on Intrepid.

The package is also no longer present in the repositories.

Thanks,

James

Changed in ov51x-jpeg:
status: New → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.