Hi, Dave

After spending some time in Apache Rat, I realised what stopped Weex from
using Apache Rat.

Note: these patterns are applied to single files or directories, not full
> paths
> coverage/* will ignore any coverage dir, but airflow/www/static/coverage/*
> will match nothing
>

.rat-exclude file works in a way that I never expected [1] , but I shall
fix it in master branch of Weex.

[1] https://github.com/apache/airflow/blob/master/.rat-excludes#L1-L2

Best Regards,
YorkShen

申远


YorkShen <[email protected]> 于2019年10月22日周二 上午10:20写道:

> Hi, Dave
>
>
>> Consider adding exclusions for the two sets of files that are in the
>> LICENSE file and shown as on-standard in the Rat check output.
>
>
> This is something I want to fix long time ago, but I didn't find any
> documentation about how to add exclusion for RAT.
>
> FYI: I always execute RAT with the following command in my computer, so I
> am a little confused about how to add exclusion rules for RAT, maybe adding
> something like .gitignore file?
>
> *java -jar apache-rat-0.12.jar $DIRECTORY*
>
>
> Best Regards,
> YorkShen
>
> 申远
>
>
> Dave Fisher <[email protected]> 于2019年10月22日周二 上午3:52写道:
>
>> +1 (binding)
>>
>> Incubating in name
>> Signature and Checksum are correct
>> LICENSE is good
>> NOTICE is good
>> DISCLAIMER is good
>> Rat check passes. Consider adding exclusions for the two sets of files
>> that are in the LICENSE file and shown as on-standard in the Rat check
>> output.
>>
>> Build did fail when it got to Android, but then I did not install an
>> Android environment.
>>
>> Regards,
>> Dave
>>
>> > On Oct 17, 2019, at 5:46 AM, YorkShen <[email protected]> wrote:
>> >
>> > Forget to send prvious email from Apache email account, sorry for that.
>> But
>> > let‘s continue voting.
>> >
>> > Best Regards,
>> > YorkShen
>> >
>> > 申远
>> >
>> >
>> > 申远 <[email protected]> 于2019年10月17日周四 下午8:33写道:
>> >
>> >> Hi, folks
>> >>
>> >> The Apache Weex community has voted and approved the proposal to
>> release
>> >> Apache Weex (Incubating) version 0.28.0-RC1, we now kindly request the
>> >> Incubator PMC members to review and vote on this incubator release.
>> >>
>> >>   - Vote thread:
>> >>
>> https://lists.apache.org/thread.html/3b56f8ceb75981487de107c90995de27ebfa9ff9acfb0e16ca07cc55@%3Cdev.weex.apache.org%3E
>> >>   - Vote result thread:
>> >>
>> https://lists.apache.org/thread.html/7df08b9020fc11c0cb01b15faea16207cb791314ef993ab73a9b2721@%3Cdev.weex.apache.org%3
>> >>   <
>> https://lists.apache.org/thread.html/7df08b9020fc11c0cb01b15faea16207cb791314ef993ab73a9b2721@%3Cdev.weex.apache.org%3E
>> >
>> >>   - Git tag for this Release:
>> >>   https://github.com/apache/incubator-weex/releases/tag/0.28.0-RC1
>> >>   - The source tarball could be found at :
>> >>
>> https://dist.apache.org/repos/dist/dev/incubator/weex/0.28.0/RC1/apache-weex-incubating-0.28.0-RC1-src.tar.gz
>> >>   - The signature of the source tarball could be found at :
>> >>
>> https://dist.apache.org/repos/dist/dev/incubator/weex/0.28.0/RC1/apache-weex-incubating-0.28.0-RC1-src.tar.gz.asc
>> >>   - The SHA-512 checksum of the source tarball could be found at :
>> >>
>> https://dist.apache.org/repos/dist/dev/incubator/weex/0.28.0/RC1/apache-weex-incubating-0.28.0-RC1-src.tar.gz.sha512
>> >>   - The source tarball is signed with
>> >>   Key:D7E6B58FFA293FB1AC9E7B624FD8BC09C7E9DB81, which could be found
>> in the
>> >>   key file:
>> >>   https://dist.apache.org/repos/dist/release/incubator/weex/KEYS
>> >>   - ChangeLog about this version:
>> >>
>> https://github.com/apache/incubator-weex/blob/release/0.28/CHANGELOG.md
>> >>
>> >>
>> >> One can build the binary from source according to
>> >>
>> https://github.com/apache/incubator-weex/blob/release/0.28/HOW-TO-BUILD.md
>> >> with build environment installed.
>> >>
>> >> *FYI: I only test the build scripts on Mac environment, it should also
>> >> work in Linux/Unit platform according to my understanding. But you may
>> get
>> >> problems if you try to build Weex from source on a windows platform.*
>> >>
>> >> This vote will remain open for at least 72 hours, until we get enough
>> >> votes. Please vote on releasing this RC.
>> >>
>> >>   -  +1 approve
>> >>   -  +0 no opinion
>> >>   -  -1 disapprove (and reason why)
>> >>
>> >>
>> >> Best Regards,
>> >> YorkShen
>> >>
>> >> 申远
>> >>
>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: [email protected]
>> For additional commands, e-mail: [email protected]
>>
>>

Reply via email to