On 2014/11/20 21:28, Donovan Watteau wrote:
> Here's a new version, with an explicit requirement on boost's latest
> revision, as suggested by Landry.
> 
> As for Doxygen: I don't see the point of requiring doxygen and
> graphviz just for documenting API internals; it's just a tool, not
> a library.

I agree - you have disabled doxygen in cmake so there's no problem with
it picking up an installed copy at configure time, API docs aren't
useful for this port, so I think this is fine.

Seems useful and the port looks good. Any OKs to import it? (or if
someone else would like to, it's OK sthen@).

I don't like upstream's hardcoded escape sequences for colour support
and overwriting text for the progress bar (it ought to use terminal
capabilities to work out what sequences to send) but at least it
provides a way to disable them on the command line.


Reply via email to