Repository navigation
html v2 #7
Description
This is a big-ticket to get the revamped html into a usable-by-default state.
This is blocked by four tasks:
- html v2: remove lxml dependency #8 remove lxml dependency
- html v2: generate by default #9 generate by default
- html v2: inline html dependencies #10 inline html depdendencies
- html v2: optimize size a little #11 optimize size a little
@davidmalcolm Could you review this plan and let me know if it needs to be altered and/or what order you'd like to see these four patches? I'll start work after I hear the go-ahead.
Activity
davidmalcolm commentedon May 19, 2014
Looks reasonable; thanks.
FWIW at some point I want to merge the "firehose" branch (on the fedorahosted site, not mirrored here), which switches the internal datamodel for analysis results to use the data model given at:
https://github.com/fedora-static-analysis/firehose
for a clean model vs view separation, and ultimately I want to move the visualization code there, so it can be shared by other analysis tools.
But don't let that be a blocker to working on this ticket; I can merge it all at a later date.
bukzor commentedon May 19, 2014
Is the firehose the same or different than the json consumed by htmlv2?
davidmalcolm commentedon May 19, 2014
It's different. It comes in both XML and JSON serializations, and there's a module of Python objects for working with them. The Firehose branch of cpychecker works by building Firehose Python objects in memory, and I have a port of the old HTML rendering code that works directly on Firehose objects. However, there were enough differecnes in testcase results that I never got around to merging that branch. (I suspect the differences were mostly minor, but it's a substantial job to doublecheck them all. Less work now that the test suite runs in parallel, I guess)
davidmalcolm commentedon May 19, 2014
Also, it would be a new dependency, on firehose. It's in Fedora, and it may be in Debian.
davidmalcolm commentedon May 20, 2014
Note to self: Firehose is in Debian:
http://packages.qa.debian.org/p/python-firehose.html
bukzor commentedon May 20, 2014
all done. yay!
.travis.yml: experiment with Python versions #7