Github user matthiasblaesing commented on the issue:

    https://github.com/apache/incubator-netbeans/pull/130
  
    Looks good. The cache is a good idea - having one cache keeps the code 
simple and does not risk corrupting a maven repository, so I would agree with 
that.
    
    If you are interested, I fixed the unittests for the module (one was 
introduced when the extended maven coordinate syntax was added and the others 
are artifacts from license header changes and missing files):
    
    
https://github.com/matthiasblaesing/incubator-netbeans/commits/unittests_nbbuild
    
    Feel free to pull the commits into this branch, else I'll rebase the 
changes and will propose these in a separate PR.


---

Reply via email to