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=530047
Paulo Roma Cavalcanti <promac(a)gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |CLOSED
Resolution| |NEXTRELEASE
Last Closed|2010-12-17 12:50:29 |2010-12-24 11:51:34
--
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.
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=530047
--- Comment #9 from Jason Tibbitts <tibbs(a)math.uh.edu> 2010-12-22 23:45:32 EST ---
Git done (by process-git-requests).
--
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.
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=530047
Paulo Roma Cavalcanti <promac(a)gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Flag| |fedora-cvs?
--- Comment #8 from Paulo Roma Cavalcanti <promac(a)gmail.com> 2010-12-22 15:44:01 EST ---
Thanks, Jason, for finishing this review.
New Package SCM Request
=======================
Package Name: mingw32-tk
Short Description: MinGW Windows graphical toolkit for the Tcl scripting
language
Owners: roma
Branches: f13 f14 el6
InitialCC:roma
--
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.
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=530047
Jason Tibbitts <tibbs(a)math.uh.edu> changed:
What |Removed |Added
----------------------------------------------------------------------------
Keywords|Reopened |
Blocks|201449(FE-DEADREVIEW) |
AssignedTo|nobody(a)fedoraproject.org |tibbs(a)math.uh.edu
Flag| |fedora-review+
--- Comment #7 from Jason Tibbitts <tibbs(a)math.uh.edu> 2010-12-22 14:12:40 EST ---
Builds fine; just one rpmlint complaint:
mingw32-tk.src:81: W: macro-in-comment %{name1}
which doesn't seem to be a problem to me. There are no comments in a spec
file, so macros are still expanded and if they have multiple lines the behavior
isn't what you expect.
Everything else looks fine.
* source files match upstream. sha256sum:
f46b33d41e71ef533140f1814961f0dc22d62aac316df4a4fd3357e1fd788b83
tk8.5.9-src.tar.gz
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK.
* dist tag is present.
* license field matches the actual license.
* license is open source-compatible.
* license text included in package.
* version packaged is the same as the native Fedora version.
* package is noarch.
* BuildRequires are proper.
* compiler flags are appropriate.
* package builds in mock (rawhide, x86_64).
* package installs properly.
* rpmlint has acceptable complaints.
* special mingw32 dependency generators are used.
* final provides and requires are sane:
mingw32(tk85.dll)
mingw32-tk = 8.5.9-1.fc15
=
mingw32-filesystem >= 63
mingw32-runtime
mingw32(gdi32.dll)
mingw32(kernel32.dll)
mingw32(msvcrt.dll)
mingw32(ole32.dll)
mingw32(tcl85.dll)
mingw32(tk85.dll)
mingw32(user32.dll)
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no generically named files
* code, not content.
* documentation is small, so no -doc subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
* no static libraries.
* .dll and .def files in _mingw32_bindir
* .dll.a and .la files in _mingw32_libdir
* dll files are listed explicitly in %files.
* libraries and executables are stripped properly.
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.
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=634621
Andrew Beekhof <andrew(a)beekhof.net> changed:
What |Removed |Added
----------------------------------------------------------------------------
Summary|Review Request: |Review Request:
|mingw32-qpid-cpp - MinGW |mingw32-qpid-cpp - MinGW
|Windows port of AMQP C++ |Windows AMQP daemons and
|Daemons and Librarie |libraries
Flag| |fedora-cvs?
--- Comment #6 from Andrew Beekhof <andrew(a)beekhof.net> 2010-12-21 17:04:46 EST ---
New Package SCM Request
=======================
Package Name: mingw32-qpid-cpp
Short Description: MinGW Windows AMQP daemons and libraries
Owners: beekhof
Branches: f14 devel
InitialCC: tross
--
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.
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=634621
Lon Hohberger <lhh(a)redhat.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Flag|fedora-review? |fedora-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.
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=634621
--- Comment #5 from Lon Hohberger <lhh(a)redhat.com> 2010-12-21 15:36:00 EST ---
Looks like there is a problem with my permissions; I can't set fedora-review+.
While I chase this down, I'll try to get someone else to set it for me.
--
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.
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=634621
--- Comment #4 from Lon Hohberger <lhh(a)redhat.com> 2010-12-21 15:32:43 EST ---
http://fedoraproject.org/wiki/Packaging/ReviewGuidelines
Following these guidelines, the spec file and package pass based on the source
RPM and spec file posted in comment #2 (after subsequent minor revision to the
spec file).
--
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.
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=634621
Lon Hohberger <lhh(a)redhat.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Flag| |fedora-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.
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=634621
Lon Hohberger <lhh(a)redhat.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Group| |windriver
--- Comment #3 from Lon Hohberger <lhh(a)redhat.com> 2010-12-21 15:29:42 EST ---
[lhh@localhost Downloads]$ rpmlint mingw32-qpid-cpp.spec
0 packages and 1 specfiles checked; 0 errors, 0 warnings.
--
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.