Bug 1488245

Summary: Review Request: mingw-Cython - MinGW Windows Python cython library
Product: [Fedora] Fedora Reporter: Sandro Mani <manisandro>
Component: Package ReviewAssignee: Robert-André Mauchin 🐧 <eclipseo>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: eclipseo, package-review
Target Milestone: ---Flags: eclipseo: fedora-review+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-09-07 18:29:59 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1488242    
Bug Blocks: 1488240    

Description Sandro Mani 2017-09-04 19:32:06 UTC
Spec URL: https://smani.fedorapeople.org/review/mingw-Cython.spec
SRPM URL: https://smani.fedorapeople.org/review/mingw-cython-0.25.2-1.fc28.src.rpm
Description: MinGW Windows Python cython library
Fedora Account System Username: smani

Comment 1 Robert-André Mauchin 🐧 2017-09-05 07:56:09 UTC
The build fails because there's no debugging symbols produced. Even with adding CFLAGS it doesn't work. I removed  %{?mingw_debug_package} to produce the build.
I don't know if you have another solution.

Besides that the package is sound. I'll accept it after your answer regarding the debugging issue.

Comment 2 Sandro Mani 2017-09-05 08:07:46 UTC
Do you have redhat-rpm-config installed? The build works fine in copr [1].

[1] https://copr.fedorainfracloud.org/coprs/smani/mingw-extras/build/597420/

Comment 3 Robert-André Mauchin 🐧 2017-09-05 08:51:03 UTC
Yeah I have it installed. I have the same issue with mingw-numpy too. If it works in copr and Koji it should be good, so I accept the package.

Comment 4 Robert-André Mauchin 🐧 2017-09-05 10:32:25 UTC
*** Bug 1488238 has been marked as a duplicate of this bug. ***

Comment 5 Gwyn Ciesla 2017-09-05 12:39:39 UTC
(fedrepo-req-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/mingw-Cython