[Bug 205136] Review Request: gg2 - GNU Gadu 2 - free talking

2008-07-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: Review Request: gg2 - GNU Gadu 2 - free talking


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium
Product|Fedora Extras   |Fedora
Version|devel   |rawhide




-- 
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 205136] Review Request: gg2 - GNU Gadu 2 - free talking

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

Summary: Review Request: gg2 - GNU Gadu 2 - free talking


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-09-17 15:40 EST ---
Okay, approved.

-- 
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 205136] Review Request: gg2 - GNU Gadu 2 - free talking

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

Summary: Review Request: gg2 - GNU Gadu 2 - free talking


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2006-09-17 16:35 EST ---
Thanks! Imported and built for devel, FC-5 branch requested.

-- 
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 205136] Review Request: gg2 - GNU Gadu 2 - free talking

2006-09-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: Review Request: gg2 - GNU Gadu 2 - free talking


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-15 09:24 EST ---
http://rpm.greysector.net/extras/gg2.spec

Updated spec builds cleanly in fc5 and fc6 mock.


-- 
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 205136] Review Request: gg2 - GNU Gadu 2 - free talking

2006-09-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: Review Request: gg2 - GNU Gadu 2 - free talking


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-15 12:44 EST ---
MUST items:
 * rpmlint output:
W: gg2 mixed-use-of-spaces-and-tabs
W: gg2 doc-file-dependency /usr/share/doc/gg2-2.2.9/gg2sms /usr/bin/perl
W: gg2 doc-file-dependency /usr/share/doc/gg2-2.2.9/mkgg2snap /bin/sh
W: gg2 doc-file-dependency /usr/share/doc/gg2-2.2.9/plugin_example/build.sh /
bin/sh
W: gg2-arts no-documentation
W: gg2-devel no-documentation
W: gg2-dockapp no-documentation
W: gg2-esound no-documentation
W: gg2-gadu-gadu no-documentation
W: gg2-jabber no-documentation
W: gg2-systray no-documentation
W: gg2-tlen no-documentation
W: gg2-xosd no-documentation
 * package is named well
 * spec file is named well
 * package meets Packaging Guidelines
 * package is licensed with an GPL open-source compatible license
 * License field matches actual license
 * license is included in %doc
 * spec is legible
 * md5sums are matching (e12c2153993dbbf66f8cefa3c628e7cd)
 * package succesfully compiles on x86_64
 * BRs are listed properly
 * spec handles locales properly
 * proper %post and %postun sections
 * not relocatable
 * package owns its directories
 * no duplicates in %files
 * every %files section includes %defattr
 * proper %clean section
 * no need to doc subpackage
 * .pc files and headers are in -devel
 * .la file in -devel

THINGS to do:
 * cosmetics: fix warning with mixed-spaces-and-tabs in spec
 * remove unnecessary dependiences (caused by %doc files) by 
adding following lines to %prep:
chmod -x contrib/gg2sms
chmod -x contrib/mkgg2snap
chmod -x contrib/plugin_example/build.sh
 * you probably should remove libgg2_core.la from -devel subpackage

Fix things mentioned above, and I'll approve this 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 205136] Review Request: gg2 - GNU Gadu 2 - free talking

2006-09-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: Review Request: gg2 - GNU Gadu 2 - free talking


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-15 19:36 EST ---
http://rpm.greysector.net/extras/gg2.spec

- removed redundant build deps
- removed .la dropping from -devel
- made scripts in %%doc non-executable

I don't know what to do about that .pc file. It's autogenerated.

-- 
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 205136] Review Request: gg2 - GNU Gadu 2 - free talking

2006-09-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: Review Request: gg2 - GNU Gadu 2 - free talking


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-15 20:00 EST ---
(In reply to comment #12)
 I don't know what to do about that .pc file. It's autogenerated.

Fix lib/gg2_core.pc.in. It is apparently incorrect.
Perhaps you also have to fix configure.in.

-- 
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 205136] Review Request: gg2 - GNU Gadu 2 - free talking

