The library seems to have afew configure options that are set to off be default, maybe you want to look into enabling and adding derivation options for them?

There are

• optional utilities based on the library
• optional functionalities like regex / signature checks and url support as well as
• a version of the library that can be used with Lua.

See configure.ac.

The library seems to have afew configure options that are set to off be default, maybe you want to look into enabling and adding derivation options for them?

There are

* optional utilities based on the library

* optional functionalities like regex / signature checks and url support as well as

* a version of the library that can be used with Lua.

See configure.ac.

That's a nice pointer, I'll look into this a bit later.

Signatures support seems out of reach at least for this version of the package: vstakhov/libucl#203

you could use an attribute set in the inputs instead of those multiple enableThis disableThat flags

Do you mean:

{ urls = true;
, regex = false;
, ...}

?

Do you mean

No. Maybe start like this:

{ stdenv
, ...
, ...
, features ? {
    urls = true;
    regex = true;
  }
}:

And then when setting configureFlags, you could use https://nixos.org/nixpkgs/manual/#function-library-lib.attrsets.mapAttrs .

@jpotier I'm making these suggestion because I think they are cool but you don't have to go with them. I just sometimes wish to encourage contributors not to afraid of writing nontrivial Nix expressions into packages. I suggest this under the hope that a committer won't object to add such a "complicated" Nix code for a simple library - due to a maintenance reasoning. See e.g: #87940 (comment) .

Oh :/ Too bad your changes weren't merged. I think using the language as much as possible is good, and I definitely like you pushing for these changes I made, and I think they make sense. Of course, these sort of changes should be evaluated on a case-by-case basis.

Oh :/ Too bad your changes weren't merged.

Don't worry I got what I wanted at #87949 :).

@GrahamcOfBorg build libucl

Alright, I guess this is ready for merging