[Bug 546998] New: Feature Request: Add mingw32-libgomp package

[Bug 523224] Review Request:...

rpms/mingw32-gcc/devel...

Red Hat Bugzilla

Saturday, 12 December 2009 Sat, 12 Dec '09

10:23 p.m.

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Feature Request: Add mingw32-libgomp package https://bugzilla.redhat.com/show_bug.cgi?id=546998 Summary: Feature Request: Add mingw32-libgomp package Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: mingw32-gcc AssignedTo: rjones(a)redhat.com ReportedBy: chris(a)cnpbagwell.com QAContact: extras-qa(a)fedoraproject.org CC: berrange(a)redhat.com, rjones(a)redhat.com, kalev(a)smartlink.ee, fedora-mingw(a)lists.fedoraproject.org Classification: Fedora The current mingw32-gcc package distributed by fedora has some base support for libgomp. That is it does understand the -fopenmp flag but it will error out since it can not find libgomp support files. The gcc package supports OpenMP on mingw platform via the libgomp library; included in gcc source. Its configure script disables it by default on that platform but can easily be enabled with --enable-libgomp flag. I am maintainer of SoX program which makes use of OpenMP. I am attempting to switch to Fedora for building mingw binaries since its much faster then same process under windows. libgomp support is only thing missing for me to be able to do this. To help me do this, I have updated the existing mingw32-gcc.spec file to enable libgomp support and create a new mingw32-libgomp package (parallel to libgomp package already in Fedora). The main item I'd like someone to review is that gcc package installs libgomp-1.dll under /usr/lib/gcc path but I think it better lives under /usr/i686-pc-mingw/sys-root since mingw apps that link in libgomp need access to that DLL at run time. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

Show replies by date

Red Hat Bugzilla

Saturday, 12 December Sat, 12 Dec

10:27 p.m.

New subject: [Bug 546998] Feature Request: Add mingw32-libgomp package

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=546998 --- Comment #1 from Chris Bagwell <chris(a)cnpbagwell.com> 2009-12-12 23:27:51 EDT --- Addition item I'm not sure about. If you link a mingw application to libgomp, it will fail to link unless you also link in pthread library. Should mingw32-pthreads be added as a requirement to install mingw32-libgomp? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

Red Hat Bugzilla

Monday, 14 December Mon, 14 Dec

5:12 a.m.

New subject: [Bug 546998] Feature Request: Add mingw32-libgomp package

...

To help me do this, I have updated the existing mingw32-gcc.spec file to enable libgomp support and create a new mingw32-libgomp package (parallel to libgomp package already in Fedora).

Can you attach the diffs for the changes you made to mingw32-gcc.spec? Also, I don't understand - why is there an existing mingw32-libgomp package, and how does it fit in with this? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

Red Hat Bugzilla

8:25 a.m.

New subject: [Bug 546998] Feature Request: Add mingw32-libgomp package

...

Can you attach the diffs for the changes you made to mingw32-gcc.spec?

I've attached diff compared to Fedora 12's mingw32-gcc.spec file (sorry, haven't check if rawhide is different yet). This makes it easy to see they are minor changes. I biggest part is I could no longer use a *.h since I needed to move one newly installed header file to the mingw32-libgomp package.

...

Also, I don't understand - why is there an existing mingw32-libgomp package, and how does it fit in with this?

Sorry for confusion. I was attempting to say that the non-mingw32 gcc.spec file will create gcc-* packages and a libgomp package. I paralleled that and added a mingw32-libgomp package to go along with the mingw32-gcc-* packages already being created by mingw32-gcc.spec. On POSIX platform's, gcc's configure will use --enable-libgomp by default. It works on mingw32 platform though but just needs to be enabled during compile. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

Red Hat Bugzilla

8:28 a.m.

New subject: [Bug 546998] Feature Request: Add mingw32-libgomp package

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=546998 --- Comment #4 from Chris Bagwell <chris(a)cnpbagwell.com> 2009-12-14 09:28:09 EDT --- Created an attachment (id=378221) --> (https://bugzilla.redhat.com/attachment.cgi?id=378221) create mingw32-libgomp -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

Red Hat Bugzilla

8:42 a.m.

New subject: [Bug 546998] Feature Request: Add mingw32-libgomp package

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=546998 --- Comment #5 from Chris Bagwell <chris(a)cnpbagwell.com> 2009-12-14 09:42:48 EDT --- Created an attachment (id=378229) --> (https://bugzilla.redhat.com/attachment.cgi?id=378229) Updated diff. Reviewing first diff, I noticed I mistakenly left in the old include/*.h line. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

Red Hat Bugzilla

9:17 a.m.

New subject: [Bug 546998] Feature Request: Add mingw32-libgomp package

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=546998 --- Comment #6 from Richard W.M. Jones <rjones(a)redhat.com> 2009-12-14 10:17:20 EDT --- Chris - a few friendly tips for future reference: Use unified diffs (diff -u) since they are much easier to review. Mark old attachments obsolete in Bugzilla. Mark patches as "[X] patch" in Bugzilla, otherwise it wants to download them. ----------------- Patch looks good to me. Erik, Kalev, any objections? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

Red Hat Bugzilla

9:18 a.m.

New subject: [Bug 546998] Feature Request: Add mingw32-libgomp package

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=546998 Richard W.M. Jones <rjones(a)redhat.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #378221|application/octet-stream |text/plain mime type| | Attachment #378221|0 |1 is patch| | Attachment #378221|0 |1 is obsolete| | --- Comment #7 from Richard W.M. Jones <rjones(a)redhat.com> 2009-12-14 10:18:27 EDT --- (From update of attachment 378221) Mark earlier patch as obsolete, patch. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

Red Hat Bugzilla

9:18 a.m.

New subject: [Bug 546998] Feature Request: Add mingw32-libgomp package

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=546998 Richard W.M. Jones <rjones(a)redhat.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #378229|application/octet-stream |text/plain mime type| | Attachment #378229|0 |1 is patch| | --- Comment #8 from Richard W.M. Jones <rjones(a)redhat.com> 2009-12-14 10:18:43 EDT --- (From update of attachment 378229) Mark latest patch as patch. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

Red Hat Bugzilla

10:06 a.m.

New subject: [Bug 546998] Feature Request: Add mingw32-libgomp package

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=546998 --- Comment #9 from Kalev Lember <kalev(a)smartlink.ee> 2009-12-14 11:06:26 EDT --- I've got a few comments. 1) The subpackage which has libgomp-1.dll probably also needs to get manual provides because the automatic dependency scripts aren't run: Provides: mingw32(libgomp-1.dll) 2) %{_libdir}/gcc/i686-pc-mingw32/%{version}/finclude/ directory looks unowned. 3) In native gcc package finclude/ is part of gfortran subpackage, and include/omp.h is part of main gcc package. Why did you choose to include these under mingw32-libgomp instead? (I'm not saying it's bad, just trying to explore different options.) 4) Does the libgomp.spec which gets installed in {_libdir}/gcc/i686-pc-mingw32/%{version}/ really belong in there? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

Red Hat Bugzilla

10:17 a.m.

New subject: [Bug 546998] Feature Request: Add mingw32-libgomp package

...

4) Does the libgomp.spec which gets installed in {_libdir}/gcc/i686-pc-mingw32/%{version}/ really belong in there?

Never mind that comment. I was confusing gcc spec files with rpm spec files for a moment. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

Red Hat Bugzilla

12:05 p.m.

New subject: [Bug 546998] Feature Request: Add mingw32-libgomp package

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=546998 Erik van Pienbroek <erik-fedora(a)vanpienbroek.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |erik-fedora(a)vanpienbroek.nl --- Comment #11 from Erik van Pienbroek <erik-fedora(a)vanpienbroek.nl> 2009-12-14 13:05:26 EDT --- (In reply to comment #6)

...

Patch looks good to me. Erik, Kalev, any objections?

The patch is against F-12's version of mingw32-gcc. In rawhide we already have a newer version (mingw32-gcc-4_4_2-1_fc13). Please post a unified diff against the rawhide version of mingw32-gcc. The .spec can be found at http://cvs.fedoraproject.org/viewvc/rpms/mingw32-gcc/devel/mingw32-gcc.sp... In addition to Kalev's remarks I also discovered something: The %package section is missing a Requires: %{name} = %{version}-%{release} Kalev, perhaps this would also be a good opportunity to enable DWARF2 exceptions which we were already planning to do anyway. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

Red Hat Bugzilla

Tuesday, 15 December Tue, 15 Dec

11:24 a.m.

New subject: [Bug 546998] Feature Request: Add mingw32-libgomp package

...

I've got a few comments. 1) The subpackage which has libgomp-1.dll probably also needs to get manual provides because the automatic dependency scripts aren't run: Provides: mingw32(libgomp-1.dll)

Makes sense. But should it be the package name? I'll add once its confirmed. Provides: mingw32-libgomp(libgomp-1.dll)

...

2) %{_libdir}/gcc/i686-pc-mingw32/%{version}/finclude/ directory looks unowned.

OK, will fix.

...

3) In native gcc package finclude/ is part of gfortran subpackage, and include/omp.h is part of main gcc package. Why did you choose to include these under mingw32-libgomp instead? (I'm not saying it's bad, just trying to explore different options.)

This was a judgment call that I'll probably revert but would like opinions first. Since libgomp has headers for C and fortran *and* is optional, some choices have to be made on where to package them. Options: 1) Include both types of headers with mingw32-gcc package which means you can see C header file omp.h even when libgomp is not installed or fortran files when fortran compile is not installed. 2) Include both of them in mingw32-libgomp which has nice side affect that gcc won't see either header nor library when no libgomp libraries installed but it can mean fortran headers installed without fortran compiler. 3) Break into a mingw32-libgomp and mingw32-libgomp-fortran with appropriate Requires. gcc.spec chose #1 which I find strange in that my configure scripts have to check for both header file omp.h and additionally do a -lgomp test to verify its really supported; when often the short cut of just looking for header file is enough. So I chose #2 to prevent gcc from seeing header file which I left slightly better. I already have to deal with issues from #1 because of i686-redhat-linux-gcc compilers, so for consistency I should probably chose #1 for i686-pc-mingw32-gcc. Agree?

...

4) Does the libgomp.spec which gets installed in {_libdir}/gcc/i686-pc-mingw32/%{version}/ really belong in there?

Ignoring this item as requested in later response. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

Red Hat Bugzilla

11:48 a.m.

New subject: [Bug 546998] Feature Request: Add mingw32-libgomp package

...