2006-09-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: Review Request: gg2 - GNU Gadu 2 - free talking


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-15 22:37 EST ---
http://rpm.greysector.net/extras/gg2.spec
http://rpm.greysector.net/extras/gg2-2.2.9-6.src.rpm

- fix build on devel
- patch .pc file not to include RPM_OPT_CFLAGS

I've tested my fix of the .pc file. A sample plugin compiles fine.

-- 
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 205136] Review Request: gg2 - GNU Gadu 2 - free talking

2006-09-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: Review Request: gg2 - GNU Gadu 2 - free talking


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-16 00:07 EST ---
Well, .pc file is now correct.
Then does -devel package really require perl?

-- 
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 205136] Review Request: gg2 - GNU Gadu 2 - free talking

2006-09-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: Review Request: gg2 - GNU Gadu 2 - free talking


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-16 00:17 EST ---
Note: 
even if -devel package doesn't require perl,
perl is anyway installed because main package requires perl.

-- 
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 205136] Review Request: gg2 - GNU Gadu 2 - free talking

2006-09-14 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: Review Request: gg2 - GNU Gadu 2 - free talking


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
OtherBugsDependingO|163776  |163778
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-09-14 16:37 EST ---
I'll review it, but as mentioned above, there is problem with mock build:
dbus_plugin.c: In function 'ofi_getProtocols':
dbus_plugin.c:143: warning: ISO C90 forbids mixed declarations and code
dbus_plugin.c: In function 'start_plugin':
dbus_plugin.c:301: warning: cast from pointer to integer of different size
/bin/sh ../../../libtool --silent --tag=CC --mode=link gcc  -O2 -g -pipe -Wall -
Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 
-m64 -mtune=generic -pthread -Wall -Wno-uninitialized -Wchar-subscripts -
Wnested-externs -Wpointer-arith -Wno-sign-compare -I/usr/include/glib-2.0 -I/
usr/lib64/glib-2.0/include-O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -
fexceptions -fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic -
pthread -Wall -Wno-uninitialized -Wchar-subscripts -Wnested-externs -Wpointer-
arith -Wno-sign-compare -I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/
include -D_REENTRANT -D_GNU_SOURCE -fno-strict-aliasing -pipe -Wdeclaration-
after-statement -I/usr/local/include -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 
-I/usr/include/gdbm  -I/usr/lib64/perl5/5.8.8/x86_64-linux-thread-multi/CORE  -
I/builddir/build/BUILD/gg2-2.2.9/lib  -lglib-2.0   -pthread -lgthread-2.0 -
lglib-2.0   -o libdbus_plugin.la -rpath /usr/lib64/gg2 -avoid-version   
libdbus_plugin_la-dbus_plugin.lo -ldbus-glib-1 -ldbus-1 -lglib-2.0   ../../../
lib/libgg2_core.la -pthread
/usr/bin/ld: cannot find -ldbus-glib-1
collect2: ld returned 1 exit status
gmake[4]: *** [libdbus_plugin.la] Error 1
gmake[4]: Leaving directory `/builddir/build/BUILD/gg2-2.2.9/src/plugins/dbus'
gmake[3]: *** [all-recursive] Error 1
gmake[3]: Leaving directory `/builddir/build/BUILD/gg2-2.2.9/src/plugins'
gmake[2]: *** [all-recursive] Error 1
gmake[2]: Leaving directory `/builddir/build/BUILD/gg2-2.2.9/src'
gmake[1]: *** [all-recursive] Error 1
gmake[1]: Leaving directory `/builddir/build/BUILD/gg2-2.2.9'
make: *** [all] Error 2
error: Bad exit status from /var/tmp/rpm-tmp.5727 (%build)


RPM build errors:
Bad exit status from /var/tmp/rpm-tmp.5727 (%build)


