simsong
New lightgrep
Ah, with the build failure, I don't think we tried running make distdir-am, probably just plain old make. Nice to have automated tests!
The most significant thing that's changed is we now have a full release of lightgrep v1.5, available here: https://github.com/strozfriedberg/lightgrep/releases/tag/1.5.0
We can change Makefile.am to fix the distdir-am target and whatever comes afterwards. We'll also adjust the CONFIGURE_ scripts to pull lightgrep v1.5.0 from the above URL. With the CONFIGURE_ scripts, is there some way to add commonality to all of them?
@simsong I would appreciate it if you could look at the diff for scan_lightgrep.cpp, as the typical scanner setup and usage is what I'm most unsure of.
Yay!----Sent from my phone.On Feb 11, 2025, at 9:55 PM, Jon Stewart @.***> wrote: Ah, with the build failure, I don't think we tried running make distdir-am, probably just plain old make. Nice to have automated tests!
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>
Sure. I will review in the next day or so.Generally, I don't review draft PRs. I missed when you tagged me. And the top of the PR said it wasn’t ready for review…----Sent from my phone.On Feb 11, 2025, at 10:05 PM, Jon Stewart @.***> wrote: The most significant thing that's changed is we now have a full release of lightgrep v1.5, available here: https://github.com/strozfriedberg/lightgrep/releases/tag/1.5.0 We can change Makefile.am to fix the distdir-am target and whatever comes afterwards. We'll also adjust the CONFIGURE_ scripts to pull lightgrep v1.5.0 from the above URL. With the CONFIGURE_ scripts, is there some way to add commonality to all of them? @simsong I would appreciate it if you could look at the diff for scan_lightgrep.cpp, as the typical scanner setup and usage is what I'm most unsure of.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>
@jonstewart - I've converted this to a draft until the crashing issue is resolved.
I believe I have a fix for lightgrep as a PR at https://github.com/strozfriedberg/lightgrep/pull/11. It may need a little more work for cleanup—unsurprising that a stupid oversight bug is hanging out in fiddly code. Once the PR is merged, I'll tag a v1.5.1 release of lightgrep for this PR to target.
@simsong can you provide some guidance about the various build scripts? do you have them all automated with github actions? In particular, what's needed to make a Windows build?
I'm not sure the status of the windows build. But I'll update it to incorporate lightgrep and build with that soon.
On Mar 20, 2025, at 3:37 PM, Jon Stewart @.***> wrote:
jonstewart left a comment (simsong/bulk_extractor#486) I believe I have a fix for lightgrep as a PR at strozfriedberg/lightgrep#11. It may need a little more work for cleanup—unsurprising that a stupid oversight bug is hanging out in fiddly code. Once the PR is merged, I'll tag a v1.5.1 release of lightgrep for this PR to target.
@simsong can you provide some guidance about the various build scripts? do you have them all automated with github actions? In particular, what's needed to make a Windows build?
— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.
https://github.com/strozfriedberg/lightgrep/pull/11 https://github.com/simsong https://github.com/simsong/bulk_extractor/pull/486#issuecomment-2740487213 https://github.com/notifications/unsubscribe-auth/AAMFHLHXIHUOUZZMEOJROT32VLAC7AVCNFSM6AAAAABW6QHWRKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDONBQGQ4DOMRRGM
jonstewart left a comment (simsong/bulk_extractor#486) https://github.com/simsong/bulk_extractor/pull/486#issuecomment-2740487213 I believe I have a fix for lightgrep as a PR at strozfriedberg/lightgrep#11 https://github.com/strozfriedberg/lightgrep/pull/11. It may need a little more work for cleanup—unsurprising that a stupid oversight bug is hanging out in fiddly code. Once the PR is merged, I'll tag a v1.5.1 release of lightgrep for this PR to target.
@simsong https://github.com/simsong can you provide some guidance about the various build scripts? do you have them all automated with github actions? In particular, what's needed to make a Windows build?
— Reply to this email directly, view it on GitHub https://github.com/simsong/bulk_extractor/pull/486#issuecomment-2740487213, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAMFHLHXIHUOUZZMEOJROT32VLAC7AVCNFSM6AAAAABW6QHWRKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDONBQGQ4DOMRRGM. You are receiving this because you were mentioned.