Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix various issues #80

Merged
merged 5 commits into from
Jun 22, 2023
Merged

Fix various issues #80

merged 5 commits into from
Jun 22, 2023

Conversation

AshleyYakeley
Copy link
Contributor

  • Stop parser from hanging and eating up memory
  • Remove timeout mechanism entirely
  • Fix all build warnings
  • Follow the dconf dump format more accurately
  • Support variants and records

There might still be output issues.

@AshleyYakeley
Copy link
Contributor Author

I think the backslash character literal parsing code may be wrong. I'm not sure how it's supposed to work.

@gvolpe
Copy link
Collaborator

gvolpe commented Jun 7, 2023

Thanks for doing this! Can you also update this to ubuntu-latest so this can be verified on CI?

@AshleyYakeley
Copy link
Contributor Author

OK done

@AshleyYakeley
Copy link
Contributor Author

Sorry, I don't know anything about cachix. Does this work in master?

@gvolpe
Copy link
Collaborator

gvolpe commented Jun 7, 2023

Need to bump the version of all actions to the latest, as this project hasn't seen an update in a while.

@AshleyYakeley
Copy link
Contributor Author

OK, I think you'll have to do that?

@gvolpe
Copy link
Collaborator

gvolpe commented Jun 8, 2023

You can do it on this PR in the same way you updated the ubuntu-latest value.

@AshleyYakeley
Copy link
Contributor Author

The CI is broken in master, is it not? Fix it in master and then I'll merge it in.

@jtojnar
Copy link
Contributor

jtojnar commented Jun 19, 2023

Opened CI fix pr #81

Copy link
Collaborator

@gvolpe gvolpe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you so much, this is pretty cool!

@gvolpe gvolpe merged commit bcb1c60 into nix-community:master Jun 22, 2023
@AshleyYakeley AshleyYakeley deleted the fix branch June 22, 2023 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants