1 patch for repository http://darcs.net/screened:

patch 60d2900f59e31985dad429447103fb80b1b47feb
Author: Ben Franksen <ben.franksen@online.de>
Date:   Sun Jan 26 17:34:03 CET 2020
  * make the index format depend on endianness
  
  Previously the index had a format that was independent from the
  architecture's endianness. This required that the code must be aware of
  endianness, which led to #ifdefs, and extra work in the custom Setup.hs. The
  new format avoids all that, at the cost of being architecture dependent.
  Since the index is just a cache, it can and should be re-created if it is
  found to be invalid. We merely need to make sure we can detect whether the
  format of the index has the same endianness as that expected by our code.
  This is done by adding a second word to the header (after the magic word)
  which should always read as 1 for the file to be considered in valid format.

Attachments

• make-the-index-format-depend-on-endianness.dpatch
• patch-preview.txt
• unnamed

Ganesh, do you see a chance we can test that this works on big endian
arch? I guess it would be difficult to find that nowadays (with either
Unix or Windows OS) except for embedded stuff...

Attachments

• pEpkey.asc

Maybe we could use sparc64 under qemu?

https://packages.debian.org/search?keywords=ghc
https://unix.stackexchange.com/questions/77691/how-i-can-emulate-a-
big-endian-platform-on-a-x86

Not sure the effort to set it up is worth it though.

I agree it's not worth the effort and i do believe it should work.

> patch 60d2900f59e31985dad429447103fb80b1b47feb
> Author: Ben Franksen <ben.franksen@online.de>
> Date:   Sun Jan 26 17:34:03 CET 2020
>   * make the index format depend on endianness


> +index_endianness :: Int32
> +index_endianness = 1

This name is a bit confusing, it's more like
"index_endianness_indicator" or something. A comment along the lines of
"-- |Stored to the index to verify we are on the same endianness when
reading it back" would help.

Looks fine otherwise.