[Rt-devel] Re: [PATCH] (#4638) cli cookie handling

[Jesse Vincent]

On Fri, May 07, 2004 at 08:47:02AM +0530, Abhijit Menon-Sen wrote:
> At 2004-05-07 08:23:51 +0530, ams at wiw.org wrote:
> >
> > The correct fix for this is the one I sent to rt-bugs@ on 2003-12-29,
> > in <20031229082831.A4759 at lustre.dyn.wiw.org>.
> 
> Not quite. The code has drifted a bit since then.
> How about something like the appended patch instead?
> 

This patch seems to miss:

         elsif (/^-o$/) {
-            $orderby = shift @ARGV;
+            $data{'orderby'} = shift @ARGV;
         }


I've applied this patch with the addition above. Was that the right
thing?