Dear diary, on Mon, May 30, 2005 at 01:00:10AM CEST, I got a letter where Mark Allen <mrallen1@yahoo.com> told me that... > Here're a couple of pretty simple patches against cg-log and cg-status (two cogito > commands I use a lot) for the non-GNU (or at least older, forked GNU) tools on Darwin. > @@ -148,10 +148,10 @@ > fi > > date=(${rest#*> }) > - pdate="$(showdate $date)" > + pdate="$(date -u -r $date)" Hmm, coudlnt' showdate be fixed instead then? And $date is not a file so -r $date makes no sense to me - what am I missing? > if [ "$pdate" ]; then > - echo -n $color$key $rest | sed "s/>.*/> $pdate/" > - echo $coldefault > + echo -n $color$key $rest | sed "s/>.*/> $pdate/" > + echo -n $coldefault I'm lost on this one too. Why do you introduce the -n? > else > echo $color$key $rest $coldefault > fi > @@ -168,11 +168,8 @@ > if [ -n "$list_files" ]; then > list_commit_files "$tree1" "$tree2" > fi > - echo; sed -re ' > - / *Signed-off-by:.*/Is//'$colsignoff'&'$coldefault'/ > - / *Acked-by:.*/Is//'$colsignoff'&'$coldefault'/ > - s/./ &/ > - ' > + echo; sed -e "/ *Signed-off-by:.*/s/ *Signed-off-by:.*/$colsignoff&$coldefault/" -e"/ > +*Acked-by:.*/s/ *Acked-by:.*/$colsignoff&$coldefault/" -e "s/./ &/" > + Is it necessary to take away the newlines? What is the real problem, actually? Just the I flag? Could you please sign the patch off, and send it as text/plain or inline the message body? Thanks, -- Petr "Pasky" Baudis Stuff: http://pasky.or.cz/ C++: an octopus made by nailing extra legs onto a dog. -- Steve Taylor - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.htmlReceived on Mon May 30 09:17:07 2005
This archive was generated by hypermail 2.1.8 : 2005-05-30 09:17:08 EST