incubator-general mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Justin Mclean <jus...@classsoftware.com>
Subject Re: [VOTE] Release Apache SkyWalking (incubating) version 5.0.0-RC2
Date Sat, 08 Sep 2018 22:50:33 GMT
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-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org


Mime
View raw message