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
gawkextlib: init at unstable #51721
gawkextlib: init at unstable #51721
Conversation
a29f4b1
to
9b83ad5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Some parts could use some rework.
@veprbl Thanks for the review. Updated PR taking your suggestions. There are a few I don't have a solid answer for at the moment. |
Are there any updates on this pull request, please? |
Not right now. I’m willing to make some changes if there is interest. |
60a4aea
to
a75fbf2
Compare
Woops. Did a massive merge by mistake. Force-pushed a more reasonable change set. |
Anything else needed? |
Fixes build error (on darwin?) In file included from xml_puller.c:18: ./xml_puller.h:17:10: fatal error: 'iconv.h' file not found ^~~~~~~~~ 1 error generated.
@tomberek I made some changes, could you review? |
Builds and functions as expected |
@GrahamcOfBorg build gawkextlib gawk-with-extensions |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gawkextlib.json
appears to be working
Motivation for this change
Using AWK recently. Found some extensions that are useful for JSON, etc.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)