Hi Maxime, ‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐ On Friday, January 14th, 2022 at 5:35 PM, Maxime Devos wrote: > > Wouldn't these also need to use use 'search-input-file'? > I think I've mentioned before why 'search-input-file' needs to be > used in these kind of contexts instead of 'which' (something with > cross-compilation), if not I can explain. > That sounds vaguely familiar and makes sense. I've updated it to do that (not sure why I switched, probably since I was using that somewhere else). > Do we need propagation here? I don't quite see the need here > and propagation can cause trouble like profile conflcts during > partial upgrades (i.e. "guix install foo" after "guix pull" > without "guix package -u") > FWIW autokey is never used as library (except for autokey scripts > but I assume autokey scripts are interpreted in the same process > as autokey?) and wrap-program sets GUIX_PYTHONPATH so I think all > python libraries here could be depropagated? Though that might > be getting out of scope of the original patch, so only if you want > to investigate. > None of the inputs are propagated now (might have been confusing looking at just this patch), all are regular inputs. Runs fine and is a good point I'll keep in mind for other python programs versus libraries. > Othderwise, the patch you attached LGTM. > Just to be clear, this patch (most recent version attached) is just to do these final tweaks on top of the original patch which added autokey (as that was already pushed). So I think everything should be good now? Thanks for the review and instructive changes! John