May 2011 - mingw - Fedora Mailing-Lists

Proposal: drop the hard dependency on mingw32-gettext from mingw32-glib2

by Erik van Pienbroek

Hi, Recently I was browsing the upstream GTK website and I noticed that upstream has used a special trick to build the Win32 binaries for GLib. In order to make the dependency on gettext a soft one they used a small static wrapper library called libproxy-intl [1]. With this wrapper library the GLib DLL doesn't depend directly anymore on libintl-8.dll (from mingw32-gettext). The wrapper library makes the hard dependency a runtime one. This means that if libintl-8.dll is bundled with the application then gettext translations will be used, otherwise nothing gets translated. I'd like to propose that we apply this change as well in our Fedora MinGW toolchain. Such a change consists of the following smaller changes: - Bundle the libproxy-intl code (1 .c file and 1 .h file) in the mingw32-gettext srpm as additional source files - Compile the libproxy-intl code with these 2 instructions: %{_mingw32_cc} -c libintl.c -o libintl.o -I. %{_mingw32_ar} rc libintl.a libintl.o - Replace the files libintl.{a,.dll.a,.la} generated by the gettext compilation with the libintl.a file from libproxy-intl. This makes any binary which tries to link against mingw32-gettext using '-lintl' have a soft-dependency on libintl. After this a simple rebuild of mingw32-glib2 should be sufficient enough to make mingw32-gettext a soft dependency. Next to glib2 there are also some other packages which have a hard dependency on libintl-8.dll: $ repoquery --whatrequires 'mingw32(libintl-8.dll)' mingw32-atk-0:1.32.0-1.fc14.noarch mingw32-gdk-pixbuf-0:2.22.0-1.fc14.noarch mingw32-glib2-0:2.26.0-1.fc14.noarch mingw32-gnutls-0:2.6.4-3.fc13.noarch mingw32-gtk-vnc-0:0.4.1-1.fc14.noarch mingw32-gtk2-0:2.22.0-1.fc14.noarch mingw32-gtkhtml3-0:3.32.0-1.fc14.noarch mingw32-gvnc-0:0.4.1-1.fc14.noarch mingw32-hunspell-0:1.2.8-11.fc12.noarch mingw32-libglade2-0:2.6.4-4.fc12.noarch mingw32-libidn-0:1.14-5.fc12.noarch Most of these packages (if not all) don't use gettext directly, but get linked against gettext because of it being mentioned in glib2's pkgconfig and .la files. A rebuild should be sufficient to replace the hard dependencies on those packages by soft ones. However, in order to prevent the libproxy-intl functions from being exported in other DLL's (as mentioned on [1]) I think it's better drop the reference to libintl from glib2's pkgconfig and .la files. As most of the packages mentioned above don't use gettext directly, but the GLib gettext functions to handle translations this shouldn't cause any major problems. As the glib2 headers depend on the libintl.h header a 'Requires: mingw32-gettext' also needs to be added to the mingw32-glib2 package. I already applied these changes in a testing repository for a generic cross compiler framework (coming soon!) [2][3] and it works fine there. Another thing GTK upstream has done in their Win32 binaries is using win-iconv instead of GNU iconv. I just put up a review request for mingw32-win-iconv at [4]. Could somebody please review that package? Does everybody agree to these changes? Kind regards, Erik van Pienbroek [1]: http://www.gtk.org/download-windows.html [2]: http://svn.openftd.org/svn/fedora_cross [3]: http://svn.openftd.org/viewvc/Fedora%20Cross%20Compiler% 20Framework/ [4]: https://bugzilla.redhat.com/show_bug.cgi?id=642208

13 years, 1 month

2
5
0 / 0

[Bug 641727] Review Request: mingw32-libffi: portable foreign function interface library for Fedora MinGW

by Red Hat Bugzilla

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=641727 Kalev Lember <kalev(a)smartlink.ee> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kalev(a)smartlink.ee --- Comment #14 from Kalev Lember <kalev(a)smartlink.ee> 2011-05-06 15:21:40 EDT --- *** Bug 702699 has been marked as a duplicate of this bug. *** -- 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.

13 years, 1 month

1
0
0 / 0

[Bug 702699] Review Request: mingw32-libffi - MinGW Windows Foreign Function Interface library

by Red Hat Bugzilla

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=702699 Kalev Lember <kalev(a)smartlink.ee> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CLOSED Resolution| |DUPLICATE Last Closed| |2011-05-06 15:21:40 --- Comment #3 from Kalev Lember <kalev(a)smartlink.ee> 2011-05-06 15:21:40 EDT --- Oh, looks like it's indeed already in. Thanks. *** This bug has been marked as a duplicate of bug 641727 *** -- 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.

