Bug 952602

+++ This bug was initially created as a clone of Bug #948771 +++

Created attachment 731888 [details]
patch

Description of problem:

createrepo-0.9.9-17.el6.noarch 

modifyrepo does not accept the same option as createrepo.
We are especially interested in 
--simple-md-filenames
--checksum

Version-Release number of selected component (if applicable):

0.9.9-17.el6 
but
My patch is against GIT HEAD.

How reproducible:

Option are not available.

Steps to Reproduce:
1.modifyrepo --simple-md-filenames --checksum sha

Actual results:

Option not available.

Expected results:

Option available.

--- Additional comment from Thomas Oulevey on 2013-04-05 08:37:45 EDT ---

Created attachment 731897 [details]
createrepo-modifyrepo-sha-simplemd_v2.diff

First patch had an error.

--- Additional comment from Thomas Oulevey on 2013-04-05 08:42:52 EDT ---

Created attachment 731898 [details]
createrepo-modifyrepo-sha-simplemd_v3.diff

--- Additional comment from Zdeněk Pavlas on 2013-04-05 09:32:50 EDT ---

Thanks, somewhat modifed and merged.  The --simple-md-filenames and --unique-md-filenames options are complementary, we should support both.  Also updated --help, to show --unique is the default.

--- Additional comment from Thomas Oulevey on 2013-04-05 09:58:33 EDT ---

There is an issue on line 189 :
-    parser.add_option("--simple-md-filenames", dest="unique_md_filenames",
+    parser.add_option("--simple-md-filenames", dest="simple_md_filenames",

--- Additional comment from Zdeněk Pavlas on 2013-04-05 10:05:22 EDT ---

Yeah, that's intended :)  Both options share the same destination variable, but there's a different action- store_true vs store_false.  I find this easier than handling two independent options later on.

--- Additional comment from Thomas Oulevey on 2013-04-05 10:11:44 EDT ---

Sorry, me not reading the full line :)
Thanks! works great.

I asked my TAM for a backport on el6 packages.