pull up r19513 from trunk
authorTom Yu <tlyu@mit.edu>
Fri, 20 Apr 2007 22:09:53 +0000 (22:09 +0000)
committerTom Yu <tlyu@mit.edu>
Fri, 20 Apr 2007 22:09:53 +0000 (22:09 +0000)
 r19513@cathode-dark-space:  kpkoch | 2007-04-20 16:41:42 -0400
 Ticket: new
 Tags: Pullup
 Target_Version: 1.6.1

 Add /REPOSITORY EXPORT option.

 Adjust abbreviations: /svntag is now 't' so /src can be 's' so /repository can have the abbreviation 'r.'

 Process /NOLOG.

 Write out ignored command line options as ones being passed to NMAKE (via build.pl).

 Generate GetOptions argument from the <Config> section of the config.xml file.

ticket: 5539
version_fixed: 1.6.1

git-svn-id: svn://anonsvn.mit.edu/krb5/branches/krb5-1-6@19515 dc483132-0cff-0310-8789-dd5450dbe970

src/windows/build/BKWconfig.xml
src/windows/build/bkw-automation.html
src/windows/build/bkw.pl

index 93167f3dca87379f02f89c351ecb63c0688018aa..30c77f91854bc683057200699220ec2b560cba3b 100644 (file)
@@ -8,6 +8,7 @@
             Parameters are grouped into Options, Directories, Repository and Environment.\r
 \r
             Attributes a parameter can have:\r
+            abbr: A one character abbreviation.\r
             def:  defined or not.  A value of 'A' for 'always' means the    \r
                option can't be negated on the command line.                 \r
             value:  string value, if the option can have a value.  If the option\r
             options:  space-delimited list, the 1st element is the default.\r
                Synonyms for a valid option are concatenated with '|'.      \r
             env: an environment variable will be set to value or deleted,  \r
-                depending on def.                                                    \r
+                depending on def.  If set and no value is given, '1' is used.                                 \r
             -->\r
         <!-- Options: -->\r
         <clean      def="0" />\r
-        <debug      def="0" />\r
-        <help       def="0" />\r
-        <logfile    def="1" value="bkw.pl.log" />\r
+        <config     def="1" abbr="f" value="bkwconfig.xml" />\r
+        <debug      def="0" abbr="d" />\r
+        <help       def="0" abbr="h|?" />\r
+        <logfile    def="1" abbr="l" value="bkw.pl.log" />\r
         <make       def="1" />\r
+        <nolog      def="0" />\r
         <package    def="1" />\r
-        <repository def="1" value="skip" options="skip checkout|co update|up" />\r
+        <repository def="A" abbr="r" value="skip" options="skip checkout|co export|ex update|up" />\r
         <sign       def="0" />\r
-        <verbose    def="0" />\r
+        <verbose    def="0" abbr="v" />\r
         <vverbose   def="0" />\r
 \r
         <!-- Directory settings: -->\r
         <!--    Sources will be checked out of repositories into <src>.  The structure\r
                 of the repositories will cause pismere/athena to be created under <src>.    -->\r
-        <src        def="A" value="C:\KfW" />        <!-- Must be absolute path. -->\r
-        <out        def="A" value="C:\KfW\public" /> <!-- Must be absolute path. -->\r
+        <src        def="A" abbr="s" value="C:\KfW" />        <!-- Must be absolute path. -->\r
+        <out        def="A" abbr="o" value="C:\KfW\public" /> <!-- Must be absolute path. -->\r
 \r
         <!-- Repository settings: -->\r
-        <cvstag     def="0" value="" />\r
-        <svntag     def="0" value="" />\r
-        <svnbranch  def="0" value="" />\r
+        <cvstag     def="0" abbr="c" value="" />\r
+        <svntag     def="0" abbr="t" value="" />\r
+        <svnbranch  def="0" abbr="b" value="" />\r
+        <username   def="0" abbr="u" value="" />    <!-- Needed for svn/plink.  Override from command line -->\r
         <CVSROOT    def="A" value=":kserver:cvs.mit.edu:/cvs/pismere" />\r
-        <SVNURL     def="A" value="svn.mit.edu" />   <!-- NB:  No protocol or slashes!! -->\r
-        <username   def="0" value="" />  <!-- Needed for svn/plink.  Override from command line -->\r
+        <SVNURL     def="A" value="svn.mit.edu" />  <!-- NB:  No protocol or slashes!! -->\r
 \r
         <!-- Environment variables: -->\r
         <KH_RELEASE def="1" env="1" value="OFFICIAL" options="OFFICIAL PRERELEASE PRIVATE" />\r
