[Bug 480056] Review Request: libchamplain-gtk - Gtk+ widget wrapper for libchamplain

2009-05-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=480056





--- Comment #6 from Denis Leroy   2009-05-17 04:26:19 EDT 
---
Hi Debarshi,

Sorry for the delay. Let's get it over with, can you post updates to your spec
and srpm ?

-- 
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 480056] Review Request: libchamplain-gtk - Gtk+ widget wrapper for libchamplain

2009-03-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=480056





--- Comment #5 from Debarshi Ray   2009-03-17 18:06:23 
EDT ---
Looks like libchamplain-gtk will continue to be a separate tarball for some
more time.

Spec: http://rishi.fedorapeople.org/libchamplain-gtk.spec
SRPM: http://rishi.fedorapeople.org/libchamplain-gtk-0.2.9-1.fc10.src.rpm
Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=1246988

-- 
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 480056] Review Request: libchamplain-gtk - Gtk+ widget wrapper for libchamplain

2009-03-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=480056


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

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


[Bug 480056] Review Request: libchamplain-gtk - Gtk+ widget wrapper for libchamplain

2009-02-22 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=480056





--- Comment #4 from Debarshi Ray   2009-02-22 05:01:17 
EDT ---
(In reply to comment #3)
> About the merged source trees, can you ask upstream when this will likely
> happen ?

While the SCMs have been merged, it will take some time, my guess is 2-3
months, before there is any release from this merged tree.

-- 
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 480056] Review Request: libchamplain-gtk - Gtk+ widget wrapper for libchamplain

2009-02-22 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=480056





--- Comment #3 from Denis Leroy   2009-02-22 04:46:00 EDT 
---
About the merged source trees, can you ask upstream when this will likely
happen ?

-- 
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 480056] Review Request: libchamplain-gtk - Gtk+ widget wrapper for libchamplain

2009-02-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=480056





--- Comment #2 from Debarshi Ray   2009-02-21 15:42:53 
EDT ---
Sorry for the delayed response.

Till now libchamplain and libchamplain-gtk had two separate source trees, which
has recently been merged into one. This happened earlier than I had predicted.
However it will take some time for the first release from this merged tree.
Therefore I am not sure whether we should go ahead with this review, because
libchamplain-gtk will be very short-lived as a separate source package. Once
the 0.4.x (or 0.3.x) releases come out from the unified tree, libchamplain-gtk
will become a sub-package of libchamplain.

What do you think?

(In reply to comment #1)
> Couple of things :
> 
> - you should not put "%{version}" macro in the patch0 source filename, since
> this will force you to rename it everytime you update the package. Although 
> not
> officially in the guidelines, most people hardcode the version that the patch
> was derived from, and keep that version in the filename until the patch no
> longer applies and has to be recreated...

Yes, you are right. The reason I do it is to force myself to rebase the patches
on every new upstream release to avoid any silly build failures due to the new
RPM's zero fuzz tolerance, or have any ancient patches lying around.

> - why the pkgconfig patch ? I can see the development headers indeed only need
> libchamplain and gtk2 but this most likely will not stay true in the future.
> Seems a bit over the top...

In case they actually add any new dependencies then we will have to add it in
the Spec (only on F10 since F11 auto-detects pkgconfig stuff) and modify/remove
the patch accordingly. I have notified the upstream author and he seems to
agree that the pkgconfig file might be faulty.

It somehow does not feel right to knowingly distribute a faulty 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.

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


[Bug 480056] Review Request: libchamplain-gtk - Gtk+ widget wrapper for libchamplain

2009-02-19 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=480056


Bug 480056 depends on bug 480050, which changed state.

Bug 480050 Summary: Review Request: libchamplain - Map view for Clutter
https://bugzilla.redhat.com/show_bug.cgi?id=480050

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE



-- 
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 480056] Review Request: libchamplain-gtk - Gtk+ widget wrapper for libchamplain

2009-01-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=480056





--- Comment #1 from Denis Leroy   2009-01-29 16:29:08 EDT 
---
Couple of things :

- you should not put "%{version}" macro in the patch0 source filename, since
this will force you to rename it everytime you update the package. Although not
officially in the guidelines, most people hardcode the version that the patch
was derived from, and keep that version in the filename until the patch no
longer applies and has to be recreated...

- why the pkgconfig patch ? I can see the development headers indeed only need
libchamplain and gtk2 but this most likely will not stay true in the future.
Seems a bit over the top...

Otherwise this is very similar to the libchamplain 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.

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


[Bug 480056] Review Request: libchamplain-gtk - Gtk+ widget wrapper for libchamplain

2009-01-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=480056


Denis Leroy  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||de...@poolshark.org
 AssignedTo|nob...@fedoraproject.org|de...@poolshark.org




-- 
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 480056] Review Request: libchamplain-gtk - Gtk+ widget wrapper for libchamplain

2009-01-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=480056


Debarshi Ray  changed:

   What|Removed |Added

 Depends on||480050




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