ketchup was Re: [kernel.org users] XZ Migration discussion

[Pavel Machek <pavel@ucw.cz>]

Hi!

> > I wouldn't worry too much about breaking the current locations. Just
> > give some time for software authors (ketchup comes to mind) to update
> > their code and it shouldn't be a big problem.
> 
> As the new maintainer of ketchup:
> 
> git://git.kernel.org/pub/scm/linux/kernel/git/rostedt/ketchup.git
> 
> I could write a fix for the new format real quick. My concern is that
> users of ketchup may not see this update for a long time.

I'd really like to fix ketchup to behave sanely in late -rc stages:
going from -rc6 to -rc8 is possible using two downloads through --rc7
(like 100KB?)  instead of two big ones through full release (like
10MB).

But first step is --only-dl option -- when you want to cache the
needed patches but not apply anything yet.

Please apply,
									Pavel

diff --git a/ketchup b/ketchup
index 0728aec..3249cbc 100755
--- a/ketchup
+++ b/ketchup
@@ -405,7 +405,7 @@ def apply_patch(ver, reverse = 0):
         r = " -R"
 
     qprint("Applying %s%s" % (os.path.basename(p), r))
-    if options["dry-run"]:
+    if options["dry-run"] or options["only-dl"]:
         return ver
 
     def cmd(patch, reverse, dry):
@@ -484,7 +484,7 @@ def install_nearest(ver):
         ver = list[0][2]
 
     qprint("Unpacking %s" % os.path.basename(f))
-    if options["dry-run"]: return ver
+    if options["dry-run"] or options["only-dl"]: return ver
     untar(f)
 
     return ver
@@ -658,6 +658,7 @@ opts = [
     ('l', 'list-trees', None, 'list supported trees'),
     ('m', 'show-makefile', None, 'output version in makefile <arg>'),
     ('n', 'dry-run', None, 'don\'t download or apply patches'),
+    ('o', 'only-dl', None, 'don\'t apply patches'),
     ('p', 'show-previous', None, 'output version previous to <arg>'),
     ('q', 'quiet', None, 'reduce output'),
     ('r', 'rename-directory', None, 'rename updated directory to %s<v>'
@@ -750,7 +751,7 @@ if not a and os.listdir("."):
 b = find_ver(args[0])
 qprint("%s -> %s" % (a, b))
 transform(a, b)
-if options["rename-directory"] and not options["dry-run"]:
+if options["rename-directory"] and not options["dry-run"] and not options["only-dl"] :
     rename_dir(b)
 
 if postcommand and os.system(postcommand):

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html