search

nix-community / dconf2nix

:feet: Convert dconf files (e.g. GNOME Shell) to Nix, as expected by Home Manager [maintainer=@jtojnar]
Apache License 2.0
228 stars 6 forks source link

Error to parse curly braces within a string #10

Closed bbigras closed 4 years ago

bbigras commented 4 years ago

When I ran dconf2nix my computer soft-froze.

My dconf.settings is 12k.

gvolpe commented 4 years ago

Can you share your dconf dump?

bbigras commented 4 years ago

try this:

[org/virt-manager/virt-manager/urls]
isos=['/home/bbigras/Downloads/something {ddsa}']

The problem is the curly braces.

gvolpe commented 4 years ago

Cool, I edited the issue title to describe the issue. You can usually run dconf2nix with the --verbose flag to see what's going on and in case it cannot parse something it would hang (that's something I should probably fix too). The high CPU and memory usage is likely due to the program hanging.

gvolpe commented 4 years ago

@bbigras if you can, I'd kindly ask you to remove these values and run it again to see if there are more issues. I will fix them as soon as I can to avoid making more releases. I'm on CET timezone so now's a bit late but it's great to see someone out there is giving dconf2nix a try and reporting issues :)

bbigras commented 4 years ago

@bbigras if you can, I'd kindly ask you to remove these values and run it again to see if there are more issues. I will fix them as soon as I can to avoid making more releases. I'm on CET timezone so now's a bit late but it's great to see someone out there is giving dconf2nix a try and reporting issues :)

Yeah it ran fine without the curly braces. :)

gvolpe commented 4 years ago

Great, I'll make another release then, thanks for reporting back :)

gvolpe commented 4 years ago

Be aware of the caveats mentioned in the README file, though. Home Manager doesn't support the full protocol so even if the dconf file is converted into nix, there's a chance not all the stuff will actually work.

gvolpe commented 4 years ago

Released a new version with the fix: https://github.com/gvolpe/dconf2nix/releases/tag/v0.0.4