Hi Ludovic, Ludovic Courtès <l...@gnu.org> writes:
> Hi Maxim, > > Maxim Cournoyer <maxim.courno...@gmail.com> skribis: > >> +;;; Define a top level polkit variable that can be built on any of the >> +;;; supported platforms. This is to work around the fact that our >> +;;; mrustc-bootstrapped rust toolchain currently only supports the x86_64 >> +;;; architecture... >> +(define-public polkit >> + (if (string-prefix? "x86_64" >> + (or (%current-system) (%current-target-system))) >> + polkit* >> + polkit-duktape)) > > This would instead have to be a macro, similar to the ‘pkg-config’ > macro, so that (%current-system) is checked at the right time rather > than when loading this module. Thanks for tipping in! > However, since the only different between polkit and polkit-duktape > (IIUC) is that an extra patch is applied for the latter, I would instead > suggest adding a conditional build phase that applies the patch on > non-x86_64 systems. I'd still rather keep the distinct package approach, so that x86* users may still find about polkit-duktape and use it instead of polkit (via input rewriting) if they so wish. Thanks, Maxim