Bug#980274: lomiri-api: Symbol mismatch when built with -O3

2021-01-17 Thread Mike Gabriel

Hi Logan,

On  So 17 Jan 2021 03:11:01 CET, Logan Rosen wrote:


Package: lomiri-api
Version: 0.1.1-3
Severity: normal
Tags: patch
User: ubuntu-de...@lists.ubuntu.com
Usertags: origin-ubuntu hirsute ubuntu-patch

Hi,

We build on ppc64el in Ubuntu with -O3 enabled, and there is a symbol
mismatch for lomiri-api. [1]


Do you enable -O3 build daemon wide? Or via debian/rules?


In Ubuntu, the attached patch was applied to achieve the following:

  * Mark two symbols as optional to fix build with -O3 on ppc64el.

Thanks for considering the patch.


If I added your patch, I presume I would break building on ppc64el in  
Debian (which looks ok):


https://buildd.debian.org/status/package.php?p=lomiri-api=sid

Thus, I am inclined to tag this bug with "wontfix" and close it. Any  
other idea (some vendor hacking in debian/rules, for example)?


Mike
--

DAS-NETZWERKTEAM
c\o Technik- und Ökologiezentrum Eckernförde
Mike Gabriel, Marienthaler Str. 17, 24340 Eckernförde
mobile: +49 (1520) 1976 148
landline: +49 (4351) 850 8940

GnuPG Fingerprint: 9BFB AEE8 6C0A A5FF BF22  0782 9AF4 6B30 2577 1B31
mail: mike.gabr...@das-netzwerkteam.de, http://das-netzwerkteam.de



pgph6LZGNMZZt.pgp
Description: Digitale PGP-Signatur


Bug#980274: lomiri-api: Symbol mismatch when built with -O3

2021-01-17 Thread Logan Rosen
Hi Mike,

On Sun, Jan 17, 2021 at 2:58 PM Mike Gabriel
 wrote:
> Do you enable -O3 build daemon wide? Or via debian/rules?

It is set for all ppc64el builds in Ubuntu via dpkg-buildflags.

> If I added your patch, I presume I would break building on ppc64el in
> Debian (which looks ok):
>
> https://buildd.debian.org/status/package.php?p=lomiri-api=sid
>
> Thus, I am inclined to tag this bug with "wontfix" and close it. Any
> other idea (some vendor hacking in debian/rules, for example)?

This should not break building on ppc64el in Debian because it just
marks two symbols as optional that are optimized away/inlined with
-O3. Whether or not they're present, the build will succeed.

I think it makes sense to mark them as optional in Debian accordingly.

Thanks,
Logan



Bug#980274: lomiri-api: Symbol mismatch when built with -O3

2021-01-16 Thread Logan Rosen
Package: lomiri-api
Version: 0.1.1-3
Severity: normal
Tags: patch
User: ubuntu-de...@lists.ubuntu.com
Usertags: origin-ubuntu hirsute ubuntu-patch

Hi,

We build on ppc64el in Ubuntu with -O3 enabled, and there is a symbol
mismatch for lomiri-api. [1]

In Ubuntu, the attached patch was applied to achieve the following:

  * Mark two symbols as optional to fix build with -O3 on ppc64el.

Thanks for considering the patch.

Logan

[1] https://launchpad.net/ubuntu/+source/lomiri-api/0.1.1-3/+build/20205674
diff -Nru lomiri-api-0.1.1/debian/liblomiri-api0.symbols 
lomiri-api-0.1.1/debian/liblomiri-api0.symbols
--- lomiri-api-0.1.1/debian/liblomiri-api0.symbols  2020-09-27 
14:30:59.0 -0400
+++ lomiri-api-0.1.1/debian/liblomiri-api0.symbols  2021-01-16 
20:01:23.0 -0500
@@ -128,8 +128,8 @@
  _ZNK6lomiri9Exception6reasonB5cxx11Ev@Base 0.1.0
  
_ZNK6lomiri9Exception9to_stringERKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEE@Base
 0.1.0
  
_ZNK6lomiri9Exception9to_stringEiRKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEE@Base
 0.1.0
- _ZNSt14_Function_baseD1Ev@Base 0.1.1
- _ZNSt14_Function_baseD2Ev@Base 0.1.1
+ (optional=templinst)_ZNSt14_Function_baseD1Ev@Base 0.1.1
+ (optional=templinst)_ZNSt14_Function_baseD2Ev@Base 0.1.1
  
(optional=templinst)_ZNSt15__exception_ptr12__dest_thunkIN6lomiri13FileExceptionEEEvPv@Base
 0.1.0
  
(optional=templinst)_ZNSt15__exception_ptr12__dest_thunkIN6lomiri14LogicExceptionEEEvPv@Base
 0.1.0
  
(optional=templinst)_ZNSt15__exception_ptr12__dest_thunkIN6lomiri16SyscallExceptionEEEvPv@Base
 0.1.0