spf13 / pflag

Drop-in replacement for Go's flag package, implementing POSIX/GNU-style --flags.
BSD 3-Clause "New" or "Revised" License
2.43k stars 348 forks source link

Flags().Uin64P broken #367

Closed blacktop closed 1 year ago

blacktop commented 1 year ago

ToUintE() fails to parse uint bigger than math.MaxInt64 in the cast dependancy. This prevents people from using numbers like macOS kernel addresses as arguents to cobra:

example:

❯ ipsw macho disass test-caches/IPSWs/20C5058d__iPhone15,2/kernelcache.release.iPhone15,2 -a 0xFFFFFFF008493FE0 -V --fileset-entry com.apple.AGXFirmwareKextG15P_A0RTBuddy

fails because it thinks I entered 0 as the --vaddr (virtual address) to be disassembled

blacktop commented 1 year ago

https://github.com/spf13/cobra/issues/1869

blacktop commented 1 year ago

My apologies, this is a viper issue, not a pflag issue!