13 years, 1 month

1
0
0 / 0

[Bug 702699] Review Request: mingw32-libffi - MinGW Windows Foreign Function Interface library

by Red Hat Bugzilla

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=702699 Thomas Sailer <t.sailer(a)alumni.ethz.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |t.sailer(a)alumni.ethz.ch --- Comment #2 from Thomas Sailer <t.sailer(a)alumni.ethz.ch> 2011-05-06 15:13:21 EDT --- What is exactly the difference compared with: https://bugzilla.redhat.com/show_bug.cgi?id=641727 -- 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.

13 years, 1 month

1
0
0 / 0

[Bug 673784] Rename Request: mingw32-filesystem -> mingw-filesystem - Cross compiler base filesystem and environment

by Red Hat Bugzilla

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=673784 Erik van Pienbroek <erik-fedora(a)vanpienbroek.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs? --- Comment #30 from Erik van Pienbroek <erik-fedora(a)vanpienbroek.nl> 2011-05-06 14:31:42 EDT --- New Package SCM Request ======================= Package Name: mingw-filesystem Short Description: MinGW cross compiler base filesystem and environment Owners: epienbro rjones kalev Branches: InitialCC: -- 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.

13 years, 1 month

1
0
0 / 0

[Bug 702699] Review Request: mingw32-libffi - MinGW Windows Foreign Function Interface library

by Red Hat Bugzilla

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=702699 Kalev Lember <kalev(a)smartlink.ee> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fedora-mingw(a)lists.fedorapr | |oject.org --- Comment #1 from Kalev Lember <kalev(a)smartlink.ee> 2011-05-06 12:03:23 EDT --- libffi is a required dependency for building glib 2.29.4. Scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3054790 -- 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.

13 years, 1 month

1
0
0 / 0

[Bug 673784] Rename Request: mingw32-filesystem -> mingw-filesystem - Cross compiler base filesystem and environment

by Red Hat Bugzilla

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=673784 Michael Cronenworth <mike(a)cchtml.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-review+ --- Comment #29 from Michael Cronenworth <mike(a)cchtml.com> 2011-05-06 11:48:32 EDT --- Scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3054732 Rpmlint: $ rpmlint mingw-filesystem mingw-filesystem-scripts mingw32-filesystem mingw64-filesystem mingw-filesystem.noarch: W: obsolete-not-provided cross-filesystem mingw-filesystem.noarch: W: no-documentation mingw-filesystem-scripts.noarch: W: obsolete-not-provided cross-filesystem-scripts mingw-filesystem-scripts.noarch: W: only-non-binary-in-usr-lib mingw-filesystem-scripts.noarch: E: incorrect-fsf-address /usr/libexec/mingw-scripts mingw32-filesystem.noarch: W: no-documentation mingw32-filesystem.noarch: W: dangling-symlink /usr/bin/mingw32-qmake-qt4 /usr/libexec/mingw-scripts mingw32-filesystem.noarch: W: dangling-symlink /usr/bin/mingw32-cmake /usr/libexec/mingw-scripts mingw32-filesystem.noarch: W: dangling-symlink /usr/bin/mingw32-configure /usr/libexec/mingw-scripts mingw32-filesystem.noarch: W: dangling-symlink /usr/bin/i686-w64-mingw32-pkg-config /usr/libexec/mingw-scripts mingw32-filesystem.noarch: W: dangling-symlink /usr/bin/mingw32-pkg-config /usr/libexec/mingw-scripts mingw32-filesystem.noarch: W: dangling-symlink /usr/bin/mingw32-make /usr/libexec/mingw-scripts mingw32-filesystem.noarch: W: no-manual-page-for-binary mingw32-make mingw32-filesystem.noarch: W: no-manual-page-for-binary i686-w64-mingw32-pkg-config mingw32-filesystem.noarch: W: no-manual-page-for-binary mingw32-qmake-qt4 mingw32-filesystem.noarch: W: no-manual-page-for-binary mingw32-cmake mingw32-filesystem.noarch: W: no-manual-page-for-binary mingw32-configure mingw32-filesystem.noarch: W: no-manual-page-for-binary mingw32-pkg-config mingw64-filesystem.noarch: W: no-documentation mingw64-filesystem.noarch: W: dangling-symlink /usr/bin/mingw64-qmake-qt4 /usr/libexec/mingw-scripts mingw64-filesystem.noarch: W: dangling-symlink /usr/bin/mingw64-configure /usr/libexec/mingw-scripts mingw64-filesystem.noarch: W: dangling-symlink /usr/bin/mingw64-make /usr/libexec/mingw-scripts mingw64-filesystem.noarch: W: dangling-symlink /usr/bin/mingw64-pkg-config /usr/libexec/mingw-scripts mingw64-filesystem.noarch: W: dangling-symlink /usr/bin/mingw64-cmake /usr/libexec/mingw-scripts mingw64-filesystem.noarch: W: dangling-symlink /usr/bin/x86_64-w64-mingw32-pkg-config /usr/libexec/mingw-scripts mingw64-filesystem.noarch: W: no-manual-page-for-binary mingw64-pkg-config mingw64-filesystem.noarch: W: no-manual-page-for-binary mingw64-make mingw64-filesystem.noarch: W: no-manual-page-for-binary x86_64-w64-mingw32-pkg-config mingw64-filesystem.noarch: W: no-manual-page-for-binary mingw64-qmake-qt4 mingw64-filesystem.noarch: W: no-manual-page-for-binary mingw64-cmake mingw64-filesystem.noarch: W: no-manual-page-for-binary mingw64-configure 4 packages and 0 specfiles checked; 1 errors, 30 warnings. Looks like there's still one FSF address issue. The man pages are obvious dupes of native tools so they aren't required. It looks good to me. APPROVED -- 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.

