<?xml version="1.0" encoding="UTF-8" ?>

<bugzilla version="5.2"
          urlbase="https://bugzilla.altlinux.org/"
          
          maintainer="jenya@basealt.ru"
>

    <bug>
          <bug_id>31473</bug_id>
          
          <creation_ts>2015-11-11 18:49:12 +0300</creation_ts>
          <short_desc>unneeded extra argument in gear-clone</short_desc>
          <delta_ts>2015-11-12 16:49:11 +0300</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>4</classification_id>
          <classification>Development</classification>
          <product>Sisyphus</product>
          <component>mithraen-build-utils</component>
          <version>unstable</version>
          <rep_platform>all</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>CLOSED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P3</priority>
          <bug_severity>minor</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Ivan Zakharyaschev">imz</reporter>
          <assigned_to name="Nobody&apos;s working on this, feel free to take it">nobody</assigned_to>
          
          
          <qa_contact>qa-sisyphus</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>153648</commentid>
    <comment_count>0</comment_count>
    <who name="Ivan Zakharyaschev">imz</who>
    <bug_when>2015-11-11 18:49:12 +0300</bug_when>
    <thetext>mithraen-build-utils-0.1.27-alt1

http://git.altlinux.org/people/imz/packages/?p=mithraen-build-utils.git;a=commitdiff;h=3d6f5cd38e2feb97babcb37c23c67b7c7d1c043b

--- a/mithraen-build-utils/bin/gear-clone
+++ b/mithraen-build-utils/bin/gear-clone
@@ -16,7 +16,7 @@ type=`girar-get-upload-method &quot;$name&quot; ||:`
 if [ &quot;$type&quot; = &quot;srpm&quot; ]; then
     git clone git://git.altlinux.org/srpms/${name:0:1}/${name}.git
 else
-    girar-clone-build-commit -o girar &quot;$name&quot; &quot;$name&quot; &quot;$name&quot;
+    girar-clone-build-commit -o girar &quot;$name&quot; &quot;$name&quot;
 fi
 
 pushd  &quot;$name&quot; &amp;&gt; /dev/null || exit -1</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>153649</commentid>
    <comment_count>1</comment_count>
    <who name="Ivan Zakharyaschev">imz</who>
    <bug_when>2015-11-11 19:58:15 +0300</bug_when>
    <thetext>Explained why in the updated commit message in http://git.altlinux.org/people/imz/packages/?p=mithraen-build-utils.git;a=commitdiff;h=2e77cfd3d17f5808c1f0ac1bacf0f504dac5c514 :

Cf. -- no 3rd argument is used:

$ girar-clone-build-commit -h
Usage:
    girar-clone-build-commit [-h] [-f] [-b *repository*] [-d *local
    destination branch*] [-n *local repository name*] [-o *name*] *name*
    [*local repository name*]

Options:
    -b *repository branch*
        Name of the repository branch. Values: sisyphus|5.1|p5|.. Default is
        sisyphus.

    -d *local destination branch name*
        Name of the checked out branch. Default is master.

    -f  Forcefully remove previous &lt;name&gt;.git and clone over even if
        &lt;name&gt;.git already exists.

    -h  Display this help and exit.

    -n *local repository name*
        Name of the local repository. Default is *name.git*. Can also be
        specified as the second argument.

    -o *remote name*
        Option passed to git-clone. Instead of using the remote name origin
        to keep track of the upstream repository, use *remote name*.

$</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>153651</commentid>
    <comment_count>2</comment_count>
    <who name="Ivan Zakharyaschev">imz</who>
    <bug_when>2015-11-11 21:14:47 +0300</bug_when>
    <thetext>Just in case, not to loose and forget this minor clean-up tweak in the code:

task #153043

Please approve if that&apos;s OK with you when it is ready.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>153659</commentid>
    <comment_count>3</comment_count>
    <who name="Denis Smirnov">mithraen</who>
    <bug_when>2015-11-12 12:21:47 +0300</bug_when>
    <thetext>Task #153043 approved</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>153660</commentid>
    <comment_count>4</comment_count>
    <who name="Repository Robot">repository-robot</who>
    <bug_when>2015-11-12 12:49:54 +0300</bug_when>
    <thetext>mithraen-build-utils-0.1.27-alt2 -&gt; sisyphus:

* Wed Nov 11 2015 Ivan Zakharyaschev &lt;imz@altlinux&gt; 0.1.27-alt2
- gear-clone: minor tweak (unneeded extra argument in the code removed)
  (ALT#31473).</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>