lnav/0.7.0-2: FTBFS on mips, powerpc, s390x and sparc

[Salvatore Bonaccorso]

Dear mips, powerpc, s390x and sparc porters

(Please CC me on replies as I'm not subscribed to the port
mailinglist).

I'm contancting your porter lists as I have a FTBFS which seems
specific to the big endian architecture.

[1] https://buildd.debian.org/status/package.php?p=lnav

Upstream has pointed me in his comment in [2] in some direction to be
investigated.

[2] https://github.com/tstack/lnav/issues/93

Can some of you help me with this issue? Thanks in advance already.

Regards,
Salvatore

[Patrick Baggett]

Seems like a tiny fix, as mentioned by the owner "tstack" in
"line_buffer.cc(185)". Upstream should be able to do this easily.

this->lb_file_time = *((int32_t *)&gz_id[4]);

should be something to the effect of:

this->lb_file_time = (gz_id[4]) | (gz_id[5] << 8) | (gz_id[6] << 16) | (
gz_id[7] << 24);

to reassemble the 32-bit value regardless of native byte order. Also, it
will removing the warning from type-punning that happens when you
do *((int32_t *)&something);

Post by Salvatore Bonaccorso
Dear mips, powerpc, s390x and sparc porters
(Please CC me on replies as I'm not subscribed to the port
mailinglist).
I'm contancting your porter lists as I have a FTBFS which seems
specific to the big endian architecture.
[1] https://buildd.debian.org/status/package.php?p=lnav
Upstream has pointed me in his comment in [2] in some direction to be
investigated.
[2] https://github.com/tstack/lnav/issues/93
Can some of you help me with this issue? Thanks in advance already.
Regards,
Salvatore
--
with a subject of "unsubscribe". Trouble? Contact

[Salvatore Bonaccorso]

Hi Patrick,

Post by Patrick Baggett
Seems like a tiny fix, as mentioned by the owner "tstack" in
"line_buffer.cc(185)". Upstream should be able to do this easily.
this->lb_file_time = *((int32_t *)&gz_id[4]);
this->lb_file_time = (gz_id[4]) | (gz_id[5] << 8) | (gz_id[6] << 16) | (
gz_id[7] << 24);
to reassemble the 32-bit value regardless of native byte order. Also, it
will removing the warning from type-punning that happens when you
do *((int32_t *)&something);

Thanks for your quick help!

Regards,
Salvatore