[bug #64061] pdfpic.tmac requires non-standard sed feature

Update of bug #64061 (project groff): Status: Ready for Merge => Fixed Open/Closed:Open => Closed Planned Release:None => 1.24.0

[bug #64061] pdfpic.tmac requires non-standard sed feature

Follow-up Comment #18, bug #64061 (project groff): Updated fix on branch (does not expand test cases as mused about in comment #17). commit 15ca5031a84d98a195403de89bf33a0b96c032a7 Author: G. Branden Robinson Date: Mon Apr 17 16:41:33 2023 -0500 [pdfpic]: Fix Savannah #64061. *

[bug #64061] pdfpic.tmac requires non-standard sed feature

Follow-up Comment #17, bug #64061 (project groff): Hi Deri, > The switch from using grep to sed, which seems to have caused issues, Not exactly; it's more like it caused us to pay a lot more attention to something that was a bit jinky in the first place. Here's what the code looked like when

[bug #64061] pdfpic.tmac requires non-standard sed feature

Follow-up Comment #16, bug #64061 (project groff): The switch from using grep to sed, which seems to have caused issues, may well have been unnecessary, given that the example pdf provided in bug #58206 was in fact an invalid pdf, which is why pdfinfo did not handle it correctly. The pdf

[bug #64061] pdfpic.tmac requires non-standard sed feature

Update of bug #64061 (project groff): Status: In Progress => Ready for Merge ___ Follow-up Comment #15: Fixed on branch. Still subject to change. commit 66b3077b7c265525ab72aa807e43047e5b86de8c

[bug #64061] pdfpic.tmac requires non-standard sed feature

Follow-up Comment #14, bug #64061 (project groff): [comment #11 comment #11:] > [comment #9 comment #9:] > True, but I thought you might have a setup that generated an expected temp file for the fake-pdfinfo input, which you could diff against mine, bypassing the eyeballs. Ah, no...that didn't

[bug #64061] pdfpic.tmac requires non-standard sed feature

Follow-up Comment #13, bug #64061 (project groff): Yet another realization has come to me. Since we're in unsafe mode anyway (duh, `sy`), *why are we messing with a temporary file only to source it*? This is what the _groff_ `pso` request was _made_ for. So I expect I'll grind on this a bit

[bug #64061] pdfpic.tmac requires non-standard sed feature

Follow-up Comment #12, bug #64061 (project groff): [comment #5 comment #5:] > 1. Run pdfinfo twice and scrape out a different datum on each > pass. I fear this would give up more than was gained by > eliminating grep from the pipeline. Not that we need _more_ options at this point, since two

[bug #64061] pdfpic.tmac requires non-standard sed feature

Follow-up Comment #11, bug #64061 (project groff): [comment #9 comment #9:] > I forgot to point you to my "private" branch on Savannah. Right. I did know about that branch, so I should have checked there. > It _looks_ good; of course one of the the challenges with this sort of > cleverness is

[bug #64061] pdfpic.tmac requires non-standard sed feature

Follow-up Comment #10, bug #64061 (project groff): [comment #8 comment #8:] > [comment #6 comment #6:] > > And because I couldn't resist testing my mad sed stunta > > skillz, here's a proof of concept. > > I got identical results to yours piping fake-pdfinfo into stunt.sed on OS X. So your

[bug #64061] pdfpic.tmac requires non-standard sed feature

Follow-up Comment #9, bug #64061 (project groff): [comment #7 comment #7:] > [comment #5 comment #5:] > > Dave, can you give this a crack on your Mac OS X 10.11.6 host? > > I _think_ I got it to work? > > You'll have to check my work, because I appear to be missing a step. I took the

[bug #64061] pdfpic.tmac requires non-standard sed feature

Follow-up Comment #8, bug #64061 (project groff): [comment #6 comment #6:] > And because I couldn't resist testing my mad sed stunta > skillz, here's a proof of concept. I got identical results to yours piping fake-pdfinfo into stunt.sed on OS X. So your stunt is even portable (to at least one

[bug #64061] pdfpic.tmac requires non-standard sed feature

Follow-up Comment #7, bug #64061 (project groff): [comment #5 comment #5:] > Dave, can you give this a crack on your Mac OS X 10.11.6 host? I _think_ I got it to work? You'll have to check my work, because I appear to be missing a step. I took the pdfpic*system macro from the file "64061.diff"

[bug #64061] pdfpic.tmac requires non-standard sed feature

Update of bug #64061 (project groff): Summary: pdfpic.tmac resorts to undocumented magic => pdfpic.tmac requires non-standard sed feature ___ Reply to this item at: