chiark / gitweb /
git-debrebase: move --name-only
[dgit.git] / git-debrebase
1 #!/usr/bin/perl -w
2 # git-debrebase
3 # Script helping make fast-forwarding histories while still rebasing
4 # upstream deltas when working on Debian packaging
5 #
6 # Copyright (C)2017 Ian Jackson
7 #
8 # This program is free software: you can redistribute it and/or modify
9 # it under the terms of the GNU General Public License as published by
10 # the Free Software Foundation, either version 3 of the License, or
11 # (at your option) any later version.
12 #
13 # This program is distributed in the hope that it will be useful,
14 # but WITHOUT ANY WARRANTY; without even the implied warranty of
15 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
16 # GNU General Public License for more details.
17 #
18 # You should have received a copy of the GNU General Public License
19 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
20
21 # usages:
22 #    git-debrebase status
23 #    git-debrebase start       # like ffqrebase start + debrebase launder
24 #    git-debrebase new-upstream [stuff]  # see below
25 #    git-debrebase <git-rebase options>  # does debrebase start if necessary
26 #
27 #    git-debrebase analyse
28 #    git-debrebase launder     # prints breakwater tip
29 #    git-debrebase create-new-upstream-breakwater [-f] <upstreaminfo>...
30 #
31 # <upstreaminfo> is
32 #    [,][<subdir>:][+]<commitid>[,...]
33 #
34 # if initial comma is supplied, entries are not positional.  Unspecified
35 # <subdir> means root (and there may be only one).
36 # xxx want auto branch names
37 # xxx too complicated
38 # how about for now
39 #    [+]<commit> [<subdir/> [+]<commit>...]
40 # ?  plus options
41 #     --new-upstream-different-subtrees
42 #
43 #  automatic case
44 #       git-debrebase new-upstream
45 #             - previous breakwater merge must be gdr-generated
46 #             - orig set is the same as before
47 #             - implicitly uses upstream branches according to orig set
48 #             - not all upstream branches need be updated
49 #             - insists on fast-forward of each branch, unless
50 #                  --force (or --force=<subdir>[/])
51 #  branch set adjustments
52 #       git-debrebase new-upstream --add <subdir>/
53 #       git-debrebase new-upstream --rm <subdir>/
54 #       git-debrebase new-upstream / [<subdir>/ ...]
55 #             - orig set is adjusted
56 #             - otherwise like auto (--add is not checked for ffness, obv)
57 #             - multiple --add and --rm may be specified
58 #             - --add makes new upstream the last contributor
59 #  explicit
60 #       git-debrebase / [<rootcommitid>] [<subdir>/ [<commitid>] ...]
61 #             - orig set is precisely as specified now
62 #             - previous breakwater merge is irrelevant
63 #             - no fast forward checks
64 #  for now only explicit with commitids
65
66 #         implicitly uses `upstream'
67 #                                     # (or multiple other branches)
68 #       git-debrebase new-upstream \
69 #             [<subdir>/]=<commitid>
70
71 #    UPSTREAM[,[[SUBDIR:]SUBUPSTREAM]
72 #    default for SUBDIR: is from previous upstream merge[xxx terminology]
73 #    
74 #
75 #xxx
76 # when starting must record original start (for ff)
77 # and new rebase basis
78 #
79 #    git-ffqrebase start [BASE]
80 #                # records previous HEAD so it can be overwritten
81 #                # records base for future git-ffqrebase
82 #    git-ffqrebase set-base BASE
83 #    git-ffqrebase <git-rebase options>
84 #    git-ffqrebase finish
85 #    git-ffqrebase status [BRANCH]
86 #
87 #  refs/ffqrebase-prev/BRANCH    BRANCH may be refs/...; if not it means
88 #  refs/ffqrebase-base/BRANCH      refs/heads/BRANCH
89 #                               zero, one, or both of these may exist
90 #
91 # git-debrebase without start, if already started, is willing
92 # to strip pseudomerges provided that they overwrite exactly
93 # the previous HEAD
94 #  xxxx is this right ?  what matters is have we pushed
95 #    I think in fact the right answer is:
96 #       git-debrebase always strips out pseudomerges from its branch
97 #       a pseudomerge is put in at the time we want to push
98 #       at that time, we make a pseudomerge of the remote tracking
99 #           branch (if raw git) or the dgit view (if dgit)
100 #       for raw git git-ffqrebase, do want preciseley to record
101 #           value of remote tracking branch or our branch, on start, so we
102 #           overwrite only things we intend to
103 #  the previous pseudomerge    check for tags and remote branches ?
104
105 use strict;
106
107 use Memoize;
108 use Carp;
109 use POSIX;
110 use Data::Dumper;
111
112 use Debian::Dgit qw(:DEFAULT $wa);
113
114 sub badusage ($) {
115     my ($m) = @_;
116     die "bad usage: $m\n";
117 }
118
119 sub cfg ($) {
120     my ($k) = @_;
121     $/ = "\0";
122     my @cmd = qw(git config -z);
123     push @cmd, qw(--get-all) if wantarray;
124     push @cmd, $k;
125     my $out = cmdoutput @cmd;
126     return split /\0/, $out;
127 }
128
129 memoize('cfg');
130
131 sub get_commit ($) {
132     my ($objid) = @_;
133     my ($type,$data) = git_cat_file $objid;
134     die unless $type eq 'commit';
135     $data =~ m/(?<=\n)\n/;
136     return ($`,$');
137 }
138
139 sub D_DEB ()     { return 0x1; } # debian/ (not including debian/patches/)
140 sub D_UPS ()     { return 0x2; } # upstream files
141 sub D_PAT_ADD () { return 0x4; } # debian/patches/ extra patches at end
142 sub D_PAT_OTH () { return 0x8; } # debian/patches other changes
143
144 our $rd = ".git/git-debrebase";
145 our $ud = "$rd/work";
146 our @git = qw(git);
147
148 sub get_differs ($$) {
149     my ($x,$y) = @_;
150     # This resembles quiltify_trees_differ, in dgit, a bit.  But: we
151     # don't care about modes, or dpkg-source-unrepresentable changes,
152     # and we don't need the plethora of different modes.
153
154     my $differs = 0;
155
156     my $f;
157     my $rundiff = sub {
158         my ($opts, $sfx, $fn) = @_;
159         $opts //= [qw(--name-only)];
160         my @cmd = (@git, qw(diff-tree -z --no-renames));
161         push @cmd, @$opts;
162         push @cmd, "$_:$sfx" foreach $x, $y;
163         my $diffs = cmdoutput @bcmd, $x, $y;
164         foreach $f (split /\0/, $diffs) { $fn->(); }
165     };
166
167     $rundiff(undef, '', sub {
168         $differs |= $f eq 'debian' ? D_DEB : D_UPS;
169     });
170
171     if ($differs & D_DEB) {
172         $differs &= ~D_DEB;
173         $rundiff(undef, ':debian', sub {
174             $differs |= $f eq 'patches' ? D_PAT_OTH : D_DEB;
175         });
176     }
177
178     if ($differs & D_PAT_OTH) {
179         $rundiff([qw(--name-status)], ':debian/patches', sub {
180             
181         });
182         
183     
184         
185         }
186             $f eq '
187
188 sub commit_pr_info ($) {
189     my ($r) = @_;
190     return Data::Dumper->dump([$r], [qw(commit)]);
191 }
192
193 sub calculate_committer_authline () {
194     my $c = cmdoutput @git, qw(commit-tree --no-gpg-sign -m),
195         'DUMMY COMMIT (git-debrebase)', "HEAD:";
196     my ($h,$m) = get_commit $c;
197     $h =~ m/^committer .*$/m or confess "($h) ?";
198     return $&;
199 }
200
201 # classify returns an info hash like this
202 #   CommitId => $objid
203 #   Hdr => # commit headers, including 1 final newline
204 #   Msg => # commit message (so one newline is dropped)
205 #   Tree => $treeobjid
206 #   Type => (see below)
207 #   Parents = [ {
208 #       Ix => $index # ie 0, 1, 2, ...
209 #       CommitId
210 #       Differs => return value from get_differs
211 #       IsOrigin
212 #       IsDggitImport => 'orig' 'tarball' 'unpatched' 'package' (as from dgit)
213 #     } ...]
214 #   NewMsg => # commit message, but with any [dgit import ...] edited
215 #             # to say "[was: ...]"
216 #
217 # Types:
218 #   Packaging
219 #   Upstream
220 #   AddPatches
221 #   Mixed
222 #   Unknown
223 #
224 #   Pseudomerge
225 #     has additional entres in classification result
226 #       Overwritten = [ subset of Parents ]
227 #       Contributor = $the_remaining_Parent
228 #
229 #   DgitImportUnpatched
230 #     has additional entry in classification result
231 #       OrigParents = [ subset of Parents ]
232 #
233 #   BreakwaterUpstreamMerge
234 #     has additional entry in classification result
235 #       OrigParents = [ subset of Parents ]
236
237 sub classify ($) {
238     my ($objid) = @_;
239
240     my ($h,$m) = get_commit $objid;
241
242     my ($t) = $h =~ m/^tree (\w+)$/m or die $objid;
243     my (@ph) = $h =~ m/^parent (\w+)$/m;
244     my @p;
245
246     my $r = {
247         CommitId => $objid,
248         Hdr => $h,
249         Msg => $m,
250         Tree => $t,
251         Parents => \@p,
252     };
253
254     foreach my $ph (@ph) {
255         push @p, {
256             Ix => $#p,
257             CommitId => $ph,
258             Differs => (get_differs $t, $ph),
259         };
260     }
261
262     my $classify = sub {
263         my ($type, @rest) = @_;
264         $r = { %$r, Type => $type, @rest };
265         return $r;
266     };
267     my $unknown = sub {
268         my ($why) = @_;
269         $r = { %$r, Type => qw(Unknown) };
270         return $r;
271     };
272
273     if (@p == 1) {
274         my $d = $r->{Parents}[0]{Differs};
275         if ($d == D_PAT_ADD) {
276             return $classify->(qw(AddPatches));
277         } elsif ($d & (D_PAT_ADD|D_PAT_OTH)) {
278             return $unknown->("edits debian/patches");
279         } elsif ($d == D_DEB) {
280             my ($ty,$dummy) = git_cat_file "$ph[0]:debian";
281             if ($ty eq 'tree') {
282                 return $classify->(qw(Packaging));
283             } elsif ($ty eq 'missing') {
284                 return $classify->(qw(BreakwaterStart));
285             } else {
286                 return $unknown->("parent's debian is not a directory");
287             }
288         } elsif ($d == D_UPS) {
289             return $classify->(qw(Upstream));
290         } elsif ($d == (D_DEB|D_UPS)) {
291             return $classify->(qw(Mixed));
292         } elsif ($d == 0) {
293             return $unknown->("no changes");
294         } else {
295             confess "internal error $objid ?";
296         }
297     }
298     if (!@p) {
299         return $unknown->("origin commit");
300     }
301
302     my @identical = grep { !$_->{Differs} } @p;
303     if (@p == 2 && @identical == 1) {
304         my @overwritten = grep { $_->{Differs} } @p;
305         confess "internal error $objid ?" unless @overwritten==1;
306         return $classify->(qw(Pseudomerge),
307                            Overwritten => $overwritten[0],
308                            Contributor => $identical[0]);
309     }
310     if (@p == 2 && @identical == 2) {
311         my @bytime = nsort_by {
312             my ($ph,$pm) = get_commit $_->{CommitId};
313             $ph =~ m/^committer .* (\d+) [-+]\d+$/m or die "$_->{CommitId} ?";
314             $1;
315         } @p;
316         return $classify->(qw(Pseudomerge),
317                            SubType => qw(Ambiguous),
318                            Overwritten => $bytime[0],
319                            Contributor => $bytime[1]);
320     }
321     foreach my $p (@p) {
322         my ($p_h, $p_m) = get_commit $p;
323         $p->{IsOrigin} = $p_h !~ m/^parent \w+$/m;
324         ($p->{IsDgitImport},) = $p_m =~ m/^\[dgit import ([0-9a-z]+) .*\]$/m;
325     }
326     my @orig_ps = grep { ($_->{IsDgitImport}//'X') eq 'orig' } @p;
327     my $m2 = $m;
328     if (!(grep { !$_->{IsOrigin} } @p) and
329         (@orig_ps >= @p - 1) and
330         $m2 =~ s{^\[(dgit import unpatched .*)\]$}{[was: $1]}m) {
331         $r->{NewMsg} = $m2;
332         return $classify->(qw(DgitImportUnpatched),
333                            OrigParents => \@orig_ps);
334     }
335
336     my ($stype, $series) = git_cat_file "$t:debian/patches/series";
337     my $haspatches = $stype ne 'missing' && $series =~ m/^\s*[^#\n\t ]/m;
338
339     # How to decide about l/r ordering of breakwater merges ?  git
340     # --topo-order prefers to expand 2nd parent first.  There's
341     # already an easy rune to look for debian/ history anyway (git log
342     # debian/) so debian breakwater branch should be 1st parent; that
343     # way also there's also an easy rune to look for the upstream
344     # patches (--topo-order).
345     if (@p == 2 &&
346         !$haspatches &&
347         !$p[0]{IsOrigin} && # breakwater merge never starts with an origin
348         !($p[0]{Differs} & ~D_DEB) &&
349         !($p[1]{Differs} & ~D_UPS)) {
350         return $classify->(qw(BreakwaterUpstreamMerge),
351                            OrigParents => [ $p[1] ]);
352     }
353     # xxx multi-.orig upstreams
354
355     return $unknown->("complex merge");
356 }
357
358 sub walk ($;$$);
359 sub walk ($;$$) {
360     my ($input,
361         $nogenerate,$report) = @_;
362     # => ($tip, $breakwater_tip)
363     # (or nothing, if $nogenerate)
364
365     # go through commits backwards
366     # we generate two lists of commits to apply:
367     # breakwater branch and upstream patches
368     my (@brw_cl, @upp_cl, @processed);
369     my %found;
370     my $upp_limit;
371     my @pseudomerges;
372
373     my $cl;
374     my $xmsg = sub {
375         my ($appendinfo) = @_;
376         my $ms = $cl->{Msg};
377         chomp $ms;
378         $ms .= "\n\n[git-debrebase $appendinfo]\n";
379         return (Msg => $ms);
380     };
381     my $rewrite_from_here = sub {
382         my $sp_cl = { SpecialMethod => 'StartRewrite' };
383         push @brw_cl, $sp_cl;
384         push @processed, $sp_cl;
385     };
386     my $cur = $input;
387
388     my $prdelim = "";
389     my $prprdelim = sub { print $report $prdelim if $report; $prdelim=""; };
390
391     my $prline = sub {
392         return unless $report;
393         print $report $prdelim, @_;
394         $prdelim = "\n";
395     };
396
397     my $bomb = sub { # usage: return $bomb->();
398         print $report " Unprocessable" if $report;
399         $prprdelim->();
400         if ($nogenerate) {
401             return (undef,undef);
402         }
403         die "commit $cur: Cannot cope with this commit";
404     };
405
406     my $build;
407     my $breakwater;
408
409     my $build_start = sub {
410         my ($msg, $parent) = @_;
411         $prline->(" $msg");
412         $build = $parent;
413         no warnings qw(exiting); last;
414     };
415
416     for (;;) {
417         $cl = classify $cur;
418         my $ty = $cl->{Type};
419         my $st = $cl->{SubType};
420         $prline->("$cl->{CommitId} $cl->{Type}");
421         $found{$ty. ( defined($st) ? "-$st" : '' )}++;
422         push @processed, $cl;
423         my $p0 = @{ $cl->{Parents} }==1 ? $cl->{Parents}[0]{CommitId} : undef;
424         if ($ty eq 'AddPatches') {
425             $cur = $p0;
426             $rewrite_from_here->();
427             next;
428         } elsif ($ty eq 'Packaging') {
429             push @brw_cl, $cl;
430             $cur = $p0;
431             next;
432         } elsif ($ty eq 'BreakwaterStart') {
433             $build_start->('FirstPackaging', $cur);
434         } elsif ($ty eq 'Upstream') {
435             push @upp_cl, $cl;
436             $cur = $p0;
437             next;
438         } elsif ($ty eq 'Mixed') {
439             my $queue = sub {
440                 my ($q, $wh) = @_;
441                 my $cls = { $cl, $xmsg->("split mixed commit: $wh part") };
442                 push @$q, $cls;
443             };
444             $queue->(\@brw_cl, "debian");
445             $queue->(\@upp_cl, "upstream");
446             $rewrite_from_here->();
447             $cur = $p0;
448             next;
449         } elsif ($ty eq 'Pseudomerge') {
450             print $report " Contributor=$ty->{Contributor}" if $report;
451             push @pseudomerges, $cl;
452             $rewrite_from_here->();
453             $cur = $ty->{Contributor};
454             next;
455         } elsif ($ty eq 'BreakwaterUpstreamMerge') {
456             $build_start->("PreviousBreakwater", $cur);
457         } elsif ($ty eq 'DgitImportUnpatched') {
458             my $pm = $pseudomerges[-1];
459             if (defined $pm) {
460                 # To an extent, this is heuristic.  Imports don't have
461                 # a useful history of the debian/ branch.  We assume
462                 # that the first pseudomerge after an import has a
463                 # useful history of debian/, and ignore the histories
464                 # from later pseudomerges.  Often the first pseudomerge
465                 # will be the dgit import of the upload to the actual
466                 # suite intended by the non-dgit NMUer, and later
467                 # pseudomerges may represent in-archive copies.
468                 my $ovwrs = $pm->{Overwritten};
469                 printf $report " PM=%s \@Overwr:%d", $pm, (scalar @$ovwrs)
470                     if $report;
471                 if (@$ovwrs != 1) {
472                     return $bomb->();
473                 }
474                 my $ovwr = $ovwrs->[0]{CommitId};
475                 printf $report " Overwr=%s", $ovwr if $report;
476                 # This import has a tree which is just like a
477                 # breakwater tree, but it has the wrong history.  It
478                 # ought to have the previous breakwater (which the
479                 # pseudomerge overwrote) as an ancestor.  That will
480                 # make the history of the debian/ files correct.  As
481                 # for the upstream version: either it's the same as
482                 # was ovewritten (ie, same as the previous
483                 # breakwater), in which case that history is precisely
484                 # right; or, otherwise, it was a non-gitish upload of a
485                 # new upstream version.  We can tell these apart by
486                 # looking at the tree of the supposed upstream.
487                 push @brw_cl, {
488                     %$cl,
489                     SpecialMethod => 'DgitImportDebianUpdate',
490                     $xmsg->("convert dgit import: debian changes")
491                 };
492                 my $differs = (get_differs $ovwr, $cl->{Tree});
493                 printf $report " Differs=%#x", $differs if $report;
494                 if ($differs & D_UPS) {
495                     printf $report " D_UPS" if $report;
496                     # This will also trigger if a non-dgit git-based NMU
497                     # deleted .gitignore (which is a thing that some of
498                     # the existing git tools do if the user doesn't
499                     # somehow tell them not to).  Ah well.
500                     push @brw_cl, {
501                         %$cl,
502                         SpecialMethod => 'DgitImportUpstreamUpdate',
503                         $xmsg->("convert dgit import: upstream changes")
504                     };
505                 }
506                 $prline->(" Import");
507                 $rewrite_from_here->();
508                 $upp_limit //= $#upp_cl; # further, deeper, patches discarded
509                 $cur = $ovwr;
510                 next;
511             } else {
512                 # Everything is from this import.  This kind of import
513                 # is already in valid breakwater format, with the
514                 # patches as commits.
515                 printf $report " NoPM" if $report;
516                 # last thing we processed will have been the first patch,
517                 # if there is one; which is fine, so no need to rewrite
518                 # on account of this import
519                 $build_start->("ImportOrigin", $cur);
520             }
521             die "$ty ?";
522         } else {
523             return $bomb->();
524         }
525     }
526     $prprdelim->();
527     return if $nogenerate;
528
529     # Now we build it back up again
530
531     workarea_fresh();
532
533     my $rewriting = 0;
534
535     my $rm_tree_cached = sub {
536         my ($subdir) = @_;
537         runcmd @git, qw(rm --quiet -rf --cached), $subdir;
538     };
539     my $read_tree_debian = sub {
540         my ($treeish) = @_;
541         $rm_tree_cached->(qw(debian));
542         runcmd @git, qw(read-tree --prefix=debian/), "$treeish:debian";
543     };
544     my $read_tree_upstream = sub {
545         my ($treeish) = @_;
546         runcmd @git, qw(read-tree), $treeish;
547         $read_tree_debian->($build);
548     };
549  
550     my $committer_authline = calculate_committer_authline();
551
552     in_workarea sub {
553         mkdir $rd or $!==EEXIST or die $!;
554         my $current_method;
555         foreach my $cl (qw(Debian), (reverse @brw_cl),
556                         { SpecialMethod => 'RecordBreakwaterTip' },
557                         qw(Upstream), (reverse @upp_cl)) {
558             if (!ref $cl) {
559                 $current_method = $cl;
560                 next;
561             }
562             my $method = $cl->{SpecialMethod} // $current_method;
563             my @parents = ($build);
564             my $cltree = $cl->{CommitId};
565             if ($method eq 'Debian') {
566                 $read_tree_debian->($cltree);
567             } elsif ($method eq 'Upstream') {
568                 $read_tree_upstream->($cltree);
569             } elsif ($method eq 'StartRewrite') {
570                 $rewriting = 1;
571                 next;
572             } elsif ($method eq 'RecordBreakwaterTip') {
573                 $breakwater = $build;
574                 next;
575             } elsif ($method eq 'DgitImportDebianUpdate') {
576                 $read_tree_debian->($cltree);
577                 $rm_tree_cached->(qw(debian/patches));
578             } elsif ($method eq 'DgitImportUpstreamUpdate') {
579                 $read_tree_upstream->($cltree);
580                 push @parents, map { $_->{CommitId} } @{ $cl->{OrigParents} };
581             } else {
582                 confess "$method ?";
583             }
584             $rewriting ||= $cl ne pop @processed;
585             my $newtree = cmdoutput @git, qw(write-tree);
586             my $ch = $cl->{Hdr};
587             $ch =~ s{^tree .*}{tree $newtree}m or confess "$ch ?";
588             $ch =~ s{^parent .*\n}{}m;
589             $ch =~ s{(?=^author)}{
590                 map { "parent $_\n" } @parents
591             }me or confess "$ch ?";
592             if ($rewriting) {
593                 $ch =~ s{^committer .*$}{$committer_authline}m
594                     or confess "$ch ?";
595             }
596             my $cf = "$rd/m$rewriting";
597             open CD, ">", $cf or die $!;
598             print CD $ch, "\n", $cl->{Msg} or die $!;
599             close CD or die $!;
600             my @cmd = (@git, qw(hash-object));
601             push @cmd, qw(-w) if $rewriting;
602             push @cmd, qw(-t commit), $cf;
603             my $newcommit = cmdoutput @cmd;
604             confess "$ch ?" unless $rewriting or $newcommit eq $cl->{CommitId};
605             $build = $newcommit;
606         }
607     };
608
609     runcmd @git, qw(diff-tree --quiet), $input, $build;
610
611     return ($build, $breakwater);
612 }
613
614 sub get_head () { return git_rev_parse qw(HEAD); }
615
616 sub update_head ($$$) {
617     my ($old, $new, $mrest) = @_;
618     runcmd @git, qw(update-ref -m), "git-debrebase $mrest", $new, $old;
619 }
620
621 sub cmd_launder () {
622     badusage "no arguments to launder allowed";
623     my $old = get_head();
624     my ($tip,$breakwater) = walk $old;
625     update_head $old, $tip, 'launder';
626     # no tree changes except debian/patches
627     runcmd @git, qw(rm --quiet -rf debian/patches);
628     printf "# breakwater tip\n%s\n", $breakwater;
629 }
630
631 sub cmd_analyse () {
632     die if ($ARGV[0]//'') =~ m/^-/;
633     badusage "too many arguments to analyse" if @ARGV>1;
634     my ($old) = @ARGV;
635     if (defined $old) {
636         $old = git_rev_parse $old;
637     } else {
638         $old = get_head();
639     }
640     my ($dummy,$breakwater) = walk $old, 1,*STDOUT;
641     print "$breakwater BREAKWATER\n";
642     STDOUT->error and die $!;
643 }
644
645 my $toplevel = cmdoutput @git, qw(rev-parse --show-toplevel);
646 chdir $toplevel or die "chdir $toplevel: $!";
647
648 my $cmd = shift @ARGV;
649 my $cmdfn = $cmd;
650 $cmdfn =~ y/-/_/;
651 $cmdfn = ${*::}{"cmd_$cmdfn"};
652
653 $cmdfn or badusage "unknown git-debrebase sub-operation $cmd";
654 $cmdfn->();