On Tue, Jan 09, 2018 at 10:42:19PM +0530, Vasudev Kamath wrote:
> Control: tag -1 +moreinfo +unreproducible
> 
> Adam Borowski <kilob...@angband.pl> writes:
> 
> > Source: ctpp2
> > Version: 2.8.3-23
> > Severity: serious
> > Justification: fails to build from source (but built successfully in the 
> > past)
> >
> > Hi!
> > I'm afraid your package fails to build:
> >
> > dh_fixperms -pctpp2-doc 
> > chown: cannot access 
> > 'debian/ctpp2-doc/usr/share/doc/ctpp2-doc/html/a01323_acf0137f96d20b71a2e2e18b489d1a33e_cgraph.dot':
> >  No such file or directory
> > chown: cannot access 
> > 'debian/ctpp2-doc/usr/share/doc/ctpp2-doc/html/inherit_graph_65.md5': No 
> > such file or directory
> > dh_fixperms: find debian/ctpp2-doc -true -print0 2>/dev/null | xargs -0r 
> > chown --no-dereference 0:0 returned exit code 123
> > dh_fixperms: Aborting due to earlier error
> >
> > Tried on armhf amd64.  Log attached.
> 
> Could not reproduce on amd64, it built fine. Was not able to check on
> armhf yet. Attaching my build log for reference.

[...]
> Yeah. Since you mentioned its random can we reduce the severity of the
> bug?.

Bad idea, as it fails a lot: on my armhf box, 17/20 times, on my amd64, 6/12
times.

Turn out, the problem is not that sometimes it fails.  The wrongness it that
it doesn't fail when it should.

Did you notice these syntax errors from dot that are found in your log as
well?  They didn't happen on official buildds five months ago.  I dismissed
them as they _usually_ are ignored, but it turns out that sometimes, with a
low probability, they do cause FTBFS.  There's quite a bunch of those syntax
errors, though, so even though every one is unlikely to cause a failure, the
overall chance is high.


So, the real bug is dot ignoring these errors.  Not only this causes problem
like intermittent failures here, it's also against the Policy (which
requires errors to be detected as a "must" clause), and common sense -- as
the docs get corrupted in random ways.


As for the errors themselves, an example:
  Node12 
[label="CTPP::CTPP2JSONParser\l::IsString",height=0.2,width=0.4,color="black", 
fillcolor="white", 
style="filled",URL="$a01311.xhtml#a9a4c251b78ce957f7e6cd9bc12ecbb6a",tooltip="JSON
 string \"blah-blah \\" clah-clah \" | &#39;blah-blah \&#39; clah-clah &#39;. 
"];
note 「\\"」 -- ie, an unescaped quotation mark.


I don't know graphviz enough to point the blame either at it or the way
ctpp2's build system calls it, I suspect the former, though.  Which would
mean reassigning to graphviz.  Not sure what's the cause and severity of the
unescaped quote; ignoring errors is definitely RC, though.


Meow!
-- 
⢀⣴⠾⠻⢶⣦⠀ 
⣾⠁⢰⠒⠀⣿⡁ Imagine there are bandits in your house, your kid is bleeding out,
⢿⡄⠘⠷⠚⠋⠀ the house is on fire, and seven big-ass trumpets are playing in the
⠈⠳⣄⠀⠀⠀⠀ sky.  Your cat demands food.  The priority should be obvious...

Reply via email to