make -i not globaly set $error to 0

bin/fcopy

@@ -102,7 +102,8 @@ sub copy_one {
 
   # if no class matches, remove destination or move it to the backup file
   unless (defined $class) {
-    ewarn("no matching file for any class for $source defined.");
+    warn "fcopy: no matching file for any class for $source defined.\n";
+    $error = 1 unless $opt_i; # do not set $error if -i was set
     # do not copy
     if ($opt_d and -f $destfile) {
       print LOGFILE "$source\tNONE\t# removed (no matching class)\n" if $logfile;
@@ -554,5 +555,4 @@ if ($opt_r) {
 }
 
 foreach (@ARGV) { copy_one($source,$_,$target); }
-$opt_i && exit 0; # ignore any warning
 exit $error;

man/fcopy.8

@@ -67,7 +67,8 @@ Show help, version and summary of options.
 If source is a symbolic link, make destination a symbolic link too.
 .TP
 .B \-i
-Ignore warnings about no matching class and always exit with 0.
+Ignore warnings about no matching class. These warnings will not set
+the exit code to 1.
 .TP
 .BI "\-I " dir[,dir]
 Override list of ignored directories when recursing. If not given, it is