Arch Linux User Repository

Disown as I no longer have this hardware

as mentioned by @hali this worked for me. Thanks a lot.

Note this doesn't compile on 6.0 kernel due to this bug: https://github.com/patjak/bcwc_pcie/issues/205

As mentioned there (until fixed upstream) you can change the 3 ioremap_nocache() calls to ioremap() in /usr/src/bcwc-pcie-r242.ea832ac/fthd_drv.c and run sudo dkms install bcwc-pcie/r242.ea832ac and it is OK.

I'm trying to build on 5.6.4-arch1-1. I changed the ioremap_nocache calls but I still get a number of errors, ie.

/home/joe/.cache/yay/bcwc-pcie-git/src/bcwc-pcie-git/fthd_drv.c: In function ‘buf_t2h_handler’:
/home/joe/.cache/yay/bcwc-pcie-git/src/bcwc-pcie-git/fthd_drv.c:164:30: error: passing argument 1 of ‘fthd_buffer_return_handler’ from incompatible pointer type [-Werror=incompatible-pointer-types]
  164 |   fthd_buffer_return_handler(dev_priv, address & ~3, request_size);
      |                              ^~~~~~~~
      |                              |
      |                              struct fthd_private *

I see that repeated for several uses of the struct fthd_private * parameter. I get this error whether I run sudo dkms install ... or just make in my aur helper's cache directory (with the ioremap changes applied to the source).

Thanks, @hali!

Note this doesn't compile on 6.0 kernel due to this bug: https://github.com/patjak/bcwc_pcie/issues/205

As mentioned there (until fixed upstream) you can change the 3 ioremap_nocache() calls to ioremap() in /usr/src/bcwc-pcie-r242.ea832ac/fthd_drv.c and run sudo dkms install bcwc-pcie/r242.ea832ac and it is OK.

Thanks for the prompt response. :)

@eschwartz thanks for pointing out the --devel flag. The last time I check it didn't work for some reason. Now it works quite well.

I'm updating all my AUR packages.

Edit: Done ;)

@Aetf

Speaking with my official TU hat on, you are required to stop using pkgver=0.0.0, effective immediately, on all AUR packages you maintain.

Users who wish to update AUR VCS-based packages with an AUR helper are expected to use the commonly implemented --devel flag to check VCS packages for updates. Adding an infinitely high pkgver= and relying on the pkgver() function to reset it is NOT acceptable, as this constantly spams users to update even when there is no update available. Furthermore, the pkgrel= is reset to 1 upon the pkgver() function successfully modifying the pkgver=, so it doesn't do anything.

@Aetf: Thanks for the explanation of intended behavior.

Thank you for maintaining the package and I can understand the current setup offloads the maintainer from needing to update the pkgver for every git upstream update.

Indeed there's no rebuild from the output below.

However, it is a matter of my tooling usage and the dissonance compared to my usual experience up to now. The AUR helper usually (with the majority of the small number of git AUR packages I use) will not be flagged for update via yay -Syu.

I will probably need to find a way for myself to configure the AUR helper or augment my usage behavior for: 1. unable to know whether I need to update, since the AUR helper can't tell due to version difference as it overwrites the PKGBUILD with the latest pkgver=0.0.0 to be mismatched from the current installed r242.ea832ac-1. (intended behavior) 2. Although rebuilding is stopped, I need to stop the auto-reinstall if there's no package rebuild. (re-install is probably unintended. A tooling behavior issue to check)

% yay -Syu
...
...
:: 1 Packages to upgrade.
1  aur/bcwc-pcie-git  r242.ea832ac-1 -> 0.0.0-4
==> Packages to exclude: (eg: "1 2 3", "1-3", "^4" or repo name)
==> 
...
...
==> Starting pkgver()...
==> Updated version: bcwc-pcie-git r242.ea832ac-1
==> Sources are ready.
==> Making package: bcwc-pcie-git r242.ea832ac-1 (Sat 14 Dec 2019 10:15:04 AM +08)
==> Checking runtime dependencies...
==> Checking buildtime dependencies...
==> WARNING: Using existing $srcdir/ tree
==> Starting pkgver()...
==> Sources are ready.
==> bcwc-pcie-git-r242.ea832ac-1 already made -- skipping build
==> Found git repo: github.com/patjak/bcwc_pcie.git
loading packages...
warning: bcwc-pcie-git-r242.ea832ac-1 is up to date -- reinstalling
resolving dependencies...
looking for conflicting packages...

Packages (1) bcwc-pcie-git-r242.ea832ac-1

Total Installed Size:  0.16 MiB
Net Upgrade Size:      0.00 MiB

:: Proceed with installation? [Y/n]
...
:: Running pre-transaction hooks...
(1/1) Remove DKMS modules
==> dkms remove bcwc-pcie/r242.ea832ac -k 5.4.2-arch1-1
:: Processing package changes...
(1/1) reinstalling bcwc-pcie-git [###]100%
:: Running post-transaction hooks...
(1/2) Arming ConditionNeedsUpdate...
(2/2) Install DKMS modules
==> dkms install bcwc-pcie/r242.ea832ac -k 5.4.2-arch1-1

@VKen That's the intended behavior. I deliberately set pkgver=0.0.0 on AUR. So AUR helpers will always remind you to update the package.

For this particular package, the PKGBUILD is unlikely to change in a long time, so the pkgver can get out-of-sync with the latest git version. (In general, I don't think it's useful to update PKGBUILD just to bump the git version)

So this workaround forces AUR helpers to download and check the latest git version. If the latest git version matches what you have locally installed, no package will be built.