[Bug 697974] Re: time-admin crashed with SIGSEGV in e_map_point_get_location()

2011-01-15 Thread Milan Bouchet-Valat
The problem isn't that I fear people won't do the work, as I know
packaging will happen anyway. It's just that the way Ubuntu devs
consider patching is plain wrong: there's generally no hurry pushing a
fix to the development release, so downstream patching only generates
useless work. I wish Ubuntu would turn to upstream when they have a
patch before even thinking of applying it: very often, feedback and
discussion can help understanding the best solution. Plus, if I hadn't
been around, would upstream have even heard of this patch one day? This
is not against you at all, it's more a general state of mind that I
consider slightly counter-productive.

I've tried to reproduce this with Compiz, and I can't. Weird. If under
Compiz we're getting leave-notify-event signals, something is really
wrong. Did you notice when these signals are sent? When simply moving
the pointer, when clicking... ?

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/697974

Title:
  time-admin crashed with SIGSEGV in e_map_point_get_location()

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 697974] Re: time-admin crashed with SIGSEGV in e_map_point_get_location()

2011-01-14 Thread Milan Bouchet-Valat
Thanks for the debugging and for the patch, but I don't think it fixes
the root cause. tzmap-point_hover should already contain the point that
was retrieved using e_map_get_closest_point() when the pointer moved. So
if tzmap-point_hover isn't valid, we have to find out why. The
questions I asked in my last comment might help understanding this.

I don't see what Compiz could be doing that makes this fail - maybe we
don't get events when the pointer moves, but this would be a real
failure from Compiz. Or is the crash only triggered when you click
without the pointer having moved too much on the map? or when it hasn't
gone out of the map once?

Also, please don't commit the fix to Natty. If we choose this patch, I'd
rather push it upstream first, rather than diverging, which will create
packaging work when the next release is out (this is not a problem for
Maverick, though).

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/697974

Title:
  time-admin crashed with SIGSEGV in e_map_point_get_location()

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 697974] Re: time-admin crashed with SIGSEGV in e_map_point_get_location()

2011-01-14 Thread Launchpad Bug Tracker
This bug was fixed in the package gnome-system-tools - 2.32.0-0ubuntu4

---
gnome-system-tools (2.32.0-0ubuntu4) natty; urgency=low

  * debian/patches/95_timezone_point_selection.patch: Fix selecting a timezone
while running on compiz: compute the point we're at when the map gets
clicked. (LP: #697974)
 -- Mathieu Trudel-Lapierre mathieu...@ubuntu.com   Fri, 14 Jan 2011 03:16:03 
+

** Changed in: gnome-system-tools (Ubuntu)
   Status: In Progress = Fix Released

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/697974

Title:
  time-admin crashed with SIGSEGV in e_map_point_get_location()

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 697974] Re: time-admin crashed with SIGSEGV in e_map_point_get_location()

2011-01-14 Thread Mathieu Trudel-Lapierre
Heh, sorry, I didn't see your message. I understand how it's more work,
but I'm prepared to take care of it (although I didn't and can't upload
this myself).

Milan, further debugging was showing that the issue seems to be that
compiz may be causing leave-notify-event signals to be sent, which would
cause tzmap-point_hover to be nullified. I haven't noticed any special
causes for this like moving lots or not much around the map, or having
the pointer get out of the map.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/697974

Title:
  time-admin crashed with SIGSEGV in e_map_point_get_location()

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 697974] Re: time-admin crashed with SIGSEGV in e_map_point_get_location()

2011-01-13 Thread Mathieu Trudel-Lapierre
Milan,

The only way I can reproduce this is while running unity (but I do
expect it's actually related to running with compiz).

I already have a fix for it, which is to compute the closest point for
the selected point when it gets clicked... either way the longitude and
latitude are already available to zoom the map.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/697974

Title:
  time-admin crashed with SIGSEGV in e_map_point_get_location()

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 697974] Re: time-admin crashed with SIGSEGV in e_map_point_get_location()

2011-01-13 Thread Mathieu Trudel-Lapierre
** Changed in: gnome-system-tools (Ubuntu)
   Status: New = In Progress

** Changed in: gnome-system-tools (Ubuntu)
 Assignee: (unassigned) = Mathieu Trudel-Lapierre (mathieu-tl)

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/697974

Title:
  time-admin crashed with SIGSEGV in e_map_point_get_location()

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 697974] Re: time-admin crashed with SIGSEGV in e_map_point_get_location()

2011-01-13 Thread Launchpad Bug Tracker
** Branch linked: lp:~mathieu-tl/gnome-system-tools/lp-697974

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/697974

Title:
  time-admin crashed with SIGSEGV in e_map_point_get_location()

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 697974] Re: time-admin crashed with SIGSEGV in e_map_point_get_location()

2011-01-10 Thread Michael Terry
I can also reproduce by clicking on America/Chicago.

** Visibility changed to: Public

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/697974

Title:
  time-admin crashed with SIGSEGV in e_map_point_get_location()

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 697974] Re: time-admin crashed with SIGSEGV in e_map_point_get_location()

2011-01-10 Thread Milan Bouchet-Valat
And only when clicking on that point? Here I'm not able to reproduce the
crash...

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/697974

Title:
  time-admin crashed with SIGSEGV in e_map_point_get_location()

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 697974] Re: time-admin crashed with SIGSEGV in e_map_point_get_location()

2011-01-10 Thread Milan Bouchet-Valat
This bug is weird because we seem to be passing NULL as location point.
This can only happen if no point was highlighted when you clicked; the
pointer must get out of the map for the point to be set to NULL. Do
different effects happen if e.g. you hover a different point before
clicking on Chicago? Are the points colored in yellow in all cases?

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/697974

Title:
  time-admin crashed with SIGSEGV in e_map_point_get_location()

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs