Re: setup.exe and release messages

[Dave Korn <dave dot korn dot cygwin at googlemail dot com>]

Christopher Faylor wrote:

> I've been talking about changing the format of setup.hint files to use
> a real markup language.  Maybe this would be the time to do that.

  Seems a bit overkill for this particular job to me.  I was thinking of
something more along these lines (for the pre-chooser overall release message,
rather than per-package ones):

Index: inilex.ll
===================================================================
RCS file: /cvs/cygwin-apps/setup/inilex.ll,v
retrieving revision 1.1
diff -p -u -r1.1 inilex.ll
--- inilex.ll   30 May 2007 05:12:42 -0000      1.1
+++ inilex.ll   22 Jul 2009 17:39:03 -0000
@@ -69,6 +69,7 @@ STR   [!a-zA-Z0-9_./:\+-]+

 "setup-timestamp:"     return SETUP_TIMESTAMP;
 "setup-version:"       return SETUP_VERSION;
+"motd:"                return MOTD;
 "Package:"             return PACKAGENAME;
 [vV]"ersion:"          return PACKAGEVERSION;
 "install:"|"Filename:" return INSTALL;
Index: iniparse.yy
===================================================================
RCS file: /cvs/cygwin-apps/setup/iniparse.yy,v
retrieving revision 1.1
diff -p -u -r1.1 iniparse.yy
--- iniparse.yy 30 May 2007 05:12:42 -0000      1.1
+++ iniparse.yy 22 Jul 2009 17:39:03 -0000
@@ -38,7 +38,7 @@ void add_correct_version();
 %}

 %token STRING
-%token SETUP_TIMESTAMP SETUP_VERSION PACKAGEVERSION INSTALL SOURCE SDESC LDESC
+%token SETUP_TIMESTAMP SETUP_VERSION MOTD PACKAGEVERSION INSTALL SOURCE SDESC
LDESC
 %token CATEGORY DEPENDS REQUIRES
 %token APATH PPATH INCLUDE_SETUP EXCLUDE_PACKAGE DOWNLOAD_URL
 %token T_PREV T_CURR T_TEST
@@ -64,6 +64,7 @@ setup_headers: /* empty */
 header /* non-empty */
  : SETUP_TIMESTAMP STRING { iniBuilder->buildTimestamp ($2); } NL
  | SETUP_VERSION STRING { iniBuilder->buildVersion ($2); } NL
+ | MOTD STRING STRING { iniBuilder->motd_title = $2; iniBuilder->motd = $3; } NL
  ;

 packages: /* empty */


    cheers,
      DaveK