search

eclipse-uprotocol / up-rust

uProtocol Language Specific Library for Rust
Apache License 2.0
14 stars 9 forks source link

Removing upayload.proto from build.rs #107

Closed PLeVasseur closed 5 months ago

PLeVasseur commented 5 months ago

Just ran into this at the end of the day today, but since upayload.proto is now gone, we have to remove this line, I think.

We're definitely going to get a nightly failure again haha.

If not handled in the morning I can fix it, but if done sooner, I wouldn't mind either :smiley: 

error: failed to run custom build command for `up-rust v0.1.5 (/mnt/a24b4262-23c6-4bf0-9346-fd9021e50551/aosp/external/rust/uprotocol-rust_fork)`

Caused by:
  process didn't exit successfully: `/mnt/a24b4262-23c6-4bf0-9346-fd9021e50551/aosp/external/rust/uprotocol-rust_fork/target/debug/build/up-rust-06d2e771912b41d9/build-script-build` (exit status: 1)
  --- stderr
  [libprotobuf WARNING ../../../../../src/google/protobuf/compiler/parser.cc:649] No syntax specified for the proto file: upayload.proto. Please use 'syntax = "proto2";' or 'syntax = "proto3";' to specify a syntax version. (Defaulted to proto2 syntax.)
  upayload.proto:1:1: Expected top-level statement (e.g. "message").
  codegen failed: parse and typecheck
AnotherDaniel commented 5 months ago

There's a lot more that needs to change. I'm on that right now...

sophokles73 commented 5 months ago

fixed via #108