[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2010-01-08 Thread 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=494695


Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

   Fixed In Version|0.2.0-2.fc12|0.2.0-4.fc11




-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2010-01-08 Thread 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=494695


Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

   Fixed In Version|0.2.0-4.fc11|0.2.0-4.fc12




-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2010-01-08 Thread 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=494695





--- Comment #40 from Fedora Update System upda...@fedoraproject.org  
2010-01-08 15:00:51 EDT ---
qutim-0.2.0-4.fc12 has been pushed to the Fedora 12 stable repository.  If
problems still persist, please make note of it in this bug report.

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2010-01-08 Thread 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=494695





--- Comment #39 from Fedora Update System upda...@fedoraproject.org  
2010-01-08 14:59:39 EDT ---
qutim-0.2.0-4.fc11 has been pushed to the Fedora 11 stable repository.  If
problems still persist, please make note of it in this bug report.

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2010-01-07 Thread 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=494695





--- Comment #35 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info  
2010-01-07 05:48:45 EDT ---
Aleksey thank you for the bugreport, but qutim already build and imported and
it is not subject of Review Request. Please, for each problems in qutim fill
separate bugs. Meantime I'll look on it. You say what plugins does not loaded
in x86_64?

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2010-01-07 Thread 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=494695





--- Comment #36 from Fedora Update System upda...@fedoraproject.org  
2010-01-07 16:51:01 EDT ---
qutim-0.2.0-2.fc12 has been pushed to the Fedora 12 stable repository.  If
problems still persist, please make note of it in this bug report.

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2010-01-07 Thread 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=494695


Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||0.2.0-2.fc12
 Resolution||ERRATA




-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2010-01-07 Thread 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=494695





--- Comment #37 from Fedora Update System upda...@fedoraproject.org  
2010-01-07 16:54:20 EDT ---
qutim-0.2.0-2.fc11 has been pushed to the Fedora 11 stable repository.  If
problems still persist, please make note of it in this bug report.

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2010-01-07 Thread 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=494695





--- Comment #38 from Fedora Update System upda...@fedoraproject.org  
2010-01-07 20:12:58 EDT ---
qutim-0.2.0-4.fc12 has been submitted as an update for Fedora 12.
http://admin.fedoraproject.org/updates/qutim-0.2.0-4.fc12

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2009-12-30 Thread 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=494695


Aleksey Davydov olexiy.davy...@gmail.com changed:

   What|Removed |Added

 CC||olexiy.davy...@gmail.com




--- Comment #32 from Aleksey Davydov olexiy.davy...@gmail.com  2009-12-30 
09:18:47 EDT ---
Plugins don't work on x86-64. 
Please update plugin directory for x86-64 to /usr/lib64/qutim/

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2009-12-30 Thread 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=494695





--- Comment #33 from Peter Lemenkov lemen...@gmail.com  2009-12-30 11:13:52 
EDT ---
(In reply to comment #32)
 Plugins don't work on x86-64. 
 Please update plugin directory for x86-64 to /usr/lib64/qutim/  

Aleksey, I'm not sure that I can confirm your report - I just checked
qutim-0.2.0-2.fc12.x86_64.rpm, qutim-icq-0.2.0-2.fc12.x86_64.rpm and
qutim-jabber-0.2.0-2.fc12.x86_64.rpm - they all contain /usr/lib64/qutim
directory (even with proper library in case of plugin sub-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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2009-12-30 Thread 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=494695





--- Comment #34 from Aleksey Davydov olexiy.davy...@gmail.com  2009-12-30 
11:43:26 EDT ---
Sorry, I wasn't clear enough.

I mean search path for plugins.

My installation gives me the following when run from command line:
Debug: (/usr/bin/plugins, /usr/lib/qutim, /usr/lib/qutim/plugins,
/usr/PlugIns, /home/nld/.config/qutim/plugins) 
Debug: (, ../share/qutim, /home/nld/.config/qutim) 
Debug: English UnitedStates 
Debug: (en_US, en_us, en) 
Debug: English UnitedStates 
Debug: (en_US, en_us, en) 
Debug: en 
Debug: QTreeView {  } 
Warning: QFSFileEngine::open: No file name specified

The first path should contain also /usr/lib64/qutim/.

Thanks.

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2009-12-21 Thread 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=494695





--- Comment #29 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info  
2009-12-21 03:12:16 EDT ---
Thank you, Kevin.

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2009-12-21 Thread 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=494695





--- Comment #30 from Fedora Update System upda...@fedoraproject.org  
2009-12-21 06:15:42 EDT ---
qutim-0.2.0-2.fc12 has been submitted as an update for Fedora 12.
http://admin.fedoraproject.org/updates/qutim-0.2.0-2.fc12

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2009-12-21 Thread 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=494695


Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|ON_QA




--- Comment #31 from Fedora Update System upda...@fedoraproject.org  
2009-12-21 23:42:15 EDT ---
qutim-0.2.0-2.fc12 has been pushed to the Fedora 12 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 qutim'.  You can provide
feedback for this update here:
http://admin.fedoraproject.org/updates/F12/FEDORA-2009-13435

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2009-12-20 Thread 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=494695


Kevin Fenzi ke...@tummy.com changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #28 from Kevin Fenzi ke...@tummy.com  2009-12-20 22:14:52 EDT ---
We no longer do F-10 branches as it's now end of life. 

otherwise, cvs done.

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2009-12-18 Thread 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=494695


Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #27 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info  
2009-12-18 08:30:34 EDT ---
Peter, thank you very much for the review.

New Package CVS Request
===
Package Name: qutim
Short Description: Multiprotocol (ICQ, Jabber, IRC etc) instant messenger with
modern Qt4 interface
Owners: hubbitus
Branches: F-10 F-11 F-12
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2009-12-16 Thread 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=494695





--- Comment #25 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info  
2009-12-16 14:36:23 EDT ---
Ok, I change multiplatform on Multiprotocol what is more important for IM.
Mention of QT4 have worth on my mind too. Summary rewritten.

Release enumeration changed, some comments stripped.

http://hubbitus.net.ru/rpm/Fedora11/qutim/qutim-0.2.0-2.fc11.src.rpm
http://hubbitus.net.ru/rpm/Fedora11/qutim/qutim.spec

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2009-12-16 Thread 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=494695


Peter Lemenkov lemen...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #26 from Peter Lemenkov lemen...@gmail.com  2009-12-17 02:03:06 
EDT ---
Ok, I don't see any outstanding issues so far, so this package is

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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2009-12-15 Thread 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=494695





--- Comment #20 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info  
2009-12-15 08:59:30 EDT ---
ping?

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2009-12-15 Thread 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=494695





--- Comment #21 from Rex Dieter rdie...@math.unl.edu  2009-12-15 09:06:54 EDT 
---
a small comments regarding package description/summary, imho, it makes little
sense to mention multiplatform or on Qt4, (fedora) users won't care (or
need to know) about what platforms are supported or the toolkit used.

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2009-12-15 Thread 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=494695


Peter Lemenkov lemen...@gmail.com changed:

   What|Removed |Added

  Status Whiteboard|NotReady|




--- Comment #22 from Peter Lemenkov lemen...@gmail.com  2009-12-15 09:44:19 
EDT ---
Cleaning up NotReady, since it is almost ready.

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2009-12-15 Thread 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=494695





--- Comment #23 from Peter Lemenkov lemen...@gmail.com  2009-12-15 09:45:48 
EDT ---
(In reply to comment #21)
 a small comments regarding package description/summary, imho, it makes little
 sense to mention multiplatform or on Qt4, (fedora) users won't care (or
 need to know) about what platforms are supported or the toolkit used.  

I would like to keep at least mention about toolkit, since some users prefer
GTK apps over Qt or vice versa.

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2009-12-15 Thread 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=494695





--- Comment #24 from Rex Dieter rdie...@math.unl.edu  2009-12-15 10:42:01 EDT 
---
OK, but please leave it out of the package summary at least, this could be
something mentionable in the package description.  that's a lesser of 2 evils.

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2009-11-29 Thread 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=494695





--- Comment #18 from Peter Lemenkov lemen...@gmail.com  2009-11-29 12:03:47 
EDT ---
(In reply to comment #17)
 Thank you. All mentioned issue fixed, except one:
  * Also consider removing entirely %doc sections from plugin sub-packages.  
 paths fixed instead, and in plugins packages included its docs (only mrim do
 not have it because it is absent in upstream tarball)
 
 http://koji.fedoraproject.org/koji/taskinfo?taskID=1832695
 
 http://hubbitus.net.ru/rpm/Fedora11/qutim/qutim-0.2.0-1.20.fc11.src.rpm  

Some new issues :)

* In main package you forgot to mark owned directory %{_libdir}/%{name} as %dir
* I believe, that instead of installing small icon to
%{buildroot}/%{_datadir}/icons/hicolor/16x16/ you intended to install to
%{buildroot}/%{_datadir}/icons/hicolor/16x16/apps

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2009-11-29 Thread 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=494695





--- Comment #19 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info  
2009-11-29 16:06:59 EDT ---
(In reply to comment #18)
 * In main package you forgot to mark owned directory %{_libdir}/%{name} as 
 %dir
Oh, yes, thank you.
 * I believe, that instead of installing small icon to
 %{buildroot}/%{_datadir}/icons/hicolor/16x16/ you intended to install to
 %{buildroot}/%{_datadir}/icons/hicolor/16x16/apps  
Fixed.

http://koji.fedoraproject.org/koji/taskinfo?taskID=1836516
http://hubbitus.net.ru/rpm/Fedora11/qutim/qutim-0.2.0-1.21.fc11.src.rpm

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2009-11-26 Thread 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=494695





--- Comment #17 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info  
2009-11-26 16:17:44 EDT ---
Thank you. All mentioned issue fixed, except one:
 * Also consider removing entirely %doc sections from plugin sub-packages.  
paths fixed instead, and in plugins packages included its docs (only mrim do
not have it because it is absent in upstream tarball)

http://koji.fedoraproject.org/koji/taskinfo?taskID=1832695

http://hubbitus.net.ru/rpm/Fedora11/qutim/qutim-0.2.0-1.20.fc11.src.rpm

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2009-11-23 Thread 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=494695





--- Comment #16 from Peter Lemenkov lemen...@gmail.com  2009-11-23 05:49:57 
EDT ---
Still not addressed:

* %global _miconsdir  %{_datadir}/icons/mini
There is no such package in Fedora repositories, which owns this directory.
Also your package doesn't own it. This should be fixed. I believe, that this
icon should be placed in some other directory than %{_datadir}/icons/mini .

* The macro %{_iconsdir} is superfluous.

%global _iconsdir   %{_datadir}/icons
%global _hiconsdir  %{_datadir}/icons/hicolor

Take a look at both places, where you're using %{_iconsdir} - actually, you
should use %{_hiconsdir} macro there. I advice you to shorten the number of
macro being used.

* Directory ownership issues:

%{_libdir}/%{name} should be owned by the main package.
%{_includedir}/%{name} should be owned by *-devel sub-package

* Also consider removing entirely %doc sections from plugin sub-packages.

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2009-11-12 Thread 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=494695





--- Comment #15 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info  
2009-11-12 20:13:58 EDT ---
Delete invoke ldconfig:
http://hubbitus.net.ru/rpm/Fedora11/qutim/qutim-0.2.0-1.19.fc11.src.rpm

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2009-11-11 Thread 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=494695





--- Comment #14 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info  
2009-11-11 06:46:12 EDT ---
Add epoch in gloox dependency:
http://hubbitus.net.ru/rpm/Fedora11/qutim/qutim-0.2.0-1.18.fc11.src.rpm

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2009-11-10 Thread 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=494695





--- Comment #13 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info  
2009-11-10 04:01:51 EDT ---
Peter, you was right - in release many changes...

(In reply to comment #12)
 Builds fine now.
 
 http://koji.fedoraproject.org/koji/taskinfo?taskID=1791960
 
 REVIEW:
 
 + rpmlint is not silent:
 
 [pe...@workplace ~]$ rpmlint Desktop/qutim-*
 qutim-debuginfo.i586: W: spurious-executable-perm
 /usr/src/debug/qutim/plugins/icq/clientidentify.h
 qutim-debuginfo.i586: W: spurious-executable-perm
 /usr/src/debug/qutim/src/3rdparty/qtwin/qtwin.cpp
 qutim-debuginfo.i586: E: wrong-script-end-of-line-encoding
 /usr/src/debug/qutim/src/3rdparty/qtwin/qtwin.cpp
 qutim-debuginfo.i586: W: spurious-executable-perm
 /usr/src/debug/qutim/plugins/icq/buddycaps.h
 qutim-debuginfo.i586: W: spurious-executable-perm
 /usr/src/debug/qutim/src/3rdparty/qtwin/qtwin.h
 qutim-debuginfo.i586: E: wrong-script-end-of-line-encoding
 /usr/src/debug/qutim/src/3rdparty/qtwin/qtwin.h
 qutim-debuginfo.i586: W: spurious-executable-perm
 /usr/src/debug/qutim/plugins/icq/clientidentify.cpp
 qutim-devel.i586: W: no-documentation
 qutim-icq.i586: W: no-documentation
 qutim-irc.i586: W: no-documentation
 qutim-jabber.i586: W: no-documentation
 qutim-mrim.i586: W: no-documentation
 7 packages and 0 specfiles checked; 2 errors, 10 warnings.
spurious-executable-perm fixed in %prep.

Add BR dos2unix and fix lineendings qtwin to avoid rpmlint complain
wrong-script-end-of-line-encoding. But is it really needed for files inytended
for Windows (just delete break build, but it nothing doing on Linux)?

 and some unneeded calls for ldconfig.
Why calls to ldconfig is unneeded?

http://fedoraproject.org/wiki/Packaging/Guidelines#Shared_Libraries says:
Whenever possible (and feasible), Fedora Packages containing libraries should
build them as shared libraries. In addition, every binary RPM package which
contains shared library files (not just symlinks) in any of the dynamic
linker's default paths, must call ldconfig in %post and %postun. If the package
has multiple subpackages with libraries, each subpackage should also have a
%post/%postun section that calls /sbin/ldconfig.

 Also, and most
 important, I can;t find a package, who provides /usr/share/icons/mini.
Hm, yes, I too. May be I simple should delete this file?

 So, you should add the following explicit requires:
 
 hicolor-icon-theme (for /usr/share/icons/hicolor/64x64/apps )
Added.
 cmake (for /usr/share/cmake/Modules )
Added to -devel subpackage.

 + The License field in the package spec file matches the actual license.
There I find some issues: Several files like (3rdparty/qtsolutions/*,
plugins/plugman/libs/QZipReader) is 3dparty and have licensing similar to QT:
Commercial/LGPL with exceptions in LGPL_EXCEPTION.txt/GPLv3.
This is free software, but it is not under GPLv2+ license as full package. And
I in frustration - can we ship it? If you can't answer on this question, please
set legal flag for Spot.

 +/- The sources used to build the package must match the upstream source, as
 provided in the spec URL. Since it's a pre-release, and new version was
 recently released, I'll wait until you update srpm to ver. 0.2.
I update it. Also several new plugins appeared in release.

 - A Fedora package must not list a file more than once in the spec file's
 %files listings. Unfortunately, you're listing %{_bindir}/%{name} twice in 
 main
 package's %files section.
Yes, it with comment from old times, when symlink was also been installed.
Fixed.

http://hubbitus.net.ru/rpm/Fedora11/qutim/qutim-0.2.0-1.17.fc11.src.rpm

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2009-11-01 Thread 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=494695





--- Comment #10 from Peter Lemenkov lemen...@gmail.com  2009-11-01 10:03:19 
EDT ---
BTW, qutim 0.2 is out! 

http://www.qutim.org/forum/viewtopic.php?f=40t=1335

Could you, please, update your package to the latest version?

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2009-11-01 Thread 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=494695





--- Comment #11 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info  
2009-11-01 10:13:25 EDT ---
Off course! Gloox release 1.0 also was released yesterday. We speak about it
with Ruslan and I prepare both updates (I think do that in 1-3 days). But it is
not contain major changes except bugfixes and api stability, can we finally
start review?? Update process in the future should be persistent continuous
process...

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2009-10-30 Thread 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=494695





--- Comment #9 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info  
2009-10-30 16:11:02 EDT ---
Oh, sorry. On my system this macros defined.

http://hubbitus.net.ru/rpm/Fedora11/qutim/qutim-0.2-0.16.beta2SVN20091018.fc11.src.rpm
http://koji.fedoraproject.org/koji/taskinfo?taskID=1778748

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2009-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.


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





--- Comment #8 from Peter Lemenkov lemen...@gmail.com  2009-10-29 11:56:57 
EDT ---
Koji scratchbuild for F-11 failed due to easy-to-fix issue in the %files
section:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1776709

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2009-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.


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





--- Comment #7 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info  
2009-10-24 04:05:37 EDT ---
Ura! We did it!

http://hubbitus.net.ru/rpm/Fedora11/qutim/qutim.spec
http://hubbitus.net.ru/rpm/Fedora11/qutim/qutim-0.2-0.15.beta2SVN20091018.fc11.src.rpm

-- 
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.

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


[Bug 494695] Review Request: qutim - Multiplatform Instant Messenger on Qt4

2009-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.


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


Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info changed:

   What|Removed |Added

Summary|Review Request: qutIM - |Review Request: qutim -
   |Multiplatform Instant   |Multiplatform Instant
   |Messenger on Qt4|Messenger on Qt4




-- 
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.

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


[Bug 494695] Review Request: qutIM - Multiplatform Instant Messenger on Qt4

2009-09-18 Thread 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=494695





--- Comment #6 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info  
2009-09-18 07:21:16 EDT ---
As you can see, bug is reopened and still unfixed. If you want speedup -
comment these bug in its bugtracker to fix error.
Previous maintainer of Qutim-Jabber plugin is unaccessible for the long time
for me. I think it is main reason...

Also, as you known Fedora do not permit bundled libraries and built with it is
prohibited.

-- 
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.

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


[Bug 494695] Review Request: qutIM - Multiplatform Instant Messenger on Qt4

2009-09-17 Thread 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=494695


Alexey Torkhov atork...@gmail.com changed:

   What|Removed |Added

 CC||atork...@gmail.com




--- Comment #5 from Alexey Torkhov atork...@gmail.com  2009-09-17 02:54:58 
EDT ---
There is build for 0.2 beta svn snapshot in this repo:
ftp://volkoff.ru/repo/fedora11/
Here is discussion: http://linuxforum.ru/index.php?showtopic=99429
But perhaps they build it with bundled gloox. Is that bug still unfixed?

-- 
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.

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


[Bug 494695] Review Request: qutIM - Multiplatform Instant Messenger on Qt4

2009-08-01 Thread 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=494695


Peter Lemenkov lemen...@gmail.com changed:

   What|Removed |Added

  Status Whiteboard||NotReady




--- Comment #4 from Peter Lemenkov lemen...@gmail.com  2009-08-01 02:39:28 
EDT ---
Ok, understood. Until then, I'm adding NotReady keyword to the whiteboard.

-- 
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.

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


[Bug 494695] Review Request: qutIM - Multiplatform Instant Messenger on Qt4

2009-07-31 Thread 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=494695





--- Comment #2 from Peter Lemenkov lemen...@gmail.com  2009-07-31 14:53:22 
EDT ---
Ver. 0.2_beta2 is out. Please, consider updating your srpm, and I'll continue.

http://www.qutim.org/uploads/src/qutim-0.2_beta2.tar.gz

-- 
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.

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


[Bug 494695] Review Request: qutIM - Multiplatform Instant Messenger on Qt4

2009-07-31 Thread 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=494695





--- Comment #3 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info  
2009-08-01 01:16:11 EDT ---
Off course! Many time ago I was try build it, but it is impossible due bug:
http://trac.qutim.org/task/258

I directly speak with maintainer, and problem occurred in directly patching
bundled copy of gloox library. This way is unacceptable for Fedora. So, waiting
until bug resolved.

-- 
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.

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


[Bug 494695] Review Request: qutIM - Multiplatform Instant Messenger on Qt4

2009-05-18 Thread 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=494695


Peter Lemenkov lemen...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||lemen...@gmail.com
 AssignedTo|nob...@fedoraproject.org|lemen...@gmail.com
   Flag||fedora-review?




--- Comment #1 from Peter Lemenkov lemen...@gmail.com  2009-05-18 23:33:08 
EDT ---
I'll review it.

-- 
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.

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


[Bug 494695] Review Request: qutIM - Multiplatform Instant Messenger on Qt4

2009-04-25 Thread 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=494695


Bug 494695 depends on bug 494693, which changed state.

Bug 494693 Summary: Review Request: gloox - A rock-solid, full-featured 
Jabber/XMPP client library
https://bugzilla.redhat.com/show_bug.cgi?id=494693

   What|Old Value   |New Value

 Status|NEW |ASSIGNED
 Resolution||NEXTRELEASE
 Status|ASSIGNED|CLOSED



-- 
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.

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


[Bug 494695] Review Request: qutIM - Multiplatform Instant Messenger on Qt4

2009-04-07 Thread 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=494695


Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info changed:

   What|Removed |Added

 Depends on||494693




-- 
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.

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


[Bug 494695] Review Request: qutIM - Multiplatform Instant Messenger on Qt4

2009-04-07 Thread 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=494695


Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info changed:

   What|Removed |Added

  Alias||qutIM




-- 
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.

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