go-macho icon indicating copy to clipboard operation
go-macho copied to clipboard

Optionally suppress output to stdout

Open wagoodman opened this issue 3 years ago • 2 comments

When using this lib I'm seeing Unsupported code directory version Runtime, please notify author. Based on how the library is being integrated (in a TUI environment) this notice mangles other output. Would you consider adding an API flag to globally disable or redirect these notices to another location (say a buffer that the caller can control)?

... and awesome library, highest-of-fives 🙌 .

wagoodman avatar Oct 18 '22 00:10 wagoodman

That's a good idea. I'll fix soon have you also tried my other tool https://github.com/blacktop/ipsw ?

blacktop avatar Oct 18 '22 09:10 blacktop

I did! Right after finding this lib -- It's an awesome tool 🥇

wagoodman avatar Oct 20 '22 17:10 wagoodman

sorry for the massive delay, I believe I turned the fmt.Printfs into return fmt.Errorfs so it shouldn't output the errors to stdout anymore, but I'll leave the issue open in case I missed one etc.

Thx again!

blacktop avatar Dec 05 '22 05:12 blacktop

no delay felt on my end, thanks for the update 😎 ! (also, we recently open sourced anchore/quill which is using go-macho for the describe command... this lib saved us a lot of time and effort 🙌 )

wagoodman avatar Dec 10 '22 21:12 wagoodman

Very cool!

Ya that's been on my todo list for a while and I started a while back here - https://github.com/blacktop/go-macho/blob/master/pkg/codesign/codesign.go#L269

blacktop avatar Dec 10 '22 21:12 blacktop