Re: [PATCH] Fix typo from commit 0c5e67e4 "xfstests 273: fix a typo"

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 03/07/2013 08:56 AM, Mark Tinguely wrote:
On 03/07/13 06:41, rjohnston@xxxxxxx wrote:
When updating the patch, the redirect was incorrect.

Reported-by: Wang Sheng-Hui<shhuiw@xxxxxxxxx>
Signed-off-by: Rich Johnston<rjohnston@xxxxxxx>

---

  273 |    2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

Index: b/273
===================================================================
--- a/273
+++ b/273
@@ -92,7 +92,7 @@ _porter()
          exit
      fi

-    cp -r $SCRATCH_MNT/origin $SCRATCH_MNT/sub_$_suffix>$seq.full 2>>&1
+    cp -r $SCRATCH_MNT/origin $SCRATCH_MNT/sub_$_suffix>>$seq.full 2>&1
      if [ $? -ne 0 ]
      then
          echo "_porter $_suffix not complete"



The original patch looked okay:

    http://oss.sgi.com/archives/xfs/2013-02/msg00518.html

Should the bad patch be reverted and the original one put in?

--Mark.
Yes the patch would not apply to top of tree and I placed the redirect in the incorrect position when I updated the and committed the patch.

I have reverted and applied the patch correctly this time. :)


commit a1f6dd481218cd95c8d48ed684070cc2b0f62db2
Author: Wang Sheng-Hui <shhuiw@xxxxxxxxx>
Date:   Wed Feb 27 06:47:54 2013 +0000

    xfstests 273: fix a typo

Make the porter log cp failure into $seq.full by appending, not overwriting,
    which can help debug.

    Signed-off-by: Wang Sheng-Hui <shhuiw@xxxxxxxxx>
    Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx>
    Signed-off-by: Rich Johnston <rjohnston@xxxxxxx>

diff --git a/273 b/273
index 67f3d6b..639e280 100755
--- a/273
+++ b/273
@@ -92,7 +92,7 @@ _porter()
                exit
        fi

-       cp -r $SCRATCH_MNT/origin $SCRATCH_MNT/sub_$_suffix >$seq.full 2>&1
+       cp -r $SCRATCH_MNT/origin $SCRATCH_MNT/sub_$_suffix >>$seq.full 2>&1
        if [ $? -ne 0 ]
        then
                echo "_porter $_suffix not complete"

commit 2a599a54aba600c299b8209d802f97c51a164dcb
Author: Rich Johnston <rjohnston@xxxxxxx>
Date:   Thu Mar 7 09:21:30 2013 -0600

    Revert "xfstests 273: fix a typo"

    This reverts commit 0c5e67e46e476b70290033efc7feb784f7ecbab1.
    Will re-apply the correct patch.

_______________________________________________
xfs mailing list
xfs@xxxxxxxxxxx
http://oss.sgi.com/mailman/listinfo/xfs


[Index of Archives]     [Linux XFS Devel]     [Linux Filesystem Development]     [Filesystem Testing]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux