Github user huor commented on the issue:

    https://github.com/apache/incubator-hawq/pull/1246
  
    @rvs, thanks for the review and feedback.
    
    You are right. We expand the LICENSE for source release by adding extra 
dependency to get that for binary release. Here is the changes:
    ```
    rhuo-mbp:incubator-hawq rhuo$ diff dist/hawq/LICENSE LICENSE
    265,270d264
    <      (BSD 2 Clause) Google Mock 
(https://github.com/google/googletest/tree/master/googlemock)
    <        depends/thirdparty/googletest/googlemock
    <
    <      (BSD 2 Clause) Google Test (https://github.com/google/googletest)
    <        depends/thirdparty/googletest
    <
    347,382d340
    <      contrib/extprotocol
    <      contrib/formatter_fixedwidth
    <      contrib/gp_cancel_query
    <      contrib/hawq-hadoop
    <      contrib/orafce
    <      contrib/pgcrypto
    <      pxf/pxf
    <      src/backend/libpq
    <      src/backend/port/beos/shm.c
    <      src/backend/port/qnx4/shm.c
    <      src/backend/utils/mb/conversion_procs/ascii_and_mic
    <      src/backend/utils/mb/conversion_procs/cyrillic_and_mic
    <      src/backend/utils/mb/conversion_procs/euc_cn_and_mic
    <      src/backend/utils/mb/conversion_procs/euc_jis_2004_and_shift_jis_2004
    <      src/backend/utils/mb/conversion_procs/euc_jp_and_sjis
    <      src/backend/utils/mb/conversion_procs/euc_kr_and_mic
    <      src/backend/utils/mb/conversion_procs/euc_tw_and_big5
    <      src/backend/utils/mb/conversion_procs/latin2_and_win1250
    <      src/backend/utils/mb/conversion_procs/latin_and_mic
    <      src/backend/utils/mb/conversion_procs/utf8_and_ascii
    <      src/backend/utils/mb/conversion_procs/utf8_and_big5
    <      src/backend/utils/mb/conversion_procs/utf8_and_cyrillic
    <      src/backend/utils/mb/conversion_procs/utf8_and_euc_cn
    <      src/backend/utils/mb/conversion_procs/utf8_and_euc_jis_2004
    <      src/backend/utils/mb/conversion_procs/utf8_and_euc_jp
    <      src/backend/utils/mb/conversion_procs/utf8_and_euc_kr
    <      src/backend/utils/mb/conversion_procs/utf8_and_euc_tw
    <      src/backend/utils/mb/conversion_procs/utf8_and_gb18030
    <      src/backend/utils/mb/conversion_procs/utf8_and_gbk
    <      src/backend/utils/mb/conversion_procs/utf8_and_iso8859
    <      src/backend/utils/mb/conversion_procs/utf8_and_iso8859_1
    <      src/backend/utils/mb/conversion_procs/utf8_and_johab
    <      src/backend/utils/mb/conversion_procs/utf8_and_shift_jis_2004
    <      src/backend/utils/mb/conversion_procs/utf8_and_sjis
    <      src/backend/utils/mb/conversion_procs/utf8_and_uhc
    <      src/backend/utils/mb/conversion_procs/utf8_and_win
    384,391d341
    <      src/bin/gpfilesystem/hdfs/gpfshdfs.c
    <      src/bin/gpoptutils
    <      src/bin/gpupgrade
    <      src/bin/pg_dump
    <      src/include/libpq
    <      src/interfaces/ecpg
    <      src/interfaces/ecpg/pgtypeslib
    <      src/interfaces/libpq
    393,398c343,345
    <      src/pl/pljava
    <      src/pl/plperl
    <      src/pl/plpgsql
    <      src/pl/plpython
    <      src/port
    <      tools/bin/pythonSrc/PSI-0.3b2_gp/src
    ---
    >      src/bin/pg_dump
    >      src/backend/port/qnx4/shm.c
    >      src/backend/port/beos/shm.c
    ```
    
    For the NOTICE, the dependencies are either ASL v2 or category B license 
with no LICENSE with themselves, so there is not change according to 
https://cwiki.apache.org/confluence/display/GEODE/License+Guide+for+Contributors.
    
    Please let us know your comments for further refinement. Thanks.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to