[Bug 226215] Merge Review: openobex

2007-12-07 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: openobex


https://bugzilla.redhat.com/show_bug.cgi?id=226215


[EMAIL PROTECTED] changed:

   What|Removed |Added

Version|devel   |rawhide




--- Additional Comments From [EMAIL PROTECTED]  2007-12-07 16:33 EST ---
man-pages-it-2.65-6.fc8 has been pushed to the Fedora 8 testing repository.  If 
problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update man-pages-it'

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 226215] Merge Review: openobex

2007-10-29 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: openobex


https://bugzilla.redhat.com/show_bug.cgi?id=226215





--- Additional Comments From [EMAIL PROTECTED]  2007-10-29 05:22 EST ---
Should be fixed and consistent in all actual branches (F7, F8, devel). Thank you
for reviewing.

jirka

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 226215] Merge Review: openobex

2007-10-29 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: openobex


https://bugzilla.redhat.com/show_bug.cgi?id=226215


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2007-10-29 13:12 EST ---
The F-7 branch still has License: GPL, different source URL and the unneeded
0: in the autoconf dependency, but fixing those is not obviously required per
this review as the F-8 and devel branches look ok.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 226215] Merge Review: openobex

2007-10-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: openobex


https://bugzilla.redhat.com/show_bug.cgi?id=226215


[EMAIL PROTECTED] 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 the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 226215] Merge Review: openobex

2007-10-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: openobex


https://bugzilla.redhat.com/show_bug.cgi?id=226215





--- Additional Comments From [EMAIL PROTECTED]  2007-10-28 16:13 EST ---
Created an attachment (id=240941)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=240941action=view)
Remaining cosmetic fixes

Ok, devel branch looks good now, patch for some remaining cosmetic issues
attached.  Approved with it applied.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 226215] Merge Review: openobex

2007-10-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: openobex


https://bugzilla.redhat.com/show_bug.cgi?id=226215





--- Additional Comments From [EMAIL PROTECTED]  2007-10-25 15:35 EST ---
Ok, looks better now.  However there's now branch confusion as the F-8 and devel
branches have changes (ipv6, source URL, License tag) that are not included in
the F-7 branch but on the other hand lack fixes in the F-7 one (cleanups,
libusb.pc fix).  And the 1.3-6%{?dist} in F-7 and F-8+ are quite different.

If the current F-7 package fixes were ported to the devel branch and then the
whole shebang back to earlier branches from devel, I think we'd be good to go. 
Let me know when one/some of the branches have all the fixes and I'll take one
more look at that.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 226215] Merge Review: openobex

2007-10-24 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: openobex


https://bugzilla.redhat.com/show_bug.cgi?id=226215





--- Additional Comments From [EMAIL PROTECTED]  2007-10-24 04:08 EST ---
Hi, it should be fixed only in F-7 branch. I will fix it in all other branches
when you'll check it.

Thank you

jirka

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 226215] Merge Review: openobex

2007-10-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: openobex


https://bugzilla.redhat.com/show_bug.cgi?id=226215





--- Additional Comments From [EMAIL PROTECTED]  2007-10-23 09:32 EST ---
Hi, I fixed the errors above, so could someone please re-check it?

Thank you

jirka

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 226215] Merge Review: openobex

2007-10-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: openobex


https://bugzilla.redhat.com/show_bug.cgi?id=226215





--- Additional Comments From [EMAIL PROTECTED]  2007-10-23 14:50 EST ---
I don't see any of the raised issues fixed in CVS.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 226215] Merge Review: openobex

2007-10-15 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: openobex


https://bugzilla.redhat.com/show_bug.cgi?id=226215


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-10-15 03:57 EST ---
adding the new owner of openobex to CC

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 226215] Merge Review: openobex

2007-10-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: openobex


https://bugzilla.redhat.com/show_bug.cgi?id=226215


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium
Product|Fedora Extras   |Fedora




--- Additional Comments From [EMAIL PROTECTED]  2007-10-13 11:03 EST ---
Ping?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 226215] Merge Review: openobex

2007-03-24 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: openobex


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226215





--- Additional Comments From [EMAIL PROTECTED]  2007-03-24 10:56 EST ---
Created an attachment (id=150826)
 -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=150826action=view)
Fix pkgconfig path on lib64 archs

- libusb.pc is searched from a wrong dir on lib64 archs by configure (see
patch), resulting in missing Requires: libusb from the .pc file

- BuildRequires and ExcludeArch for -apps seem to be dupes, they're already in
the main package

- autoreconf would be better placed in %prep

- configuring with --disable-dependency-tracking would clean up the build
output and possibly speed the build up a bit

- Summaries and descriptions could be improved quite a bit.  The description
for -devel is actually wrong.

- make -C doc does not actually seem to build anything, and (unsurprisingly)
its results aren't included in any subpackages.  Perhaps build the HTML doc
(make -C doc html) and include it in -devel, or don't build any of the docs,
and drop the make -C doc as well as docbook related build dependencies?

- Including all AUTHORS etc %docs in all subpackages doesn't look too useful,
perhaps drop them altogether from -apps and -devel since they require the main
package anyway and seem to contain only generic info about the software which
is probably best kept in the main package?

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 226215] Merge Review: openobex

2007-03-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: openobex


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226215


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |MODIFIED




--- Additional Comments From [EMAIL PROTECTED]  2007-03-23 09:07 EST ---
right

please check openobex-1.3-5.fc7

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 226215] Merge Review: openobex

2007-03-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: openobex


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226215


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|MODIFIED|ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review?




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 226215] Merge Review: openobex

2007-03-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: openobex


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226215





--- Additional Comments From [EMAIL PROTECTED]  2007-03-06 16:23 EST ---
What is the license of this package?  License tag says LGPL, upstream tarball
contains both GPL and LGPL license texts, contains both GPL (irda.h,
obex_connect.h) and LGPL sources, and only the GPL license file is included in
binary packages.  Mixture of GPL and LGPL sources which are all built in would
make all of this fall under the GPL, no?

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review