--- loncom/build/lpml_parse.pl	2012/01/27 23:50:57	1.60
+++ loncom/build/lpml_parse.pl	2020/10/07 19:55:39	1.62
@@ -12,7 +12,7 @@
 # The LearningOnline Network with CAPA
 # lpml_parse.pl - Linux Packaging Markup Language parser
 #
-# $Id: lpml_parse.pl,v 1.60 2012/01/27 23:50:57 raeburn Exp $
+# $Id: lpml_parse.pl,v 1.62 2020/10/07 19:55:39 raeburn Exp $
 #
 # Written by Scott Harrison, codeharrison@yahoo.com
 #
@@ -1230,6 +1230,7 @@ sub format_file {
     my @tokeninfo=@_;
     $file=''; $source=''; $target=''; $categoryname=''; $description='';
     $note=''; $build=''; $status=''; $dependencies=''; $installscript='';
+    $buildlink = '';
     my $text=&trim($parser->get_text('/file'));
     my $buildtest;
     $file_count++;
@@ -1323,14 +1324,24 @@ END
 		}
 		return $result;
 	    }
-            my $bflag='-b1';
-            $bflag='-b3' if (($dependencies or $buildlink) or 
+	    my $testtarget = $target;
+	    if ($categoryname eq 'setuid script') {
+		my ($path,$filename) = ($target =~ /^(.*\/)([^\/]+)$/);
+		my $alttarget = $path.'.'.$filename;
+		if ((-e "$targetroot/$target") && (-B "$targetroot/$target") &&
+		    (-e "$targetroot/$alttarget") && (-T "$targetroot/$alttarget")) {
+		    $testtarget = $alttarget;
+		}
+	    }
+            my $bflag='-b5';
+            $bflag='-b3' if ($buildlink);
+            $bflag='-b6' if (($dependencies) or
                              ($categoryname eq 'pdf manual'));
 	    if ($tokeninfo[2]{type} eq 'private') {
 		return <<END;
 $buildtest	\@if (test -e "${sourceroot}/${source}") && (test -e "${targetroot}/${target}"); then \\
 		ECODE=0; \\
-		perl filecompare.pl $bflag ${sourceroot}/${source} ${targetroot}/${target} || ECODE=\$\$?; \\
+		perl filecompare.pl $bflag ${sourceroot}/${source} ${targetroot}/${testtarget} || ECODE=\$\$?; \\
 		case "\$\$ECODE" in \\
 			1) echo "${targetroot}/${target} is unchanged";; \\
 			2) echo "**** WARNING **** target file ${targetroot}/${target} is newer than CVS source; saving current (old) target file to ${targetroot}/${target}.lpmlsave and then overwriting"$logcmd && install -o www -g www -m 0600 ${targetroot}/${target} ${targetroot}/${target}.lpmlsave && install -p $categoryhash{$categoryname} ${sourceroot}/${source} ${targetroot}/${target};; \\
@@ -1354,7 +1365,7 @@ $buildtest	\@if !(test -e "${sourceroot}
 		echo "install -p $categoryhash{$categoryname} ${sourceroot}/${source} ${targetroot}/${target}" && install -p $categoryhash{$categoryname} ${sourceroot}/${source} ${targetroot}/${target}; \\
 	else \\
 		ECODE=0; \\
-		perl filecompare.pl $bflag ${sourceroot}/${source} ${targetroot}/${target} || ECODE=\$\$?; \\
+		perl filecompare.pl $bflag ${sourceroot}/${source} ${targetroot}/${testtarget} || ECODE=\$\$?; \\
 		case "\$\$ECODE" in \\
 			1) echo "${targetroot}/${target} is unchanged";; \\
 			2) echo "**** WARNING **** target file ${targetroot}/${target} is newer than CVS source; saving current (old) target file to ${targetroot}/${target}.lpmlsave and then overwriting"$logcmd && install -o www -g www -m 0600 ${targetroot}/${target} ${targetroot}/${target}.lpmlsave && install -p $categoryhash{$categoryname} ${sourceroot}/${source} ${targetroot}/${target};; \\
@@ -1662,11 +1673,12 @@ sub format_installscript {
 # -------------------------------------------------------- Format build section
 sub format_buildlink {
     my @tokeninfo=@_;
-    $buildlink='';
     my $text=&trim($parser->get_text('/buildlink'));
     if ($text) {
 	$parser->get_tag('/buildlink');
 	$buildlink=$sourceroot.'/'.$text;
+    } else {
+        $buildlink='';
     }
     return '';
 }
@@ -1684,12 +1696,13 @@ sub format_status {
 # ------------------------------------------------- Format dependencies section
 sub format_dependencies {
     my @tokeninfo=@_;
-    #$dependencies='';
     my $text=&trim($parser->get_text('/dependencies'));
     if ($text) {
 	$parser->get_tag('/dependencies');
 	$dependencies=join(';',((map {s/^\s*//;s/\s$//;$_} split(/\;/,$text)),$dependencies));
 	$dependencies=~s/;$//;
+    } else {
+        $dependencies='';
     }
     return '';
 }