(In reply to comment #6) > Patch looks good to me. Erik, Kalev, any objections? The patch is against F-12's version of mingw32-gcc. In rawhide we already have a newer version (mingw32-gcc-4_4_2-1_fc13). Please post a unified diff against the rawhide version of mingw32-gcc. The .spec can be found at http://cvs.fedoraproject.org/viewvc/rpms/mingw32-gcc/devel/mingw32-gcc.sp...

I will do soon. I'll attempt to upgrade compiler to rawhide on my Fedora 12 box.

...

In addition to Kalev's remarks I also discovered something: The %package section is missing a Requires: %{name} = %{version}-%{release}

Make sense and I can add in next revision. Is this also a gcc.spec issue as well and if so should one of us bugzilla it? And to be safe, since I'm using "package -n mingw32-libgomp", will %{name} get converted to mingw32-gcc or mingw32-libgomp within that block or is it always fixed to "Name:" value? Chris -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

Red Hat Bugzilla

Wednesday, 16 December Wed, 16 Dec

10:35 a.m.

New subject: [Bug 546998] Feature Request: Add mingw32-libgomp package

...

I already have to deal with issues from #1 because of i686-redhat-linux-gcc compilers, so for consistency I should probably chose #1 for i686-pc-mingw32-gcc. Agree?

There's one more option: kill the mingw32-libgomp subpackage and include header files and dlls directly in the mingw32-gcc package. As I understand it, the reason why gcc has headers + .so symlinks in gcc package and .so.* files in libgomp is to split development files and runtime files. For development you'd install gcc, and get all the headers and other stuff needed to link against the library. End users, however, would only get the libgomp package, which contains the shared lib (no development stuff in this package). For mingw32 packages, splitting development and runtime in this way doesn't make much sense. All mingw32- packages are meant for development. What I'd suggest is to include the dll and import libs in mingw32-gcc package. mingw32-fortran would get the finclude/ directory, and rest of the additional headers files would go in mingw32-gcc. This setup would also reduce the complexity of the .spec file, where you are currently listing lots of header files. Those could probably be just replaced with a simple *.h glob. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

Red Hat Bugzilla

2:41 p.m.

New subject: [Bug 546998] Feature Request: Add mingw32-libgomp package

...

(In reply to comment #12) > I already have to deal with issues from #1 because of i686-redhat-linux-gcc > compilers, so for consistency I should probably chose #1 for > i686-pc-mingw32-gcc. Agree? There's one more option: kill the mingw32-libgomp subpackage and include header files and dlls directly in the mingw32-gcc package.

Rational makes sense to me and solve multiple problems. I'll attach updated diff that gets rid of libgomp subpackage. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

Red Hat Bugzilla

2:46 p.m.

New subject: [Bug 546998] Feature Request: Add mingw32-libgomp package

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=546998 --- Comment #16 from Chris Bagwell <chris(a)cnpbagwell.com> 2009-12-16 15:46:38 EDT --- Created an attachment (id=378841) --> (https://bugzilla.redhat.com/attachment.cgi?id=378841) Updated diff to rawhide. - Update unified diff to be based on current rawhide. - Fix missing Provides: for libgomp-1.dll. - Add missing %dir for finclude. - Fold mingw32-libgomp subpackage into base mingw32-gcc to remove unneeded complications. - Move fortran headers to mingw32-gcc-gfortran subpackage. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

Red Hat Bugzilla

2:46 p.m.

New subject: [Bug 546998] Feature Request: Add mingw32-libgomp package

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=546998 Chris Bagwell <chris(a)cnpbagwell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #378229|0 |1 is obsolete| | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

Red Hat Bugzilla

Thursday, 17 December Thu, 17 Dec

1:45 p.m.

New subject: [Bug 546998] Feature Request: Add mingw32-libgomp package

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=546998 Kalev Lember <kalev(a)smartlink.ee> changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|rjones(a)redhat.com |kalev(a)smartlink.ee --- Comment #17 from Kalev Lember <kalev(a)smartlink.ee> 2009-12-17 14:45:23 EDT --- Allright, mingw32-gcc is now building for rawhide with your patch. Please give it a try when it finishes: http://koji.fedoraproject.org/koji/taskinfo?taskID=1878251 I also had to add 'BuildRequires: mingw32-pthreads' so it'd build, and 'Requires: mingw32-pthreads' because libgomp-1.dll is linked against pthreads, but it doesn't get picked up automatically since we aren't running the automatic dependency scripts within this package. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

Red Hat Bugzilla

3:28 p.m.

New subject: [Bug 546998] Feature Request: Add mingw32-libgomp package

...

Allright, mingw32-gcc is now building for rawhide with your patch. Please give it a try when it finishes:

Downloaded and tried. No issues found in my Fedora 12 environment. Thank you all for help with spec file and incorporating into rawhide! -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

Red Hat Bugzilla

3:39 p.m.

New subject: [Bug 546998] Feature Request: Add mingw32-libgomp package

5254

days inactive

5258

days old

mingw@lists.fedoraproject.org

Manage subscription

20 comments

1 participants

tags (0)

participants (1)

Red Hat Bugzilla

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

[Bug 546998] New: Feature Request: Add mingw32-libgomp package