summary |
shortlog |
log |
commit | commitdiff |
tree
raw |
patch |
inline | side by side (from parent 1:
f84463d)
as we also parse dsc files here, only do the rejection on missing keywords
when we parse a changes file.
adjust all calls for dsc files to tell us they are dsc, by default we do
the check, caller has to turn it off...
Signed-off-by: Joerg Jaspert <joerg@debian.org>
for line in list_file.readlines():
f = line[:-1]
try:
for line in list_file.readlines():
f = line[:-1]
try:
- utils.parse_changes(f, signing_rules=1)
+ utils.parse_changes(f, signing_rules=1, dsc_file=1)
except InvalidDscError, line:
utils.warn("syntax error in .dsc file '%s', line %s." % (f, line))
count += 1
except InvalidDscError, line:
utils.warn("syntax error in .dsc file '%s', line %s." % (f, line))
count += 1
try:
# NB: don't enforce .dsc syntax
try:
# NB: don't enforce .dsc syntax
- dsc = utils.parse_changes(filename)
+ dsc = utils.parse_changes(filename, dsc_file=1)
except:
utils.fubar("error parsing .dsc file '%s'." % (filename))
except:
utils.fubar("error parsing .dsc file '%s'." % (filename))
if not name.endswith(".dsc"):
continue
filename = os.path.abspath(dirname+'/'+name)
if not name.endswith(".dsc"):
continue
filename = os.path.abspath(dirname+'/'+name)
- dsc = utils.parse_changes(filename)
+ dsc = utils.parse_changes(filename, dsc_file=1)
for field_name in [ "build-depends", "build-depends-indep" ]:
field = dsc.get(field_name)
if field:
for field_name in [ "build-depends", "build-depends-indep" ]:
field = dsc.get(field_name)
if field:
for f in files.keys():
if f.endswith(".dsc"):
try:
for f in files.keys():
if f.endswith(".dsc"):
try:
- dsc = utils.parse_changes(f)
+ dsc = utils.parse_changes(f, dsc_file=1)
dsc_files = utils.build_file_list(dsc, is_a_dsc=1)
except:
utils.warn("error processing '%s'; skipping it. [Got %s]" % (f, sys.exc_type))
dsc_files = utils.build_file_list(dsc, is_a_dsc=1)
except:
utils.warn("error processing '%s'; skipping it. [Got %s]" % (f, sys.exc_type))
dsc_file = utils.open_file(filename)
try:
dsc_file = utils.open_file(filename)
try:
- dsc = utils.parse_changes(filename)
+ dsc = utils.parse_changes(filename, dsc_file=1)
except:
return formatted_text("can't parse .dsc control info")
dsc_file.close()
except:
return formatted_text("can't parse .dsc control info")
dsc_file.close()
for i in source_packages.keys():
filename = "/".join(source_packages[i])
try:
for i in source_packages.keys():
filename = "/".join(source_packages[i])
try:
- dsc = utils.parse_changes(filename)
+ dsc = utils.parse_changes(filename, dsc_file=1)
except CantOpenError:
utils.warn("couldn't open '%s'." % (filename))
continue
except CantOpenError:
utils.warn("couldn't open '%s'." % (filename))
continue
# Parse the .dsc file
try:
# Parse the .dsc file
try:
- self.pkg.dsc.update(utils.parse_changes(dsc_filename, signing_rules=1))
+ self.pkg.dsc.update(utils.parse_changes(dsc_filename, signing_rules=1, dsc_file=1))
except CantOpenError:
# if not -n copy_to_holding() will have done this for us...
if not action:
except CantOpenError:
# if not -n copy_to_holding() will have done this for us...
if not action:
################################################################################
################################################################################
-def parse_changes(filename, signing_rules=0):
+def parse_changes(filename, signing_rules=0, dsc_file=0):
"""
Parses a changes file and returns a dictionary where each field is a
key. The mandatory first argument is the filename of the .changes
"""
Parses a changes file and returns a dictionary where each field is a
key. The mandatory first argument is the filename of the .changes
raise ChangesUnicodeError, "Changes file not proper utf-8"
changes = parse_deb822(content, signing_rules)
raise ChangesUnicodeError, "Changes file not proper utf-8"
changes = parse_deb822(content, signing_rules)
- # Finally ensure that everything needed is there
- must_keywords = ('Format', 'Date', 'Source', 'Binary', 'Architecture', 'Version',
- 'Distribution', 'Maintainer', 'Description', 'Changes', 'Files')
- missingfields=[]
- for keyword in must_keywords:
- if not changes.has_key(keyword.lower()):
- missingfields.append(keyword)
+ if not dsc_file:
+ # Finally ensure that everything needed for .changes is there
+ must_keywords = ('Format', 'Date', 'Source', 'Binary', 'Architecture', 'Version',
+ 'Distribution', 'Maintainer', 'Description', 'Changes', 'Files')
- if len(missingfields):
- raise ParseChangesError, "Missing mandantory field(s) in changes file (policy 5.5): %s" % (missingfields)
+ missingfields=[]
+ for keyword in must_keywords:
+ if not changes.has_key(keyword.lower()):
+ missingfields.append(keyword)
+
+ if len(missingfields):
+ raise ParseChangesError, "Missing mandantory field(s) in changes file (policy 5.5): %s" % (missingfields)
# Parse the file if needed
if dsc is None:
# Parse the file if needed
if dsc is None:
- dsc = parse_changes(dsc_filename, signing_rules=1);
+ dsc = parse_changes(dsc_filename, signing_rules=1, dsc_file=1);
if dsc_files is None:
dsc_files = build_file_list(dsc, is_a_dsc=1)
if dsc_files is None:
dsc_files = build_file_list(dsc, is_a_dsc=1)