On Wed, Mar 09, 2005 at 03:06:38AM -0500, Ryan Anderson wrote:
> Automatically append a semi-random version if the tree we're building
> isn't tagged in BitKeeper and CONFIG_LOCALVERSION_AUTO is set.
>
> This fixes the case when Linus (or someone else) does a release and tags
> it, someone else does a build of that release tree (i.e, 2.6.11), and
> installs it. Later, before another release occurs (i.e, -rc1), another
> build happens, and the actual, released 2.6.11 is overwritten with the
> -current tree.
>
> Two approachs are present here, a Perl version that is setup to handle
> other automatic version appends (i.e, a CVS version shouldn't be much
> effort to add), and a simplistic shell version that depends on "md5sum".
> Both approaches generate the same hash.
Please skip the shell version - add a note in Kconfig that enabling this
option requires perl.
>
> #export INSTALL_PATH=/boot
>
> +# If CONFIG_LOCALVERSION_AUTO is set, we automatically perform some tests
> +# and try to determine if the current source tree is a release tree, of any
> sort,
> +# or if is a pure development tree.
> +# A 'release tree' is any tree with a BitKeeper TAG associated with it.
> +# The primary goal of this is to make it safe for a native BitKeeper user to
> +# build a release tree (i.e, 2.6.9) and also to continue developing against
> the
> +# current Linus tree, without having the Linus tree overwrite the 2.6.9 tree
> +# when installed.
> +#
> +# (In the future, CVS and SVN support will be added as well.)
> +
> +ifeq ($(CONFIG_LOCALVERSION_AUTO),y)
> + ifeq ($(shell ls -d $(srctree)/BitKeeper
> 2>/dev/null),$(srctree)/BitKeeper)
> + localversion-bk := $(shell
> $(srctree)/scripts/setlocalversion.sh $(srctree) $(objtree))
> + LOCALVERSION := $(LOCALVERSION)$(localversion-bk)
> + endif
> +endif
Move the logic to determine the SCM system into the perl script.
And do not assume bk, select more generic names.
Also use:
ifdef CONFIG_LOCALVERSION_AUTO
like in rest of Makefile.
Something like this:
ifdef CONFIG_LOCALVERSION_AUTO
LOCALVERSION += $(CONFIG_SHELL) $(srctree)/scripts/setlocalversion.sh
$(srctree)
endif
note - perl script does not use objtree.
diff -Nru a/scripts/setlocalversion b/scripts/setlocalversion
> --- /dev/null Wed Dec 31 16:00:00 196900
> +++ b/scripts/setlocalversion 2005-03-09 02:51:15 -05:00
> @@ -0,0 +1,62 @@
> +#!/usr/bin/perl
> +# Copyright 2004 - Ryan Anderson <[EMAIL PROTECTED]> GPL v2
> +
> +use strict;
> +use warnings;
> +use Digest::MD5;
> +require 5.006;
> +
> +if (@ARGV != 2) {
> + print <<EOT;
> +Usage: setlocalversion <srctree> <objtree>
wrong - objtree is not needed
> +EOT
> + exit(1);
> +}
> +
> +my $debug = 0;
remove debug for such a simple script
> +
> +my ($srctree,$objtree) = @ARGV;
> +
> +my @LOCALVERSIONS = ();
> +
> +# BitKeeper Version Checks
> +
> +# We are going to use the following commands to try and determine if
> +# this repository is at a Version boundary (i.e, 2.6.10 vs 2.6.10 + some
> patches)
> +# We currently assume that all meaningful version boundaries are marked by a
> tag.
> +# We don't care what the tag is, just that something exists.
> +
> [EMAIL PROTECTED] ~/dev/linux/local$ T=`bk changes -r+ -k`
> [EMAIL PROTECTED] ~/dev/linux/local$ bk prs -h -d':TAG:\n' -r$T
- to be deleted?
> +
> +sub do_bk_checks {
> + chdir($srctree);
> + my $changeset = `bk changes -r+ -k`;
> + chomp $changeset;
> + my $tag = `bk prs -h -d':TAG:' -r'$changeset'`;
> +
> + printf("ChangeSet Key = '%s'\nTAG = '%s'\n", $changeset, $tag) if
> ($debug > 0);
> +
> + if (length($tag) == 0) {
if this imply that this is not a release tree then please write so - to
be consistent with comment in top-level makefile.
Thats good for poor sould like me that does not know perl.
> + # We do not have a tag at the Top of Tree, so we need to
> generate a localversion file
> + # We'll use the given $changeset as input into this.
> + my $localversion = Digest::MD5::md5_hex($changeset);
> + $localversion = substr($localversion,0,8);
> +
> + printf("localversion = '%s'\n",$localversion) if ($debug > 0);
> +
> + push @LOCALVERSIONS, "BK" . $localversion;
> +
> + }
> +}
> +
> +
> +if ( -d "BitKeeper" ) {
> + my $bk = `which bk`;
> + chomp $bk;
> + if (length($bk) != 0) {
> + do_bk_checks();
> + }
> +}
And this part should be prepared for cvs+svn.
Sam
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
