Bug 12091

Hi, thanks for reporting this bug.
As there is no maintainer for this package I added the committers in CC.

(Please set the status to 'assigned' if you are working on it)

Keywords: (none) => Triaged
CC: (none) => shlomif, tmb

Hi Pascal,

(In reply to Pascal COMBES from comment #0)
> Description of problem:
> DKMS will no be able to compile the module acpi_call.
> 
> That is due to the fact that the way the proc file systeme is handled has
> been changed in the new kernel version (>3.10.0). The main problem is that
> create_proc_entry() has been replaced by proc_create().
> The signature of the read/write operation has been modified too.
> 
> Version-Release number of selected component (if applicable):
> 0.1-2.mga3
> 
> How reproducible:
> I only tried once, but I suppose it will fail everytime as the kernel has
> been changed.
> 
> Steps to Reproduce:
> 1. Select package dkms-acpi_call-0.1-2.mga3 in repository
> 2. Restart your computer.
> 3. cat the make.log in /var/lib/dkms/acpi_call or subdirs. You will see
> build failures.
> 
> Solution:
> I took the new file from git an replaced the old source. It seems to work
> now.

Just a note - which git repository did you use - this one - https://github.com/mkottman/acpi_call or this one - https://github.com/Bumblebee-Project/acpi_call ?

Regards,

-- Shlomi Fish

Hello Shlomi,
    Sorry for the delay, I tried to answer by mail but it did not work. I thought it would be automatically appended to comments.
    I used the first git repository (https://github.com/mkottman/acpi_cal), the other one is a fork of it and is not up-to-date any more.
    As I said the function create_proc_entry() used at line 314 doesn't exist any more (after kernel version 3.10). It has been replaced by proc_create().
    Regards,
Pascal COMBES.

Hi Pascal,

(In reply to Pascal COMBES from comment #3)
> Hello Shlomi,
>     Sorry for the delay, I tried to answer by mail but it did not work. I
> thought it would be automatically appended to comments.
>     I used the first git repository (https://github.com/mkottman/acpi_cal),
> the other one is a fork of it and is not up-to-date any more.
>     As I said the function create_proc_entry() used at line 314 doesn't
> exist any more (after kernel version 3.10). It has been replaced by
> proc_create().
>     Regards,
> Pascal COMBES.

thanks for the reply. I can try updating this in Cauldron first, but we may wish to wait after Mageia 4 is released and the package freeze is over. In the meanwhile, we can prepare an update for Mageia 3. What do you think?

Regards,

-- Shlomi Fish

If it's currently broken, we better fix it before mga4 release.

You can request a freeze push once you have fixed it...

(In reply to Thomas Backlund from comment #5)
> If it's currently broken, we better fix it before mga4 release.
> 
> You can request a freeze push once you have fixed it...

good point but the problem is that, judging by the GitHub Bugtracker (see https://github.com/mkottman/acpi_call/issues ), the package won't build with kernel-3.12.x which is what Cauldron/Mga4 have.

Regards,

-- Shlomi Fish

Hello,
    As far as I'm concerned, every thing is (or at least seems so far) OK. The fix I gave in bug description does the job.
    I put it on bug tracker because I thought it should be officially fixed because other may have the same problem.
    Regards,
Pascal COMBES.

valid in mga4 too.

CC: (none) => stormi

Closing as OLD, the dkms-acpi_call package in mga3 and mga4 was never really used nor supported. Users wanting to use it functionalities should check the upstream repo directly.

(The package was imported for bumblebee I reckon, but it was never used since it has been superseded by the dkms-bbswitch package).

Status: NEW => RESOLVED
CC: (none) => remi
Resolution: (none) => OLD