Your message dated Sat, 3 May 2014 13:35:02 +0200
with message-id <[email protected]>
and subject line maybe this is a feature....
has caused the Debian Bug report #746711,
regarding dependencyparser should not crash on questionable Depends
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)


-- 
746711: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=746711
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: piuparts-common
Severity: important
thanks

I was checking to see if there was a problem with 'https' in
summary.json, and ran into this.

It's coughing on the syntax of an architecture-specific dependency.
That's odd, since ${python:Depends}  has resolved to an arch-specific
reference in sid for some time.

http://lists.alioth.debian.org/pipermail/piuparts-reports/Week-of-Mon-20140428/008222.html

...
13:38:43 Copying log files
13:38:44 Removing old log files
13:38:45 Writing per-dir HTML pages
13:38:45 Writing counts.txt
Traceback (most recent call last):
  File "/srv/piuparts.debian.org/share/piuparts/piuparts-report
", line 1687, in <module>
    main()
  File "/srv/piuparts.debian.org/share/piuparts/piuparts-report", line
1660, in main
    section.generate_output(output_directory, section_names,
problem_list, web_host)
  File "/srv/piuparts.debian.org/share/piuparts/piuparts-report", line
1518, in generate_output
    self.generate_html()
  File "/srv/piuparts.debian.org/share/piuparts/piuparts-report", line
1454, in generate_html
    total_packages = self.write_counts_summary()
  File "/srv/piuparts.debian.org/share/piuparts/piuparts-report", line
962, in write_counts_summary
    count = len(self._binary_db.get_pkg_names_in_state(state))
  File 
"/srv/piuparts.debian.org/lib/python2.7/dist-packages/piupartslib/packagesdb.py",
line 593, in get_pkg_names_in_state
    self._compute_package_states()
  File 
"/srv/piuparts.debian.org/lib/python2.7/dist-packages/piupartslib/packagesdb.py",
line 554, in _compute_package_states
    db._compute_package_states()
  File 
"/srv/piuparts.debian.org/lib/python2.7/dist-packages/piupartslib/packagesdb.py",
line 562, in _compute_package_states
    state = self._compute_package_state(self._packages[package_name])
  File 
"/srv/piuparts.debian.org/lib/python2.7/dist-packages/piupartslib/packagesdb.py",
line 462, in _compute_package_state
    alt_deps = package.all_dependencies(header)
  File 
"/srv/piuparts.debian.org/lib/python2.7/dist-packages/piupartslib/packagesdb.py",
line 108, in all_dependencies
    vlist += self._parse_alternative_dependencies(header)
  File 
"/srv/piuparts.debian.org/lib/python2.7/dist-packages/piupartslib/packagesdb.py",
line 86, in _parse_alternative_dependencies
    parser = DependencyParser(self[header_name])
  File 
"/srv/piuparts.debian.org/lib/python2.7/dist-packages/piupartslib/dependencyparser.py",
line 190, in __init__
    self._list = self._parse_dependencies()
  File 
"/srv/piuparts.debian.org/lib/python2.7/dist-packages/piupartslib/dependencyparser.py",
line 219, in _parse_dependencies
    raise DependencySyntaxError("Expected comma", self._cursor)
piupartslib.dependencyparser.DependencySyntaxError: Error: pos 6:
Expected comma (text at error: ':all, libg', full text being parsed:
'python:all, libgtest-dev (>= 1.6.0)')
Fri May  2 13:38:48 UTC 2014


-- 
"Le mieux est l'ennemi du bien" - Voltaire

--- End Message ---
--- Begin Message ---
control: severity -1 normal

Hi,

the exception handling for DependencySyntaxError in 
piupartslib/dependencyparser.py treats several cases of Packages file 
brokeness as severe errors leading to a crash^wexception, and I'm now 
pondering whether this is sensible behaviour and if this bug should simply be 
closed.

After all, google-mock and #746659 were fixed within hours, as that bug broke 
more than just piuparts...

So I'm now thinking this bug should be closed indeed.


cheers,
        Holger


Attachment: signature.asc
Description: This is a digitally signed message part.


--- End Message ---

Reply via email to