Summary: | dev-gap/cohomolo-1.6.11 - [gcc-15] gpd.c: error: type of gpprog does not match original declaration [-Werror=lto-type-mismatch] | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Toralf Förster <toralf> |
Component: | Current packages | Assignee: | Michael Orlitzky <mjo> |
Status: | CONFIRMED --- | ||
Severity: | normal | CC: | dimpase+gentoo, dimpase, frp.bissey, mjo, sci-mathematics |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | https://github.com/gap-packages/cohomolo/issues/36 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 618550 | ||
Attachments: |
emerge-info.txt
dev-gap:cohomolo-1.6.11:20240925-082614.log emerge-history.txt environment etc.clang.tar.xz etc.portage.tar.xz qlist-info.txt temp.tar.xz |
Description
Toralf Förster
![]() Created attachment 903729 [details]
emerge-info.txt
Created attachment 903730 [details]
dev-gap:cohomolo-1.6.11:20240925-082614.log
Created attachment 903731 [details]
emerge-history.txt
Created attachment 903732 [details]
environment
Created attachment 903733 [details]
etc.clang.tar.xz
Created attachment 903734 [details]
etc.portage.tar.xz
Created attachment 903735 [details]
qlist-info.txt
Created attachment 903736 [details]
temp.tar.xz
This is known upstream and probably won't be fixed any time soon. I think I only added this because it's included in sage's optional gap_packages? Maybe nobody needs it and it can be removed. For now filter-lto seems like a good idea, but I wonder how to avoid build failures on the tinderbox. Append -Wno-error-* ? CCing our local GAP expert. Is there a reason to keep cohomolo around? (In reply to Michael Orlitzky from comment #9) > This is known upstream and probably won't be fixed any time soon. I think I > only added this because it's included in sage's optional gap_packages? Maybe > nobody needs it and it can be removed. > > For now filter-lto seems like a good idea, but I wonder how to avoid build > failures on the tinderbox. Append -Wno-error-* ? FWIW, filter-lto handles that for you for lto-type-mismatch because it can only be diagnosed at LTO-time. For aliasing issues, -fno-strict-aliasing should be passed. For ODR, filter-lto and pray. So just filter-lto should be OK here please report upstream to https://github.com/gap-packages/cohomolo (In reply to Dima Pasechnik from comment #12) > please report upstream to https://github.com/gap-packages/cohomolo It was reported upstream a year and a half ago at $URL :) Max's response was, Sure, there are lots of issues with this code base. It is legacy code, and in fact used to be K&R C until a few years ago. Would be great if someone spent the time to carefully update it. BTW one thing that makes this harder to resolve is that different files get linked together, and some genuinely use the same variable name for different things and with different types (e.g. space sometimes is an integer and sometimes an array); so one can't just blindly add all of these extern into a single header. No, one has to carefully identify which variables (of which type) are shared by which C files and then ideally add headers for each combo and move them there... To be clear, I won't work on any of these. I am more interested in modern re implementations of these algorithms. Interfacing to this ancient C code via shell scripts is not the best model going forward :-) After inspection, no other gap package seem to depend on it which makes it ripe for deletion. But I would be more confident if upstream did the move too. At this point the C code is abandonware unless someone step up (not me). The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=9c80e23c9c0782ed7e21532ecbc6d25323726e14 commit 9c80e23c9c0782ed7e21532ecbc6d25323726e14 Author: Michael Orlitzky <mjo@gentoo.org> AuthorDate: 2025-05-29 02:20:44 +0000 Commit: Michael Orlitzky <mjo@gentoo.org> CommitDate: 2025-05-29 02:29:10 +0000 profiles: mask dev-gap/cohomolo for removal Bug: https://bugs.gentoo.org/940232 Signed-off-by: Michael Orlitzky <mjo@gentoo.org> profiles/package.mask | 7 +++++++ 1 file changed, 7 insertions(+) |