Bug 2305344 - Errors in sagemath caused by cddlib
Summary: Errors in sagemath caused by cddlib
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: cddlib
Version: 40
Hardware: x86_64
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Jerry James
QA Contact: Fedora Extras Quality Assurance
URL: https://groups.google.com/g/sage-rele...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2024-08-16 15:55 UTC by artal
Modified: 2024-08-28 02:36 UTC (History)
2 users (show)

Fixed In Version: cddlib-0.94m-19.fc40
Clone Of:
Environment:
Last Closed: 2024-08-28 02:36:28 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description artal 2024-08-16 15:55:10 UTC
If sagemath is built in Fedora 40 using system cddlib some errors arise. These errors do not happen if the sagemath package is used, both are in the same version

Reproducible: Always

Steps to Reproduce:
1.Build sagemath with system cddlib
2.Test src/sage/rings/polynomial/multi_polynomial_ideal.py and src/sage/rings/polynomial/groebner_fan.py
Actual Results:  
Failure, see https://groups.google.com/g/sage-release/c/gQjpC6oesRM/m/VJ_-JNv5BwAJ

Expected Results:  
Tests passed

Comment 1 Jerry James 2024-08-16 17:16:13 UTC
The only important difference between the Fedora 39 and 40 cddlib package builds is the addition of this upstream commit:

https://github.com/cddlib/cddlib/commit/1210faf9ee9cf02e29fc67e11a8ea4e847c58ddd

Does sagemath access cddlib from multiple threads and assume they will share data?

Comment 2 artal 2024-08-17 09:45:06 UTC
Thanks for your answer. It is above my knowledge but I transmitted your question in https://groups.google.com/g/sage-release/c/gQjpC6oesRM

In one reply, it was suggested to compare some output (using sagemath with sage package or system package) and there are some differencies.

Comment 3 Jerry James 2024-08-19 15:41:18 UTC
Okay, it seems evident that patch is causing problems, and it doesn't actually fix the problem it was added to address.  I have removed it and done new builds.  It would be great if you could see if the new builds (for F40, F41, and Rawhide) resolve this issue.

Comment 4 Fedora Update System 2024-08-19 15:41:25 UTC
FEDORA-2024-03c543987c (cddlib-0.94m-19.fc40) has been submitted as an update to Fedora 40.
https://bodhi.fedoraproject.org/updates/FEDORA-2024-03c543987c

Comment 5 artal 2024-08-19 18:36:05 UTC
(In reply to Jerry James from comment #3)
> Okay, it seems evident that patch is causing problems, and it doesn't
> actually fix the problem it was added to address.  I have removed it and
> done new builds.  It would be great if you could see if the new builds (for
> F40, F41, and Rawhide) resolve this issue.

Thanks, but apparently it is not the cause of the problems. Still the same errors, but actually I do not know when the problems started. I am trying with previous builds to check.

Comment 6 Fedora Update System 2024-08-20 01:54:10 UTC
FEDORA-2024-03c543987c has been pushed to the Fedora 40 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --refresh --advisory=FEDORA-2024-03c543987c`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2024-03c543987c

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 7 artal 2024-08-21 16:29:47 UTC
Following the discussion in https://groups.google.com/g/sage-release/c/gQjpC6oesRM I can confirm that your change was successful. Thanks!

Comment 8 artal 2024-08-22 07:22:48 UTC
(In reply to artal from comment #7)
> Following the discussion in
> https://groups.google.com/g/sage-release/c/gQjpC6oesRM I can confirm that
> your change was successful. Thanks!

If sagemath is using gfan rpm, it needs to be rebuilt. Anyway, I think the issue has been solved. I am not sure who should close it.

Comment 9 Fedora Update System 2024-08-28 02:36:28 UTC
FEDORA-2024-03c543987c (cddlib-0.94m-19.fc40) has been pushed to the Fedora 40 stable repository.
If problem still persists, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.