Re: X-Face: Header Interpretation

[Nemo Thorx]

Quoting Hokan from 29 May (a Wednesday in 2019) at 1421 hours...

> page.  So I replaced this line in the script:
> 
> | { echo '/* Width=48, Height=48 */'; $UNCOMPFACE; } \
> 
> with this:
> 
> | { echo '/* Width=48, Height=48, Depth=1, Format_version=1, 
> Valid_bits_per_item=16 */'; $UNCOMPFACE; } \
> 
> and all worked.

My setup is a debian9, and uncompface works without that change for me.
I applied that tweak to my local script[12] - and still works equally well. 

If I had to guess, I'd think there is a debian-specific patch somewhere
which changes the behaviour. 

Certainly shows how these things should be better tested across
platforms)


[12] My script is somewhat tweaked - I use formail to get the headers
more reliably, and also save the resulting pbm to disk in parallel to
displaying it. But the header|icontopbm|viewer block is identical :)

.../Nemo


-- 
  - -
earth native

Re: X-Face: Header Interpretation

[Kevin J. McCarthy]

On Wed, May 29, 2019 at 02:21:22PM -0500, Hokan wrote:

Should I submit a bug report to suggest that the Mutt Wiki page be
updated to suggest this modification to the script?


Sure, don't be shy. :-)

You can also create a pull-request if you want to clone the wiki: 



--
Kevin J. McCarthy
GPG Fingerprint: 8975 A9B3 3AA3 7910 385C  5308 ADEF 7684 8031 6BDA


signature.asc
Description: PGP signature

X-Face: Header Interpretation

[Hokan]

Hello Mutters,

For kicks, I thought I'd try making the X-Face header work and followed
the instructions here:
https://gitlab.com/muttmua/mutt/wikis/MuttGuide/XFace

I installed the needed programs from the FreeBSD Ports system.

I was able to create a header, but not view the image it contained.  I
got this message from icontopbm:
"invalid sun icon file header: only 2 out of required 5 fields present".

Looking at the code for icontopbm it appears that we need more header
fields than is provided by the view-x-face script referred to by the Wiki
page.  So I replaced this line in the script:

| { echo '/* Width=48, Height=48 */'; $UNCOMPFACE; } \

with this:

| { echo '/* Width=48, Height=48, Depth=1, Format_version=1, 
Valid_bits_per_item=16 */'; $UNCOMPFACE; } \

and all worked.

Should I submit a bug report to suggest that the Mutt Wiki page be
updated to suggest this modification to the script?

-- 
Hokan
Bicyclist
Sysadmin


signature.asc
Description: PGP signature