Hello Adrian, On Sat, Apr 01, 2017 at 05:42:31PM +0200, John Paul Adrian Glaubitz wrote: ... > The attached patch adds build support to Thunderbird for m68k. It is > based on a number of patches I am currently trying to submit to > Firefox upstream [1]. > > All patches have been reviewed by positively by upstream with the > exception of the alignment fixes where upstream wants to use a > C++11 solution instead of the other suggestions I made. This patch > currently uses __attribute__((aligned(4))) to ensure the alignment > is at least 4 bytes. This method is safe and works on gcc and clang > and unlike the suggested alignas() from C++11 does not break on > architectures which require stricter alignment (e.g. alignas(4) > would break on x86_64 while __attribute__((aligned(4))) does not > as it still allows for 8 bytes alignment. > > Would be great to have this patch merged to the Debian package. I'm > happy to provide support for any of the patches for the ports > architectures even for future releases in case something breaks.
your patch(es) are being applied into the debian/sid branch. Thanks for preparing the set. Right now I can't say if we will upload another 45.8.0 version to unstable. Upstream isn't clear on preparing a 45.9.0 or go over to 52.0 directly. An maybe we need to drop those patches for the Stretch release, we need to ask the release team if they accept this two patches. But they are not lost in that case, they will need some adoptions for 52.x for sure. I'm having some trouble with building 52.0 for now and dropping before all extra modifications to get a successful build. Regards Carsten
