Summary: | app-crypt/seahorse-3.10.2: fails to build with clang-3.4* | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Austin English (RETIRED) <wizardedit> |
Component: | [OLD] Development | Assignee: | Gentoo Clang Team (OBSOLETE) <clang+obsolete> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | fsvm88, gnome |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | AMD64 | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 408963 | ||
Attachments: |
emerge log
Fix return value of seahorse_gkr_backend_constructor Fix return value of seahorse_gkr_backend_instance Fix return value of seahorse_ssh_key_properties_show |
Description
Austin English (RETIRED)
2014-02-26 00:48:09 UTC
Created attachment 371276 [details]
emerge log
Thanks for the report and sorry for such a late reply. I can reproduce this with the newest version as well. I'm afraid that the clang team in Gentoo is understaffed right now, and we're unable to handle issues of building various packages with clang ourselves. I'm CC-ing the package maintainers in case they'd be interested in the issue but since clang is not officially supported, they're not required to take any specific action. If you could report the issue upstream, that would be great. If you could provide a working patch, that would be even better. Otherwise, I'm afraid we aren't be able to fix the issue anytime soon. well, we are badly understaffed as well and afaik, none of us is running a system with clang as main compiler. However such reports may be of interest for upstream. Created attachment 382572 [details, diff]
Fix return value of seahorse_gkr_backend_constructor
Created attachment 382574 [details, diff]
Fix return value of seahorse_gkr_backend_instance
Created attachment 382576 [details, diff]
Fix return value of seahorse_ssh_key_properties_show
Latest version in portage is 3.12.2, which compiles and tests fine with clang. I have attached three patches, backported from 3.12.2 codebase, which fix the offending functions. After applying the patches, seahorse-3.10.2-r1 compiles and tests fine with both gcc 4.8.3 and clang-3.4.2 (like the latest version already does). We only support 3.12.x now, 3.10.x will be removed in the near future |