Arch Linux User Repository

if you disto dosent patch the kernel with the module pls install lenovolegionlinux-dkms-git NOTE: For now only CachyOS does

@ptr1337 is done Now dkms package is: lenovolegionlinux-dkms-git

@MrDuartePT I don't think people will be confused regarding what the packages do, if there was a disclaimer in the pinned comments. The dependency for DKMS is not needed if this package.

This error occurs during the update

==> Starting pkgver()...
../.cache/yay/lenovolegionlinux-dkms-git/PKGBUILD: line 26: cd: lenovolegionlinux-dkms-git: No such file or directory
==> ERROR: A failure occurred in pkgver().
    Aborting...
 -> error making: lenovolegionlinux-dkms-git-exit status 4
==> Making package: lenovolegionlinux-git r255.84de130-1 (Sat 29 Apr 2023 10:54:04 PM +08)
==> Checking runtime dependencies...
==> Missing dependencies:
  -> lenovolegionlinux-dkms-git
==> Checking buildtime dependencies...
==> ERROR: Could not resolve all dependencies.
 -> error making: lenovolegionlinux-git-exit status 8
 -> Failed to install the following packages. Manual intervention is required:
lenovolegionlinux-dkms-git - exit status 4
lenovolegionlinux-git - exit status 8

Hi @ptr1337 i can do this change because people will use this package expected the kernel module to be included this will only create more confusion to the user, and dont make sense because the package not work without the kernel module. Nobara is the only distro that patch the kernel and most the times the module is outdated dont make sense in a arch base distro to have a kernel module completed outdated. You can clone the package and make the changes to not depend on dkms and included on your distro. Edit: I really dont know very well how provide work

Hi,

I think you should remove the depend on lenovolegionlinux-dkms-git. This would make force you to use the dkms package, even if you have patched this module to your kernel. This really makes not much sense.

The only good solution I see, would be to add a provide like this: 'LENOVOLEGION-MODULE'

This can be then added to kernels, which provide the patchset included and the package above does not depend all time on the dkms package.

@ptr1337 i fix the sed command. Well i made the chnage now this only have the python package and dkms is seperate.

Another thing:

Why is this required in the non(dkms) version which provides only the GUI?

install -Dm644 kernel_module/*.ko "${pkgdir}/usr/lib/modules/$(uname -r)/kernel/drivers/platform/x86"

The module should be provided from the patch itself and not copied to the package

@MrDuartePT

One more issue:

Cloning into 'lenovolegionlinux'...
done.
==> Starting prepare()...
error: pathspec '.84de130' did not match any file(s) known to git
==> ERROR: A failure occurred in prepare().
    Aborting...

Is happening when trying to build in a (docker) chroot. Did not tested locally

@MrDuartePT

Thank you very much for providing it correctly! :)

One issue I found is that "linux-headers" can be removed from the depends, as well as base-devel. :)

@MrDuartePT: Since you split off the DKMS package, that should only contain the DKMS module and this package should only include the GUI.

@ptr1337 is done Now dkms package is: lenovolegionlinux-dkms-git