Hi,

Sorry but I still cannot vote +1 own this as the README.md still points to 
https://github.com/apache/incubator-skywalking/blob/master/docs/en/How-to-build.md
 for build instructions which gives a 404, please include the instructions on 
how the build the source in the release in the README.md or in a BUILD.md (or 
similarly named file) in the base directory.

Try to think like a user of this software they download it and want to compile 
it, they read instructions, are asked to follow a link (annoying) and then the 
link gives a 404, at this point some of them are going to give up and you will 
never hear from them again. Sure some may be able to find How-to-build.md in 
the docs and get it working but their first experience of the project is not 
ideal. Releases need to include simple easy to follow instructions on how to 
build. Those instruction may changed form release to release so pointing people 
to URLs is not helpful. What for instance if you want to compile an older 
version and the URL only has instructions for the most recent version?

I checked:
- incubating in name
- signatures and hashes good
- DISCLAIMER exists
- LICENSE and NOTICE good
- no unexpected binary files
- all source files have ASF headers
- unable to build from source

Thanks for fixing the license issue, however you also need to fix it in the 
binary release.

Thanks,
Justin
---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org

Reply via email to