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

Update Main.java #49

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update Main.java #49

wants to merge 1 commit into from

Conversation

leo-on-hill
Copy link

fixed: The invalid import of PluginProtos caused compile errors

fixed: The invalid import of PluginProtos caused compile errors
@leo-on-hill
Copy link
Author

PROTOBUF_VERSION=2.6.1

@Atry
Copy link
Owner

Atry commented Aug 23, 2015

Could you please update the build script as well?

@mroohian
Copy link

hi, is it also possible to compile against v3 of protobuf with this fix?

@pm-nsimic
Copy link

I get this error when i try to run with 2.6.1 (compilation works):

../protobuf-2.6.1/src/protoc \
    --plugin=protoc-gen-as3=dist/protoc-gen-as3 \
    "--proto_path=../protobuf-2.6.1/src" \
    --as3_out=descriptor.proto.as3 \
    "../protobuf-2.6.1/src/google/protobuf/descriptor.proto"
Error: A JNI error has occurred, please check your installation and try again
Exception in thread "main" java.lang.NoClassDefFoundError: com/google/protobuf/Extension

@Atry
Copy link
Owner

Atry commented Apr 16, 2016

Sorry but I don't understand the purpose of your guys.

Isn't the published binary of this plugin works with protoc 2.6.x ?

@johndpope
Copy link

johndpope commented Nov 4, 2016

probably best to include a Gemfile with hardcoded protoc version

Gemfile
source 'https://rubygems.org'
gem "protobuf", "2.6.1"

Ruby Version Manager

curl -sSL https://get.rvm.io | bash 
rvm install ruby-2.2.3 
gem install rvm  (restart terminal)
gem install bundler  
bundle install
bundle exec pod install

then use
bundle exec protoc --plugin=protoc-gen-as3=path/to/protoc-gen-as3[.bat]--as3_out=output-path your.proto
(which will use older version of protoc)

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.

5 participants