[Setup-tool-hackers] [PATCH] for hardware-conf and platform.pl

[Richard Bos <allabos freeler nl>]

please find attached 2 patches; 1 for hardware-conf and the other for 
platform.pl.  The one for hardware-conf  is to get it running at all, at this 
moment it misses 2 requires, and the description field is not recognized 
correctly.

The one for platform.pl is needed to get the correct error message, when a 
platform is not supported by xst, but has been recognized.

Currently the error message is: Unable to determine host platform
while it should be: Your platform [%s] is not supported.::unknown
This is due to un incorrect located "}" :)

Can someone with cvs access please commit.

Question1: the error messages are now spit out in plain text, shouldn't these 
be outputted in xml, so the frontend can easily parse them and display them 
in the gui/frontend?

Question2: Is it already possibly to configure a remote machine via the gui 
and backends.  I imagine that the backends have been installed on the remote 
machine and the frontends are run at the local machine.  Whether it is yes or 
no, how (will be) is this implemented (using ssh e.g., and how is the 
frontend told to configure the remote system and the local one)?

-- 
Richard Bos
For those who have no (/)home the journey is endless
--- hardware-conf.in.old	Thu Jun  7 09:53:46 2001
+++ hardware-conf.in	Thu Jun  7 09:55:54 2001
@@ -51,7 +51,8 @@
 
 
 
-require "___scriptsdir___/be.pl";
+require "___scriptsdir___/general.pl";
+require "___scriptsdir___/xml.pl";
 
 
 
@@ -60,7 +61,7 @@ require "___scriptsdir___/be.pl";
 $name = "hardware";
 $version = "0.1.0";
 
-$description =<<"end_of_description";
+$description =<<"end_of_description;";
        Configures hardware.
 end_of_description;
 
--- platform.pl	Thu Jun  7 09:49:49 2001
+++ platform.new	Thu Jun  7 09:51:42 2001
@@ -231,9 +231,9 @@ sub xst_platform_guess
       $xst_dist = $dist;
       return;
     }
+  }
 
     $xst_dist = "unknown";
-  }
 }