Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove useless file that doesn't load #275

Merged
merged 1 commit into from Apr 8, 2015
Merged

Remove useless file that doesn't load #275

merged 1 commit into from Apr 8, 2015

Conversation

tamird
Copy link
Contributor

@tamird tamird commented Apr 8, 2015

This file was broken in ada6556.

@haberman this accounts for the problems I was seeing in #165.

@haberman
Copy link
Member

haberman commented Apr 8, 2015

Oh, yes it looks like that is a remnant of the (now-deleted) v1 of the C++ extension!

This should definitely be deleted. But I can't merge yet because it looks like this change breaks the build. Looks like this filename needs to be removed from one more place first:
https://github.com/google/protobuf/blob/master/Makefile.am#L205

@tamird
Copy link
Contributor Author

tamird commented Apr 8, 2015

@haberman thanks! Should be good now

@tamird
Copy link
Contributor Author

tamird commented Apr 8, 2015

@haberman green :)

haberman added a commit that referenced this pull request Apr 8, 2015
Remove useless file that doesn't load
@haberman haberman merged commit 8750f72 into protocolbuffers:master Apr 8, 2015
@haberman
Copy link
Member

haberman commented Apr 8, 2015

Thanks!

@tamird tamird deleted the remove-useless-file branch April 9, 2015 04:29
taoso pushed a commit to taoso/protobuf that referenced this pull request Aug 1, 2018
taoso pushed a commit to taoso/protobuf that referenced this pull request Aug 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants