Bug 979566 - Review Request: mingw-postgresql - MinGW Windows PostgreSQL library
Summary: Review Request: mingw-postgresql - MinGW Windows PostgreSQL library
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Sandro Mani
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-06-28 20:09 UTC by Michael Cronenworth
Modified: 2013-07-30 17:45 UTC (History)
3 users (show)

Fixed In Version: mingw-postgresql-9.2.4-1.fc19
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-07-30 17:45:03 UTC
Type: ---
Embargoed:
manisandro: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Michael Cronenworth 2013-06-28 20:09:52 UTC
Spec URL: http://michael.cronenworth.com/RPMS/mingw-postgresql.spec
SRPM URL: http://michael.cronenworth.com/RPMS/mingw-postgresql-9.2.4-1.fc18.src.rpm
Description: MinGW Windows PostgreSQL library
Fedora Account System Username: mooninite

Comment 1 Sandro Mani 2013-07-14 09:02:40 UTC
Issues:
- Use macros instead of hardcoded path names, i.e.
rm $RPM_BUILD_ROOT/usr/i686-w64-mingw32/sys-root/mingw/bin/*.exe
-> rm $RPM_BUILD_ROOT%{mingw32_bindir}/*.exe

- I would use %{_builddir}/%{buildsubdir} instead of the relative paths when creating the symlinks:
ln -s %{_builddir}/%{buildsubdir}/src/interfaces/libpq/libpqdll.def ./build_win32/src/interfaces/libpq/
etc
(Maybe add a comment on why the links are necessary? Package seems to build also without the links, though I have not verified its functionality)

- Nitpicking: fedora-review says:
  Note: These BR are not needed: gawk
  See: http://fedoraproject.org/wiki/Packaging/Guidelines#Exceptions_2

Rest looks good.

Comment 2 Michael Cronenworth 2013-07-14 16:37:22 UTC
(In reply to Sandro Mani from comment #1)
> (Maybe add a comment on why the links are necessary? Package seems to build
> also without the links, though I have not verified its functionality)

Required to build on F18 at least or you get a build failure:
/usr/bin/i686-w64-mingw32-dlltool: Can't open def file: libpqdll.def
/usr/bin/i686-w64-mingw32-dllwrap: /usr/bin/i686-w64-mingw32-dlltool exited with status 1

>   Note: These BR are not needed: gawk

Copied from postgresql.spec. If you want to file a bug with them be my guest.

New spec: http://michael.cronenworth.com/RPMS/mingw-postgresql.spec
New SRPM: http://michael.cronenworth.com/RPMS/mingw-postgresql-9.2.4-2.fc18.src.rpm

Comment 3 Sandro Mani 2013-07-14 16:52:03 UTC
All ok!

Comment 4 Michael Cronenworth 2013-07-14 17:02:52 UTC
Thank you.

New Package SCM Request
=======================
Package Name: mingw-postgresql
Short Description: MinGW Windows PostgreSQL library
Owners: mooninite
Branches: f18 f19
InitialCC:

Comment 5 Kalev Lember 2013-07-14 17:05:13 UTC
I pointed out on IRC yesterday that the package needs to ship import libraries in the main package to able to link against the DLLs. At least libpq.a is an import library and must go into the main package (and should be renamed to libpq.dll.a); I didn't check what the other .a files are.

Comment 6 Michael Cronenworth 2013-07-14 17:22:44 UTC
I didn't see your comment that they were import libs. I should have verified that they were.

New spec: http://michael.cronenworth.com/RPMS/mingw-postgresql.spec
New SRPM: http://michael.cronenworth.com/RPMS/mingw-postgresql-9.2.4-3.fc18.src.rpm

P.S. You can link against a .dll, but, yes, a .dll.a is more appropriate.

Comment 7 Gwyn Ciesla 2013-07-15 12:42:42 UTC
Git done (by process-git-requests).

Comment 8 Fedora Update System 2013-07-16 00:59:02 UTC
mingw-postgresql-9.2.4-1.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/mingw-postgresql-9.2.4-1.fc19

Comment 9 Fedora Update System 2013-07-16 01:00:07 UTC
mingw-postgresql-9.2.4-1.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/mingw-postgresql-9.2.4-1.fc18

Comment 10 Fedora Update System 2013-07-17 03:05:46 UTC
Package mingw-postgresql-9.2.4-1.fc18:
* should fix your issue,
* was pushed to the Fedora 18 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing mingw-postgresql-9.2.4-1.fc18'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2013-13078/mingw-postgresql-9.2.4-1.fc18
then log in and leave karma (feedback).

Comment 11 Fedora Update System 2013-07-30 17:45:03 UTC
mingw-postgresql-9.2.4-1.fc18 has been pushed to the Fedora 18 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 12 Fedora Update System 2013-07-30 17:45:18 UTC
mingw-postgresql-9.2.4-1.fc19 has been pushed to the Fedora 19 stable repository.


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