@@ -72,8 +75,8 @@
                         </Prunes>\r
                     <Requires>\r
                         <Switch dummy="foo" />\r
-                        <Switch name="REPOSITORY" value="CHECKOUT"/>\r
-                        <ErrorMsg text="Can't build source zip unless /REPOSITORY CHECKOUT is specified." />\r
+                        <Switch name="REPOSITORY" value="CHECKOUT|EXPORT"/>\r
+                        <ErrorMsg text="Can't build source zip unless /REPOSITORY CHECKOUT or /REPOSITORY EXPORT is specified." />\r
                         </Requires>\r
                     </Zip>\r
                 </Zips>\r
                             <ReleaseTag  value="%release%" /> <!-- Pick a string that won't appear in a path. -->\r
                             </Config>\r
                         <Files>\r
-                            <Include path="corebinaryfiles.xml" />  <!-- Includeed file is relative to location of bkw.pl. -->\r
+                            <Include path="corebinaryfiles.xml" />  <!-- Included file is relative to location of bkw.pl. -->\r
                             </Files> \r
                         </CopyList>\r
                 </Zip>\r
index 4583e54b7dd604051e03d475ff7143d9823427c7..ab0ef7d3c1e24318b30752ac84a6a36b6fa35acc 100644 (file)
                     &nbsp; /config /f path&nbsp;&nbsp; Path to config file. Default is \r
                     bkwconfig.xml.\r
                     <BR>\r
-                    &nbsp; /src /r dir&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; Source directory to use. \r
+                    &nbsp; /src /s dir&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; Source directory to use. \r
                     Should contain\r
                     <BR>\r
                     &nbsp;&nbsp;&nbsp; \r
                     <BR>\r
                 </TT><TT>&nbsp; /repository checkout | co \ What repository action to take.\r
                     <BR>\r
-                    &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; \r
-                    update&nbsp;&nbsp; | up&nbsp; ) Options are to checkout, update or\r
-                    <BR>\r
-                    &nbsp;&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; \r
-                    skip&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;/ take no \r
-                    action [skip].\r
-                    <BR>\r
+                    &nbsp;&nbsp;&nbsp; &nbsp;/r&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; \r
+                    update&nbsp;&nbsp; | up&nbsp;\ Options are to checkout, update, export<BR>\r
+                    &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;export&nbsp;&nbsp; \r
+                    | ex \ or take no action [skip].&nbsp;<BR>\r
+                    &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; \r
+                    skip<BR>\r
                     &nbsp; /username /u name username used to access svn if checking out.\r
                     <BR>\r
                     &nbsp; /cvstag /c tag&nbsp;&nbsp;&nbsp; use -r &lt;tag&gt;\r
                     <TAG>in cvs \r
 command <BR>&nbsp; /svnbranch /b tag use \r
-/branches/&lt;tag&gt;<TAG>&nbsp;instead of /trunk.<BR>&nbsp; /svntag /s tag&nbsp;&nbsp;&nbsp; use \r
+/branches/&lt;tag&gt;<TAG>&nbsp;instead of /trunk.<BR>&nbsp; /svntag /t tag&nbsp;&nbsp;&nbsp; use \r
 /tags/&lt;tag&gt;<TAG>&nbsp;instead of /trunk.<BR>&nbsp; /debug \r
 /d&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; Do&nbsp;debug make instead of \r
 release make. <BR>&nbsp; \r
index 8b8b1ab6fea09104c0d0e791a1288966065bcc50..08a168243f6540476436ee2602e2f52284927ab0 100644 (file)
@@ -41,17 +41,18 @@ Usage: $0 [options] NMAKE-options
   Options:\r
     /help /?           usage information (what you now see).\r
     /config /f path    Path to config file.  Default is bkwconfig.xml.\r
-    /srcdir /r dir     Source directory to use.  Should contain \r
+    /srcdir /s dir     Source directory to use.  Should contain \r
                        pismere/athena.  If cvstag or svntag is null, \r
                        the directory should be prepopulated.\r
     /outdir /o dir     Directory to be created where build results will go\r
     /repository checkout | co \\  What repository action to take.\r
-                update   | up  ) Options are to checkout, update or \r
-                skip          /  take no action [skip].\r
+       /r       update   | up \\  Options are to checkout, update, export\r
+                export   | ex \\  or skip (take no action).\r
+                skip        \r
     /username /u name  username used to access svn if checking out.\r
     /cvstag /c tag     use -r <tag> in cvs command\r
     /svnbranch /b tag  use /branches/<tag> instead of /trunk.\r
