From 059d3052056b72c50ec80d8c9d356c950503057b Mon Sep 17 00:00:00 2001 From: Alon-Lukatch-Starkware Date: Wed, 30 Oct 2024 11:29:34 +0200 Subject: [PATCH] chore: clearify messages on protoc usage in papyrus_protobuf (#1565) --- crates/papyrus_protobuf/build.rs | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/crates/papyrus_protobuf/build.rs b/crates/papyrus_protobuf/build.rs index 23166bfc9c..f6da55625d 100644 --- a/crates/papyrus_protobuf/build.rs +++ b/crates/papyrus_protobuf/build.rs @@ -40,12 +40,15 @@ fn main() -> io::Result<()> { // If Protoc is installed use it, if not compile using prebuilt protoc. println!("Building"); if get_valid_preinstalled_protoc_version().is_none() { - println!("Protoc is not installed. Downloading it before building"); + println!( + "Protoc is not installed. Adding a prebuilt protoc binary via gh actions before \ + building." + ); let (protoc_bin, _) = protoc_prebuilt::init("27.0").expect( - "Please run `gh auth login` to enable protoc compilation.\n + "Build failed due to Github's rate limit. Please run `gh auth login` to lift the rate limit and allow protoc compilation to proceed. \ If this issue persists please download Protoc following the instructions at https://github.com/starkware-libs/sequencer/blob/main/docs/papyrus/README.adoc#prerequisites", ); - println!("Finished downloading Protoc. Now building"); + println!("Prebuilt protoc added to the project."); env::set_var("PROTOC", protoc_bin); } prost_build::compile_protos(