-- 
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 205136] Review Request: gg2 - GNU Gadu 2 - free talking

2006-09-14 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: Review Request: gg2 - GNU Gadu 2 - free talking


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-14 17:05 EST ---
This is due to missing BuildRequires: dbus-glib. I'll post an updated spec 
tomorrow.

-- 
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 205136] Review Request: gg2 - GNU Gadu 2 - free talking

2006-09-14 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: Review Request: gg2 - GNU Gadu 2 - free talking


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||206576




--- Additional Comments From [EMAIL PROTECTED]  2006-09-14 23:43 EST ---
Well, autually:

A. my mock failure was due to dbus-glib-devel missing. This is on
   FC6-devel.

B. The mock failure by Michał is on FC5, perhaps? This is because
   /usr/lib/libdbus-glib-1.so points to nothing.
   I reported this to dbus maintainer.
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=206576

-- 
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 205136] Review Request: gg2 - GNU Gadu 2 - free talking

2006-09-14 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: Review Request: gg2 - GNU Gadu 2 - free talking


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-14 23:45 EST ---
Forgot to comment one more:

On FC5, dbus-glib(-devel) does not exist ,included in dbus(-devel).

-- 
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 205136] Review Request: gg2 - GNU Gadu 2 - free talking

2006-09-14 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: Review Request: gg2 - GNU Gadu 2 - free talking


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-14 23:49 EST ---
Oh, sorry. My misunderstanding..

A. my mock failure was due to dbus-glib-devel missing. this is correct on
   FC6-devel.
B. On FC5, dbus-glib surely exists, however, dbus-devel does not require
   dbus-glib so the symlink is broken.

-- 
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 205136] Review Request: gg2 - GNU Gadu 2 - free talking

2006-09-14 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: Review Request: gg2 - GNU Gadu 2 - free talking


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn|206576  |




-- 
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 205136] Review Request: gg2 - GNU Gadu 2 - free talking

2006-09-14 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: Review Request: gg2 - GNU Gadu 2 - free talking


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||206576




-- 
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 205136] Review Request: gg2 - GNU Gadu 2 - free talking

2006-09-08 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: Review Request: gg2 - GNU Gadu 2 - free talking


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-08 07:23 EST ---
I cannot rebuild this in mock:

checking for X11/extensions/scrnsaver.h... yes
checking for GTK... yes
checking for DBUS... no
configure: error: DBUS_PKG_ERRORS
error: Bad exit status from /var/tmp/rpm-tmp.38783 (%build)


RPM build errors:
Bad exit status from /var/tmp/rpm-tmp.38783 (%build)

config.log says:

configure:34077: checking for DBUS
configure:34085: $PKG_CONFIG --exists --print-errors dbus-1 dbus-glib-1
Package dbus-glib-1 was not found in the pkg-config search path.
Perhaps you should add the directory containing `dbus-glib-1.pc'
to the PKG_CONFIG_PATH environment variable
No package 'dbus-glib-1' found
configure:34088: $? = 1
configure:34103: $PKG_CONFIG --exists --print-errors dbus-1 dbus-glib-1
Package dbus-glib-1 was not found in the pkg-config search path.
Perhaps you should add the directory containing `dbus-glib-1.pc'
to the PKG_CONFIG_PATH environment variable
No package 'dbus-glib-1' found
configure:34106: $? = 1
No package 'dbus-glib-1' found
configure:34134: result: no
configure:34147: error: DBUS_PKG_ERRORS


-- 
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 205136] Review Request: gg2 - GNU Gadu 2 - free talking

2006-09-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: Review Request: gg2 - GNU Gadu 2 - free talking


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


Bug 205136 depends on bug 205127, which changed state.

Bug 205127 Summary: Review Request: ekg - A client compatible with Gadu-Gadu
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=205127

   What|Old Value   |New Value

 Resolution||NEXTRELEASE
 Status|ASSIGNED|CLOSED



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