Skip to content

Commit

Permalink
add notes on 32-bit building
Browse files Browse the repository at this point in the history
  • Loading branch information
MestreLion committed Jul 5, 2020
1 parent 8f14080 commit 4daa751
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 0 deletions.
8 changes: 8 additions & 0 deletions README
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,14 @@ $ make && sudo make install # by default installs to /usr/local/bin/wavegain
or
$ make && sudo make install prefix=/usr # to install to /usr/bin/wavegain

Currently, 64-bit binaries build fine but can generate corrupted audio on output
files, so to prevent this by default it builts a 32-bit executable instead,
which runs and works fine on 64-bit architectures. However, multi-arch libraries
and headers might be needed as a pre-requiste for compiling. On Debian/Ubuntu this
can be installed by:

sudo apt install gcc-multilib


Pre-compiled binaries and old versions
-------------------------------------------------------------------------------
Expand Down
5 changes: 5 additions & 0 deletions TODO
Original file line number Diff line number Diff line change
@@ -1,3 +1,8 @@
- Investigate why 64-bit executable corrupt some test files by chopping audio and
changing its pitch. Original windows binary was 32 bits, and to prevent this bug
so is this build, forcedly by using GCC's -m32 flag. Proof-reading is required to
see what triggers the bug in 64-bits binaries and, to hopefully allow for a non-buggy
native AMD64 build in the future.
- Consider generating a 32-bit binary instead of 64 using GCC's `-m32` flag.
Original binary was 32 bits, the convertion to 64 never had any proof-reading,
and this might be the root cause of several bugs.
Expand Down

0 comments on commit 4daa751

Please sign in to comment.