The video is very helpful, thanks for the work.
On 2024/04/29 04:57:13 Shawn Yang wrote:
> Hi everyone,
>
> I recorded a video about how to use Scanoss to audit the code.
>
> Hope it can be useful for other people and podlings to inspect their code,
> and make the release more efficient:
> Scre
Hello Incubator Community:
I'm cancelling this vote:
https://lists.apache.org/thread/6z7ryxbd59tc0f7fjpblpfjy2nwnxwp4
Because of License issues. I'll fix them and start the new round vote
process.
Thanks a lot for all your help.
Best,
Shaokang Lv
Thanks for your review, this vote is closed.
Best,
Shaokang Lv
Huajie Wang 于2024年4月29日周一 18:17写道:
> Thanks for your review work with profound respect. we have always
> attached great importance to License compliance issues. We will close
> the current vote later and initiate a new round of voti
If the `incubator-` prefix significantly burdens the graduation process or
introduces inconveniences in package or dependency management, its removal
might be considered.
However, IMO, maintaining incubating in the description, and website,
especially in the *release version*, clearly signals that
Thanks for your review work with profound respect. we have always
attached great importance to License compliance issues. We will close
the current vote later and initiate a new round of voting.
Best,
Huajie Wang
Justin Mclean 于2024年4月29日周一 17:52写道:
> Hi,
>
> > Thanks for your clarification.
Hi,
> Thanks for your clarification. We have submitted a PR[1] to address this
> issue. I believe this issue will be resolved quickly.
>
> [1] https://github.com/apache/incubator-streampark/pull/3689
That looks good to me, thanks.
Kind Regards,
Justin
--
Before starting the vote, I found (perhaps) a final question: Shall we thus
guide all the new podlings to enter the incubator without incubator- prefix
and thus converge all the current podlings are in form apache/foo?
I'm afraid that if new podling can still have the incubator- prefix, it can
giv
Thanks for your clarification. We have submitted a PR[1] to address this
issue. I believe this issue will be resolved quickly.
[1] https://github.com/apache/incubator-streampark/pull/3689
Best,
Huajie Wang
tison 于2024年4月29日周一 16:02写道:
> > So a condition of including MIT licensed code is to
> But I found a potential issue that it's possible StreamPark copied some
code from flink-sql-gateway to streampark-flink-sql-gateway.
Thanks for your review work, the "streampark-flink-sql-gateway" module is
in the dev branch and is currently in the development phase, As you said,
some of the cod
> Even if they have changed a lot, that could still be an issue. Copying an
earlier version of a file is still an issue that needs to be dealt with.
Even copying 5% of something needs to be treated correctly. The files seem
to have permissive licenses, so there is no category X licensing issue, at
Hi,
> * Some false positive on testing assertions utils. Match rate < 50%, file
> is small, and those utilities are trivial. Even when I go to the "origins",
> they are lost or changed a lot. Clearly not the same origin.
Even if they have changed a lot, that could still be an issue. Copying an
e
But I found a potential issue that it's possible StreamPark copied some
code from flink-sql-gateway to streampark-flink-sql-gateway.
If that's the case, we should convey this info in the LICENSE file. But
since Flink is also an ASF project, we don't need to convey another copy of
ALv2 and no need
> It looks like additional third-party code might also be in the release,
but it is difficult to tell.
Yeah. The result gives a lot of noise. I run SCANOSS locally, and don't
find any other third-party code included:
* Many of the reference to streamx is the original project before
StreamPark don
> vue-vben-admin is under MIT[1] license, In the LICENST[2] file of
> StreamPark, we listed which files are copied from vue-vben-admin
The issue here is that, as MIT license writes:
> The above copyright notice and this permission notice shall be included
in all copies or substantial portions of
> So a condition of including MIT licensed code is to include the relevant
MIT license text. That seems to be missing for for vue-vben-admin , as I
can’t find it anywhere in the release
vue-vben-admin is under MIT[1] license, In the LICENST[2] file of
StreamPark, we listed which files are copied f
Here is a draft that participants in this thread can review: [1]
[1] https://github.com/apache/incubator-streampark/pull/3689
Best,
tison.
tison 于2024年4月29日周一 16:01写道:
> > So a condition of including MIT licensed code is to include the
> relevant MIT license text.
>
> Confirmed this is the ca
> So a condition of including MIT licensed code is to include the relevant
MIT license text.
Confirmed this is the case.
I found the mention to vue-vben-admin is licensed under MIT at [1] but I
don't find the LICENSE of vue-vben-admin bundled also.
I agree that the LICENSE file at [2] seems to b
Hi,
-1 (binding) from me
I checked:
- incubating in name
- signatures and hashes correct
- disclaimer exists
- LICENSE is missing some info on the MIT license
- NOTICE looks fine
- I didn't compile from source
So a condition of including MIT licensed code is to include the relevant MIT
license
18 matches
Mail list logo