[PATCH 01/18] xfstests: Remove obsolete remake script

[PATCH 01/18] xfstests: Remove obsolete remake script

[Philip White]

From: Phil White <pwhite@sgi.com>


This is a rebasing & resubmit of a dchinner patch.  His comments on the
original:
-----------------
We don't ever do wholesale rebuilds of the output files anymore, so
kill the script that does it to reduce the dependency on common and
common.rc.
-----------------

Signed-off-by: Phil White <pwhite@sgi.com>
---
 remake |   72 ----------------------------------------------------------------
 1 files changed, 0 insertions(+), 72 deletions(-)
 delete mode 100755 remake

diff --git a/remake b/remake
deleted file mode 100755
index a95aae7..0000000
--- a/remake
+++ /dev/null
@@ -1,72 +0,0 @@
-#! /bin/bash
-#
-# Copyright (c) 2000-2001 Silicon Graphics, Inc.  All Rights Reserved.
-#
-# This program is free software; you can redistribute it and/or
-# modify it under the terms of the GNU General Public License as
-# published by the Free Software Foundation.
-#
-# This program is distributed in the hope that it would be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program; if not, write the Free Software Foundation,
-# Inc.,  51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
-#
-#
-# Rebuild NNN.out files
-#
-# $Header: /build2/depot/linux/pcp/dev/qa/RCS/remake,v 2.11 1999/09/20 03:42:19 kenmcd Exp $
-#
-
-tmp=/tmp/$$
-trap "rm -f NO-PREVIOUS-OUTPUT $tmp.*; exit" 0 1 2 3 15
-
-# generic initialization
-iam=remake
-. ./common.rc
-
-. ./common
-
-[ -f check.time ] || touch check.time
-
-for seq in $list
-do
-    if [ ! -f $seq ]
-    then
-	echo "Remake: cannot find \"$seq\""
-    else
-	echo -n "$seq"
-	lasttime=`sed -n -e "/^$seq /s/.* //p" <check.time`
-	[ "X$lasttime" != X ] && echo -n " ${lasttime}s"
-	rm -f $seq.bak $seq.full.bak
-	if sh $seq >$seq.new 2>&1
-	then
-	    if [ -f $seq.out ]
-	    then
-		$diff $seq.out $seq.new
-		rm -f $seq.out.bad
-	    else
-		touch NO-PREVIOUS-OUTPUT
-		$diff NO-PREVIOUS-OUTPUT $seq.new
-		rm -f NO-PREVIOUS-OUTPUT
-	    fi
-	    echo ""
-	    [ -f $seq.out ] && mv $seq.out $seq.bak
-	    if [ -f $seq.full ]
-	    then
-		[ -f $seq.full.ok ] && mv $seq.full.ok $seq.full.bak
-		mv $seq.full $seq.full.ok
-	    fi
-	    mv $seq.new $seq.out
-	else
-	    echo " - failed (exit status $?)"
-	    sed 's/^/	/' $seq.new
-	    exit 1
-	fi
-    fi
-done
-
-exit 0
-- 
1.5.3.8

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

Re: [PATCH 01/18] xfstests: Remove obsolete remake script

[Carlos Maiolino]

On Thu, Mar 14, 2013 at 06:06:11AM -0700, Philip White wrote:
> From: Phil White <pwhite@sgi.com>
> 
> 
> This is a rebasing & resubmit of a dchinner patch.  His comments on the
> original:
> -----------------
> We don't ever do wholesale rebuilds of the output files anymore, so
> kill the script that does it to reduce the dependency on common and
> common.rc.
> -----------------
> 
> Signed-off-by: Phil White <pwhite@sgi.com>
> ---
>  remake |   72 ----------------------------------------------------------------
>  1 files changed, 0 insertions(+), 72 deletions(-)
>  delete mode 100755 remake
> 
> diff --git a/remake b/remake
> deleted file mode 100755
> index a95aae7..0000000
> --- a/remake
> +++ /dev/null
Looks good,

Reviewed-by: Carlos Maiolino <cmaiolino@redhat.com>

-- 
Carlos

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

Re: [PATCH 01/18] xfstests: Remove obsolete remake script

[Dave Chinner]

On Thu, Mar 14, 2013 at 06:06:11AM -0700, Philip White wrote:
> From: Phil White <pwhite@sgi.com>
> 
> 
> This is a rebasing & resubmit of a dchinner patch.  His comments on the
> original:
> -----------------
> We don't ever do wholesale rebuilds of the output files anymore, so
> kill the script that does it to reduce the dependency on common and
> common.rc.
> -----------------
> 
> Signed-off-by: Phil White <pwhite@sgi.com>

Hi Phil - it's great that you've picked this up, but it would have
been really nice to know that you were doing this. Just a short
email on the list with me CC'd or even just a private message on
IRC. Last I heard from anyone at SGI was that SGI still didn't want
to remove the old bench scripts. Now I find out that you've picked
up the patchset that removes it and forward ported it....

If you let me know you were doing this, I could have let you know
that my local version has several more patches in it and bug fixes
that I haven't posted out. Indeed, I coul dhave taken an hour to
apply all the changes to the most recent additions to xfstests and
posted an up-to-date version, because I've been maintaining the
patchset locally.

Indeed, one of my test VMs runs this version of xfstests all the
time:

$ sudo MKFS_OPTIONS="-b size=512" ./check -g auto
FSTYP         -- xfs (debug)
PLATFORM      -- Linux/x86_64 test-1 3.9.0-rc2-dgc+
MKFS_OPTIONS  -- -f -b size=512 /dev/vdb
MOUNT_OPTIONS -- /dev/vdb /mnt/scratch

generic/001      2s
generic/002      0s
generic/005      0s
....

So I could have saved you a lot of work if you'd simply told us that
SGI had decided that "dropping bench is OK". If you make a decision
like this, then please tell us when you make it - sometimes it will
save you a lot of unnecessary time and effort....

/me knows what his afternoon is going to be spent doing: updating
all the recent tests added to xfstests and posting a current patch
set....

----

For future reference for everyone, a few technical guidelines for
reposting patch sets written by other people.  This is important to
get right, as we all know that the correct attribution of the
original of the code has legal significance.

- when you are reposting a mostly unmodified patch written by
  someone else, you should not claim it as your own, nor strip off
  the original s-o-b lines. i.e. reposted patches should have a
  description that looks like:

From: Dave Chinner <dchinner@redhat.com>

<unmodified original patch description>

<[pwhite] modifications added (if any) during rebasing>

Signed-off-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Phil White <pwhite@sgi.com>

  See the way I have done this in the CRC patchset attributing
  various patches in the series to Christoph and how I've modified
  them from Christoph's original versions. e.g:

  http://oss.sgi.com/pipermail/xfs/2013-March/024766.html

  This is especially important if the patches contain copyright
  title updates - they can only be made by the author of the code in
  question. Hence if the patch is listed as "from X" and the
  copyright statement says "(c) Y." then that is a big problem...

- comments like "this is a rebasing and resubmit" don't belong in
  commit messages, they belong in the patch 0 description.  Patch 0
  should also contain a high level description of the patch set and
  outline the changes you've made to the original code.

- large patch series need to be correctly threaded with a patch 0
  describing the series. git-send-email should be able to do this
  for you automatically.

- rebasing a patch set to a current tree means you need to apply all
  the changes to new files that have been added since this patch set
  was originally written. Part of rebasing is "making new stuff
  work"... ;)

Cheers,

Dave.
-- 
Dave Chinner
david@fromorbit.com

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