<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugzilla.yoctoproject.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugzilla.yoctoproject.org/"
          
          maintainer="it-coreprojects-helpdesk@linuxfoundation.org"
>

    <bug>
          <bug_id>1689</bug_id>
          
          <creation_ts>2011-10-20 08:21:00 +0000</creation_ts>
          <short_desc>matchbox SRCREV upgrade broke sato desktop</short_desc>
          <delta_ts>2011-11-23 21:38:29 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>6</classification_id>
          <classification>Yocto Project Subprojects</classification>
          <product>Sato</product>
          <component>Matchbox</component>
          <version>1.2</version>
          <rep_platform>x86</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>Medium</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>1082</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Tom Zanussi">tom.zanussi</reporter>
          <assigned_to name="Edwin Zhai">edwin.zhai</assigned_to>
          <cc>sgw</cc>
    
    <cc>wenhuax.fan</cc>
          
          
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>---</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>16943</commentid>
    <comment_count>0</comment_count>
    <who name="Tom Zanussi">tom.zanussi</who>
    <bug_when>2011-10-20 08:21:00 +0000</bug_when>
    <thetext>The below commit causes icons to never show up.  This not a duplicate of 1082, but new behavior due to the below commit.  At this point, I&apos;ve narrowed down the problem to the recipes-sato/matchbox-desktop part(s) of the commit.

Reverting the commit causes the icons to reappear.


&gt; This &apos;100%&apos; loss of icon is due to the below commit.  Reverting it should bring
&gt; back the icons.
&gt; 
&gt; commit d39c6738df4b7e8ef22f2a7154274c1d08b1d51f
&gt; Author: Zhai Edwin &lt;edwin.zhai@intel.com&gt;
&gt; Date:   Mon Sep 26 19:55:47 2011 +0800
&gt; 
&gt;     matchbox: Upgrade SRCREV to reflect recent accpeted patches by upstream
&gt; 
&gt;     (From OE-Core rev: 33a1a05ef988c69f8ff8e38c6723922082e5d1aa)

In other words, this isn&apos;t the same bug, though it has a similar but more sever
e symptom.  A separate new bug should be opened for it</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>16951</commentid>
    <comment_count>1</comment_count>
    <who name="Saul Wold">sgw</who>
    <bug_when>2011-10-21 11:38:15 +0000</bug_when>
    <thetext>Duplicate

*** This bug has been marked as a duplicate of bug 1587 ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>16953</commentid>
    <comment_count>2</comment_count>
    <who name="Tom Zanussi">tom.zanussi</who>
    <bug_when>2011-10-21 12:47:17 +0000</bug_when>
    <thetext>(In reply to comment #1)
&gt; Duplicate
&gt; 
&gt; *** This bug has been marked as a duplicate of bug 1587 ***

Marking this as a duplicate doesn&apos;t seem correct.  Please expand on why.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>16959</commentid>
    <comment_count>3</comment_count>
    <who name="Edwin Zhai">edwin.zhai</who>
    <bug_when>2011-10-23 16:19:58 +0000</bug_when>
    <thetext>(In reply to comment #2)
&gt; (In reply to comment #1)
&gt; &gt; Duplicate
&gt; &gt; 
&gt; &gt; *** This bug has been marked as a duplicate of bug 1587 ***
&gt; 
&gt; Marking this as a duplicate doesn&apos;t seem correct.  Please expand on why.

Agree with Tom, they are different bugs. 1687 exists for a long time in a random way, while 1689 comes recently after a new patch in upstream. We can&apos;t verify 1687 until fix 1689.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>17198</commentid>
    <comment_count>4</comment_count>
    <who name="Edwin Zhai">edwin.zhai</who>
    <bug_when>2011-11-14 15:33:40 +0000</bug_when>
    <thetext>The blank screen of matchbox-desktop comes from wrong parameter of &quot;_NET_WORKAREA&quot;

create_desktop =&gt; x_monitor_workarea =&gt; net_workarea_changed, which will:
1. get the _NET_WORKAREA atom via XGetWindowProperty
2. use these 4 parameters to call workarea_changed for screen refresh later

The good parameter should be 0,0,640,432(on qemux86), but we get something like 0,640,0,0.


This atom is set by matchbox-window-manager:
wm_init_existing =&gt; ewmh_update_rects, which will:
1. get the right parameters
2. set the atom via XChangeProperty

The issue is we get right parameter as 0,0,640,432, but XChangeProperty always set something like 0,640....

Root cause is the parameter parsing, doc of XChangeProperty said &quot;If the specified format is 32, the property data must be a long array&quot;. But CARD32 is used instead of long. So 0,0,640,432 is parsed by libX11 as 0|0, 640|432.

Patch will be out soon.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>17200</commentid>
    <comment_count>5</comment_count>
    <who name="Edwin Zhai">edwin.zhai</who>
    <bug_when>2011-11-14 16:30:23 +0000</bug_when>
    <thetext>*** Bug 1687 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>17201</commentid>
    <comment_count>6</comment_count>
    <who name="Edwin Zhai">edwin.zhai</who>
    <bug_when>2011-11-14 16:33:45 +0000</bug_when>
    <thetext>(In reply to comment #3)
&gt; (In reply to comment #2)
&gt; &gt; (In reply to comment #1)
&gt; &gt; &gt; Duplicate
&gt; &gt; &gt; 
&gt; &gt; &gt; *** This bug has been marked as a duplicate of bug 1587 ***
&gt; &gt; 
&gt; &gt; Marking this as a duplicate doesn&apos;t seem correct.  Please expand on why.
&gt; 
&gt; Agree with Tom, they are different bugs. 1687 exists for a long time in a
&gt; random way, while 1689 comes recently after a new patch in upstream. We can&apos;t
&gt; verify 1687 until fix 1689.

1687 &amp; 1689 should be duplicated bugs. 1082 is the long time bug that depends on 1689.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>17202</commentid>
    <comment_count>7</comment_count>
    <who name="Edwin Zhai">edwin.zhai</who>
    <bug_when>2011-11-14 18:28:45 +0000</bug_when>
    <thetext>When update SRCREV for matchbox-desktop, one patch to fix bug 658 is taken place of by a new one. But this old fix (http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=a4c6bcb895cb7888b5045ed76f96f056acd7b4ab)
can serve as one work around for bug 1689, so removing it expose this hidden bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>17358</commentid>
    <comment_count>8</comment_count>
    <who name="Edwin Zhai">edwin.zhai</who>
    <bug_when>2011-11-23 21:38:29 +0000</bug_when>
    <thetext>Got fixed @
http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=40fe6457d5ca49fe113bc26c60d5add59d85b34f</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>