Github user mike-jumper commented on a diff in the pull request:

    
https://github.com/apache/incubator-guacamole-client/pull/122#discussion_r110818228
  
    --- Diff: extensions/guacamole-auth-radius/src/licenses/bundled/README ---
    @@ -0,0 +1,4 @@
    +Apache Guacamole includes a number of subcomponents with separate copyright
    +notices and license terms. Your use of these subcomponents is subject to 
the
    --- End diff --
    
    > ... Your use of these subcomponents is subject to the terms and 
conditions of their respective licenses
    
    This may actually not be correct here because of the LGPL dependency. My 
understanding is that the LGPL is "Category X" primarily because it imposes 
restrictions on larger works, thus this extension would be under additional 
restrictions imposed by the LGPL once built in binary form (hence why we can't 
distribute those binaries as a project).
    
    Perhaps this is worth opening a thread in the Incubator general list for 
guidance - someone likely has dealt with this before, but I think I'm 
officially out of my element. My best guess would be:
    
    1. We can't distribute binaries of this (as we already expect)
    2. We shouldn't include the `src/licenses/...` stuff which we normally 
include, since that's not technically correct in this case
    3. Rather than building a `.tar.gz` which contains all the licenses of all 
bundled dependencies alongside the `.jar`, unless other things will be in that 
`.tar.gz`, we may just want to restrict the build to the `.jar`.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to