Why do we depend on python3 but install golang? I can relate to those who don't want random binaries installed and executed on their system.
Kind Regards, Brandon On Thu, Apr 24, 2025 at 3:39 AM Mick Semb Wever <m...@apache.org> wrote: > > it's for the website docs. these are in-tree under doc/ > there's stuff that's auto-generated. e.g. nodetool, yaml, native protocol > pages from their corresponding classes. > > it was an oversight to not have raised a discuss thread for the introduction > of golang into the build system. this was only discussed within > CASSANDRA-19915 > > > On Thu, 24 Apr 2025 at 10:33, Alex Petrov <al...@coffeenco.de> wrote: >> >> Could you elaborate how specifically having a generate docs task on by >> default is helping newcomers, and how code changes break docs? Also, are we >> talking about javadoc or documentation website? >> >> On Thu, Apr 24, 2025, at 8:25 AM, Mick Semb Wever wrote: >> >> Does this also apply to gradle, which now gets downloaded and installed, and >> is the most recent addition ? >> >> The python requirement from gen-doc has been around for over three years now. >> >> I agree with the rationale that `ant` should default to `ant check`, keeping >> newcomers in mind while being more just a final pre-commit action for >> seasoned devs where `ant jar` is the typical dev loop. I'm sure this has >> been discussed in the past. >> >> >> >> On Thu, 24 Apr 2025 at 08:16, Alex Petrov <al...@coffeenco.de> wrote: >> >> >> I didn’t say we should disable checks. I thought this was a purpose of CI : >> to check everything: there’s likely more to break semantically anyways, also >> harder to detect and fix. I’m also not proposing removing something that was >> long in place, these dependencies were introduced very recently. >> >> On Wed, Apr 23, 2025, at 11:51 PM, Jeremiah Jordan wrote: >> >> I think the default build should be to build and check everything. I think >> that if someone is new it is better to have everything built and checked by >> default to flag issues. >> >> If someone knows what they are doing and wants to speed up the process it is >> very easy to add the right settings to the ant command so things are faster. >> >> -Jeremiah >> >> On Wed, Apr 23, 2025 at 4:36 PM Jordan West <jorda...@gmail.com> wrote: >> >> Should we consider making that the default and then passing false explicitly >> in CI/builds? I agree with Alex it’s a bit surprising and shorter build >> times when developing would be helpful. >> >> Jordan >> >> On Wed, Apr 23, 2025 at 13:37 Mick Semb Wever <m...@apache.org> wrote: >> >> Python and Go are used by the gen-doc target. >> >> Code changes can break these, hence it is part of `ant check`. >> It is not called by `ant jar` >> >> If you want to run check but skip it, it's to add `-Dant.gen-doc.skip=true` >> >> >> >> On Wed, 23 Apr 2025 at 22:06, Alex Petrov <al...@coffeenco.de> wrote: >> >> >> Hi folks, >> >> Building Cassandra jar has been getting increasingly slow, and now it looks >> like we depend not only on python3 (which was already not optimal), but also >> on go: >> >> ant -Dno-checkstyle=true >> >> ... >> >> [exec] python3 ./scripts/gen-nodetool-docs.py >> [exec] python3 ./scripts/convert_yaml_to_adoc.py ../conf/cassandra.yaml >> ./modules/cassandra/pages/managing/configuration/cass_yaml_file.adoc >> [exec] ./scripts/process-native-protocol-specs-in-docker.sh >> [exec] Go env not found in your system, proceeding with installation. >> [exec] Downloading Go 1.23.1... >> [exec] Installing Go 1.23.1... >> [exec] Building the cqlprotodoc... >> [exec] Cloning into 'cassandra-website'... >> [exec] Your branch is up to date with 'origin/trunk'. >> [exec] go: downloading github.com/mvdan/xurls v1.1.0 >> [exec] Processing the .spec files... >> >> I personally consider this extremely dangerous, but also unnecessary. My >> current stance is that functionality introducing python3 and go should be >> moved to a separate task that only runs on demand / ci / release. I welcome >> convincing arguments that would suggest otherwise. >> >> If you agree we should not require python and go to run `ant >> -Dno-checkstyle=true`, please also write a short message, this will be very >> helpful as well. >> >> Thank you, >> --Alex >> >> >>