13 years, 1 month

1
0
0 / 0

[Bug 673784] Rename Request: mingw32-filesystem -> mingw-filesystem - Cross compiler base filesystem and environment

by Red Hat Bugzilla

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=673784 --- Comment #28 from Erik van Pienbroek <erik-fedora(a)vanpienbroek.nl> 2011-05-06 11:21:56 EDT --- For the rpmlint folder I didn't use one global %{_sysconfdir}/rpmlint entry as the file which is placed in that folder isn't a regular file, but a %config file. Spec URL: http://svn.openftd.org/svn/fedora_cross/mingw-filesystem/mingw-filesystem... SRPM URL: http://ftd4linux.nl/contrib/mingw-filesystem-71-1.fc15.src.rpm * Fri May 06 2011 Erik van Pienbroek <epienbro(a)fedoraproject.org> - 71-1 - Dropped the BR and R for setup, rpm and rpmlint as they aren't needed anymore - Own the folder %{_sysconfdir}/rpmlint (to avoid dependency on rpmlint) - Sorted the list with mingw32(xxx) and mingw64(xxx) provides for improved readability -- 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.

13 years, 1 month

1
0
0 / 0

[Bug 673784] Rename Request: mingw32-filesystem -> mingw-filesystem - Cross compiler base filesystem and environment

by Red Hat Bugzilla

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=673784 --- Comment #27 from Michael Cronenworth <mike(a)cchtml.com> 2011-05-06 08:47:02 EDT --- (In reply to comment #26) > +%dir %{_sysconfdir}/rpmlint > %config(noreplace) %{_sysconfdir}/rpmlint/mingw-rpmlint.config Thanks for the help, Kalev. I think for the rpmlint folder simply: %{_sysconfdir}/rpmlint is enough. (no %dir) After the above fixes are applied I'll give it one more review. -- 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.

13 years, 1 month

1
0
0 / 0

[Bug 673784] Rename Request: mingw32-filesystem -> mingw-filesystem - Cross compiler base filesystem and environment

by Red Hat Bugzilla

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=673784 --- Comment #26 from Kalev Lember <kalev(a)smartlink.ee> 2011-05-06 07:07:37 EDT --- The Provides: mingw32() and Provides: mingw64() would probably look better in alphabetical order. Sorry for not noticing it earlier, but the Requires: setup, rpm, rpmlint and BuildRequires: rpmlint were all dropped from mingw32-filesystem a while ago: http://pkgs.fedoraproject.org/gitweb/?p=mingw32-filesystem.git;a=commit;h... http://pkgs.fedoraproject.org/gitweb/?p=mingw32-filesystem.git;a=commit;h... I think the rationale for NOT Requiring 'setup' and 'rpm' was that these are all so low level packages that they are in all Fedora installations and therefore it's not worth requiring them, similar to not requiring the 'filesystem' package. With rpmlint there's a different story. Rpmlint used to be required dependency because we install a file in /etc/rpmlint/. However, directory ownership guidelines were updated a while ago and now it's perfectly fine to have multiple directory ownership on /etc/rpmlint/ in order to avoid needlessly dragging in the rpmlint package. http://fedoraproject.org/wiki/PackagingGuidelines#The_directory_is_owned_... I'd suggest the following changes: -BuildRequires: rpmlint >= 0.85-2 -Requires: setup -Requires: rpm -Requires: rpmlint >= 0.85-2 +%dir %{_sysconfdir}/rpmlint %config(noreplace) %{_sysconfdir}/rpmlint/mingw-rpmlint.config -- 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.

13 years, 1 month

1
0
0 / 0

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

mingw May 2011