[LON-CAPA-cvs] cvs: loncom /build piml_parse.pl

harris41 lon-capa-cvs@mail.lon-capa.org
Tue, 03 Dec 2002 21:37:08 -0000


harris41		Tue Dec  3 16:37:08 2002 EDT

  Modified files:              
    /loncom/build	piml_parse.pl 
  Log:
  Rollback to version 1.7....
  
  This is being tracked now as BUG 1035.
  
  > Documentation changes - it's always nice to see how things really work
  > instead of how they claim to work.
  1) Better stated, it is always amazing to see anything in LON-CAPA work.
  2) It is always nice to actually find readable documentation inside the
     LON-CAPA source code.
  3) Sometimes goals #1 and #2 actually help pinpoint problems, such as now.
  
  > Fixed bug which caused abortion when all four possible arguements were
  > provided.
  4) This code is now being developed at http://lpml.sourceforge.net/.
  5) This bug is still not fixed.
  
  I am rolling back to version 1.7 since the real fix is just a matter
  of removing the processing of the mode variable (this was leftover
  code from lpml_parse.pl).
  
  
  
Index: loncom/build/piml_parse.pl
diff -u loncom/build/piml_parse.pl:1.8 loncom/build/piml_parse.pl:1.9
--- loncom/build/piml_parse.pl:1.8	Tue Nov 26 10:25:21 2002
+++ loncom/build/piml_parse.pl	Tue Dec  3 16:37:08 2002
@@ -8,7 +8,7 @@
 # The LearningOnline Network with CAPA
 # piml_parse.pl - Linux Packaging Markup Language parser
 #
-# $Id: piml_parse.pl,v 1.8 2002/11/26 15:25:21 matthew Exp $
+# $Id: piml_parse.pl,v 1.9 2002/12/03 21:37:08 harris41 Exp $
 #
 # Written by Scott Harrison, codeharrison@yahoo.com
 #
@@ -66,12 +66,11 @@
 my $usage=(<<END);
 **** ERROR ERROR ERROR ERROR ****
 Usage is for piml file to come in through standard input.
-1st is install, configinstall, or build
-2st argument is the category permissions to use (runtime or development)
-3nd argument is the distribution (default,redhat6.2,debian2.2,redhat7.1,etc).
-4rd argument is to manually specify a targetroot
+1st argument is the category permissions to use (runtime or development)
+2nd argument is the distribution (default,redhat6.2,debian2.2,redhat7.1,etc).
+3rd argument is to manually specify a targetroot
 
-The first 3 arguments are required for the program to run.
+Only the 1st argument is mandatory for the program to run.
 
 Example:
 
@@ -82,7 +81,7 @@
 # ------------------------------------------------- Grab command line arguments
 
 my $mode;
-if (@ARGV>=3) {
+if (@ARGV==3) {
     $mode = shift @ARGV;
 }
 else {