Wireshark · Wireshark-commits: [Wireshark-commits] master a30dcd9: cmake: fix missing docbook dependency

[Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>]

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=a30dcd9951ed5814ca1974a31db83756bfb5b233
Submitter: Peter Wu (peter@xxxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

a30dcd9 by Peter Wu (peter@xxxxxxxxxxxxx):

    cmake: fix missing docbook dependency
    
    Running "ninja developer_guide_html" somehow did not produce a new HTML
    docbook once wsluarm.ascii (a dependency of developer-guide.xml) was
    modified.
    
    It turns out that output file docbook/wsdg_html/index.html only had a
    ordering-only dependency on target generate_developer-guide.xml. An
    extra dependency is needed on the output file to ensure that the HTML
    file gets rebuild on changes.
    
    See the last note on
    https://samthursfield.wordpress.com/2015/11/21/cmake-dependencies-between-targets-and-files-and-custom-commands/
    
    Fixes v2.1.0rc0-2137-gd544ecd ("cmake: fix parallel docbook build").
    
    Change-Id: I7689c71994f13b29cf7f8561f7c993aa8298632d
    Reviewed-on: https://code.wireshark.org/review/18225
    Reviewed-by: Graham Bloice <graham.bloice@xxxxxxxxxxxxx>
    Reviewed-by: Peter Wu <peter@xxxxxxxxxxxxx>
    

Actions performed:

    from  632b7d1   Fix typo in a message
    adds  a30dcd9   cmake: fix missing docbook dependency


Summary of changes:
 cmake/modules/FindXSLTPROC.cmake |   11 +++++++----
 docbook/CMakeLists.txt           |   13 +++++++++++++
 2 files changed, 20 insertions(+), 4 deletions(-)