Skip to content

Copy transcoding code#6

Merged
qiwzhang merged 4 commits intoistio:masterfrom
qiwzhang:transcoding
Dec 6, 2016
Merged

Copy transcoding code#6
qiwzhang merged 4 commits intoistio:masterfrom
qiwzhang:transcoding

Conversation

@qiwzhang
Copy link
Contributor

@qiwzhang qiwzhang commented Dec 6, 2016

No description provided.

@qiwzhang qiwzhang assigned kyessenov and louiscryan and unassigned kyessenov Dec 6, 2016
@@ -0,0 +1,77 @@
// Copyright (C) Extensible Service Proxy Authors
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change license header.

#
################################################################################
#
package(default_visibility = ["//visibility:private"])
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make it public or same as api_manager?

@lizan
Copy link
Contributor

lizan commented Dec 6, 2016

You should keep BULD file, and change protobuf_bzl to protobuf_git, it should work.

@qiwzhang
Copy link
Contributor Author

qiwzhang commented Dec 6, 2016

Not easy to make it bazel buildable. Need to change many includes too. We can do that later. For now, not to add BUILD as not to break build for other codes.

Copy link
Contributor

@lizan lizan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please open an issue for making this buildable.

@qiwzhang qiwzhang merged commit 6a92ee4 into istio:master Dec 6, 2016
@qiwzhang qiwzhang deleted the transcoding branch December 6, 2016 23:30
duderino pushed a commit that referenced this pull request Aug 13, 2019
fpesce pushed a commit that referenced this pull request Dec 10, 2019
kyessenov pushed a commit to kyessenov/proxy that referenced this pull request Jul 12, 2022
YaoZengzeng pushed a commit to YaoZengzeng/proxy that referenced this pull request Jun 28, 2024
sync with the master of istio proxy
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.

4 participants