~flokli

[::]

Recent activity

Re: [v2] pcsclite, pcsc{-d,tools}: fix cross, {systemd,udev,dbus}Support flags a month ago

From Florian Klink to ~andir/nixpkgs-dev

On Tue, Apr 16, 2024 at 10:57:24PM +0300, Florian Klink wrote:
>I tried building `pkgsCross.aarch64-multiplatform.pcsc-tools` from
>a `x86_64-linux` system.
>
>At least on master, there was a cross regression for gnupg (which I hit
>when building pcsc-tools with systemd support, as systemd (or importd more
>specifically) uses gnupg.
>
>Though if I disable that, it successfully builds, it's unrelated to
>your patchset anyways (and maybe fixed in staging, didn't check).
>
>On Mon, Apr 15, 2024 at 01:09:16PM -0700, Adam Joseph wrote:
>>Changes from v1:
>>

Re: [PATCH 2/2] ricochet-irc: init at 3.0.11.2 a month ago

From Florian Klink to ~andir/nixpkgs-dev

On Thu, Apr 18, 2024 at 04:44:15PM +0200, sternenseemann wrote:
>
>
>On 4/15/24 01:11, Adam Joseph wrote:
>>+  src = (fetchFromGitHub {
>>+    owner = "wfr";
>>+    repo = "ricochet-irc";
>>+    rev = "irc-v${finalAttrs.version}";
>>+    sha256 = "sha256-hN2XaKGurPVx1rbfBPFvvOYyh1CuPFfycPee/MGrkak=";
>>+    fetchSubmodules = false;
>>+    forceFetchGit = true;
>>+  }).overrideAttrs (previousAttrs: {
>>+    env = (previousAttrs.env or {}) // {
>>+      # We need to fetch the `src/extern/tor` git submodule, but it

Re: [v2] pcsclite, pcsc{-d,tools}: fix cross, {systemd,udev,dbus}Support flags 2 months ago

From Florian Klink to ~andir/nixpkgs-dev

I tried building `pkgsCross.aarch64-multiplatform.pcsc-tools` from
a `x86_64-linux` system.

At least on master, there was a cross regression for gnupg (which I hit
when building pcsc-tools with systemd support, as systemd (or importd more
specifically) uses gnupg.

Though if I disable that, it successfully builds, it's unrelated to
your patchset anyways (and maybe fixed in staging, didn't check).

On Mon, Apr 15, 2024 at 01:09:16PM -0700, Adam Joseph wrote:
>Changes from v1:
>
>1. (Thanks flokli for noticing!): first patch (pcsclite: remove isLinux

Re: [PATCH 1/6] pcsclite: remove isLinux condition from --enable-ipcdir configureFlag 2 months ago

From Florian Klink to ~andir/nixpkgs-dev

On 24-04-14 15:42:54, Adam Joseph wrote:
>The `--enable-ipcdir=/run/pcscd` flag was added by commit 2b93e96d0bdf5
>"pcsclite: add policy kit support".  However the pcscd IPC mechanism is
>completely independent from polkit, systemd, and dbus.  There is no reason to
>disable the IPC mechanism on all non-Linux platforms.
>---
> pkgs/tools/security/pcsclite/default.nix | 1 -
> 1 file changed, 1 deletion(-)
>
>diff --git a/pkgs/tools/security/pcsclite/default.nix b/pkgs/tools/security/pcsclite/default.nix
>index 956bf451c7bf..44a045f1fa49 100644
>--- a/pkgs/tools/security/pcsclite/default.nix
>+++ b/pkgs/tools/security/pcsclite/default.nix
>@@ -37,7 +37,6 @@ stdenv.mkDerivation (finalAttrs: {

Re: [PATCH] git: Tell users how to install removed sub-commands 3 years ago

From Florian Klink to ~andir/nixpkgs-dev

On 20-12-16 15:50:03, Klemens Nanni wrote:
>On Tue, Nov 24, 2020 at 07:06:35PM +0100, Florian Klink wrote:
>> But yeah, no need to re-roll the patch if you don't want to, let's give
>> this some more testing first.
>Is there any chance you've carried out "some more testing" by now?

I didn't do some testing, and currently don't have the bandwidth to do
so. This should be made more visible to the broader community (through a
PR, and a post in discourse for example).

>Would it be possible for you to open a PR with this change?
>Sadly enough, it seems the community is not yet interested in alternative
>ways of contribution.

Re: [PATCH] git: Tell users how to install removed sub-commands 3 years ago

From Florian Klink to ~andir/nixpkgs-dev

>> s/install nixos.gitFull instead/use nixos.gitFull instead/, otherwise
>> LGTM. Haven't tested it yet, though.
>*User has "git" installed, sees the error and decides to do a one-off to
>for git-send-email(1) instead of switching permanently*:
>
>	$ nix-shell -p gitFull --run 'git send-email ...'
>
>Does that count as installation or usage?  I think either is fine but
>only the latter includes the former;  I'll gladly send an updated and
>final diff if need be to avoid bike-shedding once overall consensus is
>reached but will avoid such churn for now.

This is a usage, not an installation, "installing" is very much
understood to be "adding it to your profile, either by changing the

Re: [PATCH] git: Tell users how to install removed sub-commands 3 years ago

From Florian Klink to ~andir/nixpkgs-dev

s/install nixos.gitFull instead/use nixos.gitFull instead/, otherwise
LGTM. Haven't tested it yet, though.

On 20-11-23 23:08:43, Klemens Nanni wrote:
>On Thu, Nov 05, 2020 at 02:59:31PM +0100, andreas@rammhold.de wrote:
>> The file with the implementation (and docstrings) can be found here: https://github.com/NixOS/nixpkgs/blob/bd40824f97242271ec1704763bb40874ef05c9fc/pkgs/build-support/trivial-builders.nix#L192-L217
>
>Thanks, so the difference between `writeShellScript{,Bin}' is that the latter
>merely puts it under a different path inside the nix store.
>
>> Yes, it will be the same file for all the the calls. The string
>> interpolation is done once while Nix expression is being evaluated. In
>> the resulting builder script there will only be a reference to the
>> script in the path but no instructions on how that came to be (that is

Re: [PATCH] nixos/users-groups: createHome: Ensure HOME permissions, fix description 3 years ago

From Florian Klink to ~andir/nixpkgs-dev

Thanks for the patch! This should probably still have a entry in
nixos/doc/manual/release-notes/rl-2009.xml. Can you add one?

On 20-11-22 19:13:32, Klemens Nanni wrote:
>configuration.nix(1) states
>
>    users.extraUsers.<name>.createHome
>        [...] If [...] the home directory already exists but is not
>        owned by the user, directory owner and group will be changed to
>        match the user.
>
>i.e. ownership would change if and only if the user mismatched;  this is
>wrong as ownership is always ensured iff `createHome' is set:
>