<?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>9712</bug_id>
          
          <creation_ts>2016-06-06 13:36:52 +0000</creation_ts>
          <short_desc>buildhistory-diff depends on python3-git, which is not in Debian Stable</short_desc>
          <delta_ts>2017-12-06 16:55:15 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>7</classification_id>
          <classification>Build System, Metadata &amp; Runtime</classification>
          <product>OE-Core</product>
          <component>Scripts and Tools</component>
          <version>unspecified</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>8 June 2016: IN PROGRESS REVIEW</status_whiteboard>
          <keywords></keywords>
          <priority>Medium+</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>2.2 M1</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Jussi Kukkonen">jku</reporter>
          <assigned_to name="Ed Bartosh">eduard.bartosh</assigned_to>
          <cc>jose.perez.carranza</cc>
    
    <cc>leonardo.sandoval.gonzalez</cc>
    
    <cc>ross.burton</cc>
    
    <cc>srifenbark</cc>
          
          
          <cf_os>---</cf_os>
          <cf_regression_type>Regression (Used to work)</cf_regression_type>
          
          <cf_docchange>Done (doc changes complete)</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>62652</commentid>
    <comment_count>0</comment_count>
    <who name="Jussi Kukkonen">jku</who>
    <bug_when>2016-06-06 13:36:52 +0000</bug_when>
    <thetext>buildhistory-diff fails on Debian Stable because python3-git is not available.

python3-git is in jessie-backports but that&apos;s not enabled by default. So if we consider this as just a documentation bug, we would have to document how to enable backports on Debian...</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>62700</commentid>
    <comment_count>1</comment_count>
    <who name="Ed Bartosh">eduard.bartosh</who>
    <bug_when>2016-06-07 12:41:27 +0000</bug_when>
    <thetext>This also true for OpenSUSE 13.2 and 42.1
It&apos;s even worse on OpenSUSE as it doesn&apos;t have even python-git in default installation.

There are plenty of choices how to deal with this:
- find and install python3-git as Jussi suggested
- install GitPython with pip3: pip3 install GitPython --user
- run script with python 2 explicitly: python ../scripts/buildhistory-diff
- refer to python 2 in script&apos;s shebang: #!/usr/bin/env python

Changing documentation looks better than going back to python 2 from my point of view.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>62714</commentid>
    <comment_count>2</comment_count>
    <who name="Scott Rifenbark">srifenbark</who>
    <bug_when>2016-06-07 17:34:09 +0000</bug_when>
    <thetext>This section (http://www.yoctoproject.org/docs/2.2/mega-manual/mega-manual.html#examining-build-history-information) talks about running buildhistory-diff.  Outside of a mention in the migration section, which probably does not apply, that is the only area where buildhistory-diff is specifically shown. 

Would a &quot;NOTE&quot; be sufficient here?  

One thing I thought was going to happen with post 2.1 is that Python 3 would be part of the release.  If so, wouldn&apos;t this issue go away for future releases of YP?

Scott</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>62756</commentid>
    <comment_count>3</comment_count>
    <who name="Ed Bartosh">eduard.bartosh</who>
    <bug_when>2016-06-08 09:16:18 +0000</bug_when>
    <thetext>I think NOTE should be enough.

Would something like this be ok?

Buildhistory-diff requries GitPython package. Please, install it with pip3:
pip3 install GitPython --user

Or install python3-git package using distro package manager(apt-get, dnf or zipper)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>62761</commentid>
    <comment_count>4</comment_count>
    <who name="Ed Bartosh">eduard.bartosh</who>
    <bug_when>2016-06-08 11:05:02 +0000</bug_when>
    <thetext>&gt; One thing I thought was going to happen with post 2.1 is that Python 3 would be part of the release.  If so, wouldn&apos;t this issue go away for future releases of YP?

No it wouldn&apos;t as it was caused by the move to Python 3.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>62773</commentid>
    <comment_count>5</comment_count>
    <who name="Scott Rifenbark">srifenbark</who>
    <bug_when>2016-06-08 15:52:36 +0000</bug_when>
    <thetext>Hi, 

Check out the new note at http://www.yoctoproject.org/docs/2.2/ref-manual/ref-manual.html#examining-build-history-information and let me know if this is okay. 

Thanks, 
Scott</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>62774</commentid>
    <comment_count>6</comment_count>
    <who name="Scott Rifenbark">srifenbark</who>
    <bug_when>2016-06-08 15:53:31 +0000</bug_when>
    <thetext>Oops... I just realized I am not the owner of this bug.  I did change the state Ed.  Sorry.

Scott</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>62802</commentid>
    <comment_count>7</comment_count>
    <who name="Ed Bartosh">eduard.bartosh</who>
    <bug_when>2016-06-09 07:15:01 +0000</bug_when>
    <thetext>Thank you for updating the documentation! The change looks good to me. There is just one small typo in there: &quot;Alternatively, you can install python3-3&quot; should be &quot;Alternatively, you can install python3-git&quot;

Jussi, are you ok with the change?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>62804</commentid>
    <comment_count>8</comment_count>
    <who name="Jussi Kukkonen">jku</who>
    <bug_when>2016-06-09 07:23:09 +0000</bug_when>
    <thetext>(In reply to comment #7)
&gt; Jussi, are you ok with the change?

Sure.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>62816</commentid>
    <comment_count>9</comment_count>
    <who name="Leonardo Sandoval Gonzalez">leonardo.sandoval.gonzalez</who>
    <bug_when>2016-06-09 14:39:04 +0000</bug_when>
    <thetext>a similar issue: https://bugzilla.yoctoproject.org/show_bug.cgi?id=9690</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>62842</commentid>
    <comment_count>10</comment_count>
    <who name="Scott Rifenbark">srifenbark</who>
    <bug_when>2016-06-09 16:25:37 +0000</bug_when>
    <thetext>Ed, 

Don&apos;t know how I managed that typo :)

Fixed it - see http://www.yoctoproject.org/docs/2.2/ref-manual/ref-manual.html#examining-build-history-information

Thanks, 
Scott</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>62905</commentid>
    <comment_count>11</comment_count>
    <who name="Scott Rifenbark">srifenbark</who>
    <bug_when>2016-06-10 17:23:19 +0000</bug_when>
    <thetext>Thanks... I am putting the doc flag to Done.  It was set at &quot;No&quot; but it did involve the docs. 

Scott</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>62913</commentid>
    <comment_count>12</comment_count>
    <who name="Ed Bartosh">eduard.bartosh</who>
    <bug_when>2016-06-13 07:43:21 +0000</bug_when>
    <thetext>Thank you!</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>