makemenu mis-handles the output of identify

[tovid-suite]

From br...@Awfulhak.org on December 02, 2010 04:46:40

When using ImageMagick's identify(1) to get the size of a .png file, makemenu assumes that the .png filename does not contain spaces and uses "awk '{print $3}'" to obtain the size field.

I would suggest using "identify - <pngfile" rather than "identify pngfile" as identify will never use spaces in a temporary file name.

Original issue: http://code.google.com/p/tovid/issues/detail?id=138

[tovid-suite]

From grepper@gmail.com on December 14, 2010 21:08:58

It appears that you are using a version of makemenu that is many years old - 0.30 by the look of the issue you have raised. I don't see anywhere in current makemenu (tovid 0.33) where this issue exists. Please confirm, or quote line numbers and code.

thanks, grepper

Status: Questions

[tovid-suite]

From br...@Awfulhak.org on December 15, 2010 01:50:16

Good grief, you're absolutely right! That's embarrassing!

I'll look at updating the FreeBSD port right away. Please feel free to assign this issue to the "idiot at the keyboard" category!

[tovid-suite]

From grepper@gmail.com on July 02, 2011 20:08:43

Heh, thanks for taking the time in any case :)

grepper

Status: Invalid