Thank you Brennan for your work! > Found an unexpected comped fine in the source release [1]
I think that file shouldn't be there in the first place since it's generated by U-Boot mkimage tool. It slipped with a PR and the ignore file didn't come until few days ago. We can still exclude it from the tarballs with ./tools/zipme.sh -e "uImage" On Fri, Apr 24, 2020 at 8:23 AM Justin Mclean <jus...@classsoftware.com> wrote: > > Hi, > > I checked: > - incubating in artefact names > - signatures and hashes fine > - LICENSE and NOTICE are OK > - Found an unexpected comped fine in the source release [1] > - ASF files don’t have correct headers > - Had some issues compiling > > From the ASF side there some minor things, most of them are likely to get a > -1 vote with a WIP disclaimer. One may do as some IPMC members may question > the lack of ASF headers without that being explicitly called out. > > For the Nuttx package: > - It's a good idea to publish your keys to a public server > - It’s a good idea to sign it with your apache.org email address > - It would be nice to see more than " * Releases may have incomplete > licensing conditions.” in the WIP disclaimer > - In NOTICE I’d move "This product includes software developed at” to the top > - There is a large number of files without ASF headers that have been donated > to the ASF (this should also be mentioned in the disclaimer) > - The source release contains what looks to be compiled code [1] (I would > also mention this in the disclaimer) > - LICENSE is missing some licenses (again should be mentioned in the > disclaimer) > - README needs updated e.g "Download and unpack” section, the word Apache is > not used once even though it’s now teh project is “Apache Nuttx (incubating)” > not “Nuttx”, and you usually have some pointers to the community where you > can go for for help or to submit patches. "Downloading from Repositories” is > also still referring to bitbucket. > > Some of the above also applies to for the app package as well. > > What a good board to try on OSX? I tried a few at random that are popular or > that I know but couldn't get the config script to work. > > Thanks, > Justin > > > 1. ./boards/arm/sama5/sama5d3-xplained/boot/uImage
