[jira] [Comment Edited] (MDEP-551) recursive dep:tree format dot just appends

["Hervé Boutemy (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/MDEP-551?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15970964#comment-15970964 ] 

Hervé Boutemy edited comment on MDEP-551 at 4/17/17 2:36 PM:
-------------------------------------------------------------

The dot graph I made for that monorepo experiment was via https://github.com/ferstl/depgraph-maven-plugin which can make an integral (only one 'digraph \{ ... }' section per .dot file).

If I was to reduce my complaint to the smallest it could be, I'd say that the MDEP plugin should not allow -DappendOutput=true -DoutputType=dot to be used together for a multi-pom situation. It'll complete, but make a .dot file that can't be loaded.



was (Author: paul):
The dot graph I made for that monorepo experiment was via https://github.com/ferstl/depgraph-maven-plugin which can make an integral (only one 'digraph { ... }' section per .dot file).

If I was to reduce my complaint to the smallest it could be, I'd say that the MDEP plugin should not allow -DappendOutput=true -DoutputType=dot to be used together for a multi-pom situation. It'll complete, but make a .dot file that can't be loaded.


> recursive dep:tree format dot just appends
> ------------------------------------------
>
>                 Key: MDEP-551
>                 URL: https://issues.apache.org/jira/browse/MDEP-551
>             Project: Maven Dependency Plugin
>          Issue Type: Bug
>          Components: tree
>            Reporter: Paul Hammant
>             Fix For: waiting-for-feedback
>
>
> For a multi-module build..
> {noformat}mvn dependency:tree -DappendOutput=true -DoutputType=dot -DoutputFile=/path/to/file.dot{noformat}
> ... just appends multiple dot files together rather than composes a cohesive large dot (graphviz) file. 
> No graphviz capable tool can load the output :(



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)