-    /svntag /s tag     use /tags/<tag> instead of /trunk.\r
+    /svntag /t tag     use /tags/<tag> instead of /trunk.\r
     /debug /d          Do debug make instead of release make.\r
     /[no]make          Control the make step.\r
     /clean             Build clean target.\r
@@ -89,27 +90,38 @@ sub main {
            'ignore_case', 'pass_through',\r
            'prefix_pattern=(--|-|\+|\/)',\r
            );\r
-    GetOptions($OPT,\r
-           'help|h|?',\r
-           'cvstag|c:s',\r
-           'svntag|s:s',\r
-           'svnbranch|b:s',\r
-           'src|r:s',\r
-           'out|o:s',\r
-           'debug|d',\r
-           'nodebug',\r
-           'config|f=s',\r
-           'logfile|l:s',\r
-           'nolog',\r
-           'repository:s',\r
-           'username|u:s',\r
-           'verbose|v',\r
-           'vverbose',\r
-           'make!',\r
-           'clean',\r
-           'package!',\r
-           'sign!',\r
-           );\r
+\r
+    local @goargs   = ('config|f=s');\r
+    if (!GetOptions($OPT, @goargs)) {\r
+        Usage();\r
+        exit(0);\r
+        }\r
+\r
+    if (! exists $OPT->{config}) {$OPT->{config}  = "bkwconfig.xml";}\r
+    my $configfile  = $OPT->{config};\r
+    print "Info -- Reading configuration from $configfile.\n";\r
+    my $xml         = new XML::Simple();\r
+    $config         = $xml->XMLin($configfile); ## Read in configuration file.\r
+\r
+    # Set up convenience variables:\r
+    local $odr  = $config->{Config};    ## Options, directories, repository, environment.\r
+\r
+    # Build argument description from Config section of the XML,\r
+    #  to parse the rest of the arguments:\r
+    local @xmlargs;\r
+    while (($sw, $val) = each %$odr) {\r
+        local $arg  = $sw;\r
+        if (exists $val->{abbr})    {$arg .= "|$val->{abbr}";}\r
+        if (exists $val->{value})   {       ## Can't do both negations and string values.\r
+            $arg .= ":s";\r
+            }\r
+        else {    \r
+            if (! ($val->{def} =~ /A/)) {$arg .= "!";}\r
+            }\r
+        push @xmlargs, $arg;\r
+        }\r
+\r
+    if (!GetOptions($OPT, @xmlargs)) {$OPT->{help} = 1;}\r
 \r
     if ( $OPT->{help} ) {\r
         usage();\r
@@ -120,15 +132,6 @@ sub main {
 \r
 ##++ Validate required conditions:\r
 \r
-    local $argvsize = @ARGV;\r
-    if ($argvsize > 0) {\r
-        print "Error -- invalid argument:  $ARGV[0]\n";\r
-        usage();\r
-        die;\r
-        }\r
-        \r
-    if (! exists $OPT->{config}) {$OPT->{config}  = "bkwconfig.xml";}\r
-\r
     # List of programs which must be in PATH:\r
     my @required_list = ('sed', 'awk', 'which', 'cat', 'rm', 'cvs', 'svn', 'doxygen', \r
                          'hhc', 'candle', 'light', 'makensis', 'nmake', 'plink', 'filever');\r
@@ -160,17 +163,8 @@ sub main {
 \r
 ##++ Assemble configuration from config file and command line:\r
 \r
-    my $configfile      = $OPT->{config};\r
     my $bOutputCleaned  = 0;\r
 \r
-    print "Info -- Reading configuration from $configfile.\n";\r
-\r
-    # Get configuration file:\r
-    my $xml = new XML::Simple();\r
-    $config = $xml->XMLin($configfile);\r
-    # Set up convenience variables:\r
-    local $odr  = $config->{Config};    ## Options, directories, repository, environment.\r
-\r
 #while ($v = each %$OPT) {print "$v: $OPT->{$v}\n";}\r
 \r
     # Scan the configuration for switch definitions:\r
@@ -265,11 +259,14 @@ sub main {
         die "Fatal -- Can't specify both /SVNTAG and /SVNBRANCH.";\r
         }\r
 \r
+    # /logfile and /nolog interact:\r
+    if ($odr->{nolog}->{def})  {$odr->{logfile}->{def} = 0;}\r
+\r
 ##-- Assemble configuration from config file and command line.\r
 \r
     local $rverb = $odr->{repository}->{value};\r
-    if ( ($rverb =~ /checkout/) && $clean) {\r
-        print "Warning -- Because sources afe being checked out, make clean will not be run.\n";\r
+    if ( (($rverb =~ /checkout/) || ($rverb =~ /export/)) && $clean) {\r
+        print "Warning -- Because sources are being checked out, make clean will not be run.\n";\r
         $clean  = $odr->{clean}->{def}    = 0;\r
         }\r
 \r
@@ -285,9 +282,18 @@ sub main {
             }\r
         }\r
 \r
-    if ( ($rverb =~ /checkout/) && (-d $wd) ){\r
+    print "Executing $cmdline\n";\r
+    local $argvsize = @ARGV;\r
+    if ($argvsize > 0) {\r
+        print "\nArguments for NMAKE: ";\r
+        map {print " $_ "} @ARGV;\r
+        print "\n";\r
+        }\r
+       \r
+    #                (------------------------------------------------)\r
+    if ( (-d $wd) && ( ($rverb =~ /export/) || ($rverb =~ /checkout/) ) ) {\r
         print "\n\nHEADS UP!!\n\n";\r
-        print "/REPOSITORY CHECKOUT will cause everything under $wd to be deleted.\n";\r
+        print "/REPOSITORY ".uc($rverb)." will cause everything under $wd to be deleted.\n";\r
         print "If this is not what you intended, here's your chance to bail out!\n\n\n";\r
         print "Are you sure you want to remove everything under $wd? ";\r
         my $char = getc;\r
@@ -305,8 +311,6 @@ sub main {
         $l->no_die_handler;        ## Needed so XML::Simple won't throw exceptions.\r
         }\r
 \r
-    print "Executing $cmdline\n";\r
-       \r
 ##++ Begin repository action:\r
     if ($rverb =~ /skip/) {print "Info -- *** Skipping repository access.\n"    if ($verbose);}\r
     else {\r
@@ -328,7 +332,7 @@ sub main {
             $cvscmdroot .= " -r $odr->{cvstag}->{value}";\r
             }\r
 \r
-        if ($rverb =~ /checkout/) {        \r
+        if (($rverb =~ /checkout/) || ($rverb =~ /export/)) {        \r
             chdir($src)                                     or die "Fatal -- couldn't chdir to $src\n";\r
             print "Info -- chdir to ".`cd`."\n"             if ($verbose);\r
             my @cvsmodules    = (    \r
@@ -337,11 +341,11 @@ sub main {
                 'pismere/athena/util/lib/getopt', \r
                 'pismere/athena/util/guiwrap'\r
                 );\r
-\r
+            local $logging = $odr->{logfile}->{def} ? ">> $odr->{logfile}->{value} 2>&1" : " ";\r
             foreach my $module (@cvsmodules) {\r
                 local $cvscmd = $cvscmdroot." ".$module;\r
                 if ($verbose) {print "Info -- cvs command: $cvscmd\n";}\r
-                !system($cvscmd)    or die "Fatal -- command \"$cvscmd\" failed; return code $?\n";\r
+                !system("$cvscmd") or die "Fatal -- command \"$cvscmd\" failed; return code $?\n";\r
                 }\r
             }\r
         else {                ## Update.\r
@@ -364,8 +368,12 @@ sub main {
         chdir($krb5dir)                                 or die "Fatal -- Couldn't chdir to $krb5dir";\r
         print "Info -- chdir to ".`cd`."\n"             if ($verbose);\r
         my $svncmd = "svn $rverb ";\r
-        if ($rverb =~ /checkout/) {        # Append the rest of the checkout command:\r
+        if (($rverb =~ /checkout/) || ($rverb =~ /export/)) {        # Append the rest of the checkout/export command:\r
             chdir("..");\r
+            if ($rverb =~ /export/) {\r
+                ## svn export will fail if the destination directory exists\r
+                rmdir "krb5";\r
+            }       \r
             $svncmd .= "svn+ssh://".$odr->{username}->{value}."@".$odr->{SVNURL}->{value}."/krb5/";\r
             if (length $odr->{svntag}->{value} > 0) {\r
                 $svncmd .= "tags/$odr->{svntag}->{value}";\r
@@ -446,7 +454,7 @@ sub main {
     ##-- Read in the version information & set config info.\r
 \r
 ##++ Repository action, part 2:\r
-    if ($rverb =~ /checkout/) {        \r
+    if (($rverb =~ /checkout/) || ($rverb =~ /export/)) {        \r
        if (! $bOutputCleaned) {                    ## In case somebody cleaned $out before us.\r
            if (-d $out)    {!system("rm -rf $out/*")   or die "Fatal -- Couldn't clean $out."}    ## Clean output directory.\r
            else            {mkdir($out);}\r