From: root <devnull@localhost>
Date: Tue, 2 Jul 1996 21:40:10 +0000 (+0000)
Subject: add some strict spec file checking
X-Git-Tag: tznext/4.11.0.1.tizen20130304~11241
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=8fded05eba5ccb28ff56b7ca798a77860b092a13;p=tools%2Flibrpm-tizen.git

add some strict spec file checking

CVS patchset: 719
CVS date: 1996/07/02 21:40:10
---

diff --git a/build/vspec.c b/build/vspec.c
index 592e61c..edb0e41 100644
--- a/build/vspec.c
+++ b/build/vspec.c
@@ -5,16 +5,18 @@
 Here's what we do
 
 . check for duplicate fields
-. check for duplicate sources/patch numbers
 . make sure a certain set of fields are present
 . in subpackages, make sure a certain set of fields are present
 . in subpackages, make sure certain fields are *not* present
+
+. check for duplicate sources/patch numbers
 . do some checking on the field values for legit characters
 
 ****************/
 
 #include <stdio.h>
 #include <stdlib.h>
+#include <ctype.h>
 
 #include "header.h"
 #include "spec.h"
@@ -27,16 +29,135 @@ Here's what we do
 
 #define EMPTY(s) ((!(s)) || (!(*(s))))
 #define OOPS(s) {fprintf(stderr, "verifySpec: %s\n", s); res = 1;}
+#define MUST 1
+
+struct packageFieldsRec {
+    int tag;
+    int present;
+    int shouldBePresent;
+};
+
+struct fieldNamesRec {
+    int tag;
+    char *name;
+};
+
+static int checkHeaderTags(Header inh, struct packageFieldsRec *pfr);
+static char *tagName(int tag);
+
+/* This is a list of tags related to the "main" package. */
+/* Only list those that must or must not be present.     */
+
+static struct packageFieldsRec packageFields[] = {
+    { RPMTAG_NAME,            0, MUST },
+    { RPMTAG_VERSION,         0, MUST },
+    { RPMTAG_RELEASE,         0, MUST },
+/*    { RPMTAG_SUMMARY,         0, MUST }, */
+    { RPMTAG_DESCRIPTION,     0, MUST },
+    { RPMTAG_COPYRIGHT,       0, MUST },
+/*    { RPMTAG_PACKAGER,        0, MUST }, */
+    { RPMTAG_GROUP,           0, MUST },
+    { 0, 0, 0 },
+};
+
+/* This is a list of tags related to "sub" packages. */
+/* Only list those that must or must not be present. */
+
+static struct packageFieldsRec subpackageFields[] = {
+    { RPMTAG_NAME,            0, MUST },  /* This is inserted automaticaly */
+/*    { RPMTAG_SUMMARY,         0, MUST }, */
+    { RPMTAG_DESCRIPTION,     0, MUST },
+    { RPMTAG_GROUP,           0, MUST },
+    { RPMTAG_NAME,            0, 0 },
+    { RPMTAG_COPYRIGHT,       0, 0 },
+    { RPMTAG_PACKAGER,        0, 0 },
+    { 0, 0, 0 },
+};
+
+static struct packageFieldsRec *findTag(int tag, struct packageFieldsRec *pfr)
+{
+    struct packageFieldsRec *p = pfr;
+
+    while (p->tag) {
+	if (p->tag == tag)
+	    return p;
+	p++;
+    }
+
+    return NULL;
+}
+
+static char *tagName(int tag)
+{
+    int i = 0;
+    static char nameBuf[1024];
+    char *s;
 
-int checkHeaderDups(Header h)
+    strcpy(nameBuf, "(unknown)");
+    while (i < rpmTagTableSize) {
+	if (tag == rpmTagTable[i].val) {
+	    strcpy(nameBuf, rpmTagTable[i].name + 7);
+	    s = nameBuf+1;
+	    while (*s) {
+		*s = tolower(*s);
+		s++;
+	    }
+	}
+	i++;
+    }
+
+    return nameBuf;
+}
+
+static int checkHeaderTags(Header inh, struct packageFieldsRec *pfr)
 {
-    return 0;
+    Header h;
+    HeaderIterator headerIter;
+    int res = 0;
+    int_32 tag, lastTag, type, c;
+    void *ptr;
+    struct packageFieldsRec *p;
+
+    /* This actually sorts the index, so it'll be easy to catch dups */
+    h = copyHeader(inh);
+
+    headerIter = initIterator(h);
+    lastTag = 0;
+    while (nextIterator(headerIter, &tag, &type, &ptr, &c)) {
+	if (tag == lastTag) {
+	    error(RPMERR_BADSPEC, "Duplicate fields for     : %s",
+		  tagName(tag));
+	    res = 1;
+	}
+	p = findTag(tag, pfr);
+	if (p) 
+	    p->present = 1;
+	lastTag = tag;
+    }
+    freeIterator(headerIter);
+    freeHeader(h);
+
+    p = pfr;
+    while (p->tag) {
+	if (p->shouldBePresent != p->present) {
+	    error(RPMERR_BADSPEC, "Field must%s be present%s: %s",
+		  p->present ? " NOT" : "", p->present ? "" : "    ",
+		  tagName(p->tag));
+	    res = 1;
+	}
+	p->present = 0;  /* hack to clear it for next time :-) */
+	p++;
+    }
+    
+    return res;
 }
 
 int verifySpec(Spec s)
 {
-    int res;
+    int res = 0;
     struct PackageRec *pr;
+    struct packageFieldsRec *fields;
+    char name[1024];
     
     if (EMPTY(s->name)) {
 	OOPS("No Name field");
@@ -44,12 +165,25 @@ int verifySpec(Spec s)
 
     /* Check each package/subpackage */
     pr = s->packages;
+    fields = packageFields;
     while (pr) {
-	if (checkHeaderDups(pr->header)) {
+	if (! pr->subname) {
+	    if (pr->newname) {
+		strcpy(name, pr->newname);
+	    } else {
+		strcpy(name, s->name);
+	    }
+	} else {
+	    sprintf(name, "%s-%s", s->name, pr->subname);
+	}
+	printf("* Package: %s\n", name);
+	
+	if (checkHeaderTags(pr->header, fields)) {
 	    res = 1;
 	}
 	pr = pr->next;
+	fields = subpackageFields;
     }
     
-    return 0;
+    return res;
 }