Add support for Windows binaries

appliedgocode / goman

The missing man page for Go binaries. Goman fetches the repo's readme as a man page replacement.

Other

123 stars 3 forks source link

Add support for Windows binaries #4

Closed christophberger closed 7 years ago

christophberger commented 7 years ago

Symtoms (Windows only):

$ goman -v goman
No source path in D:\Go\bin\goman.exe - goman is perhaps no Go binary
Main path not found: invalid magic number in record at byte 0x0

christophberger commented 7 years ago

Source: https://www.reddit.com/r/golang/comments/6jrohy/goman_the_missing_man_pages_for_go_binaries/djk86rg/

Current status:

Right now I see these possible causes:

An old compiler version produces wrong or no symbols
The linker has stripped the symbol table from the binary
Something else has stripped the symbol table

Could any of the above apply to your case?

christophberger commented 7 years ago

https://github.com/christophberger/goman/blob/master/dwarf.go#L81

christophberger commented 7 years ago

Commit 8217019df9c110fa6611528d63e1943a0967a0f7 adds support for PE file format.

christophberger commented 7 years ago

Labeled as enhancement as the observed behavior is a result of missing support for the PE file format.

Changed the title accordingly.

christophberger commented 7 years ago

Commit ebee6137d1b1fe65c21cc377844ca4f472cba21b completes this issue.