Skip to content

gtfs-realtime.proto does not compile in protoc - #700

Open
stevenb-mcgrath wants to merge 4 commits into
MobilityData:mainfrom
stevenb-mcgrath:main
Open

gtfs-realtime.proto does not compile in protoc#700
stevenb-mcgrath wants to merge 4 commits into
MobilityData:mainfrom
stevenb-mcgrath:main

Conversation

@stevenb-mcgrath

Copy link
Copy Markdown
 protoc --dependency_out="C:\Users\Steven\...\deps.d" --csharp_out="C:\Users\Steven\..." --proto_path=. --proto_path="C:\Users\Steven\...protoc\include" "TransitRealtime.FeedMessage.proto"
TransitRealtime.FeedMessage.proto:630:22: "active_period" is already defined in "transit_realtime.Alert".
TransitRealtime.FeedMessage.proto:630:38: Field number 1 has already been used in "transit_realtime.Alert" by field "active_period". Next available field number is 4.

gtfs-realtime.proto does not compile in protoc because of duplicate Alert.active_period fields
Update gtfs-realtime.proto - remove duplicate Alert.active_period
Update gtfs-realtime.proto - remove stray whitespace
@CLAassistant

CLAassistant commented Aug 6, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@stevenb-mcgrath

Copy link
Copy Markdown
Author

The issue was introduced in commit 9ad360a here: 9ad360a#diff-b0a93da1d2b3f799d4b604c4dfe28faa6cd89ff8d9eac96f9855270c1e5b4974

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants