Skip to content

https://github.com/jabrena/ev3dev-lang-java/issues/6 - #134

Closed
jabrena wants to merge 1 commit into
ev3dev:developfrom
ev3dev-lang-java:develop
Closed

https://github.com/jabrena/ev3dev-lang-java/issues/6#134
jabrena wants to merge 1 commit into
ev3dev:developfrom
ev3dev-lang-java:develop

Conversation

@jabrena

Copy link
Copy Markdown
Contributor

Hi, this is my first pull request about the files which the content is generated by Autogen.

The file file: EV3DevConstants.java will be used by other Java classes.

This pull request is related with this issue:
ev3dev-lang-java/ev3dev-lang-java#6

Cheers

Juan Antonio

@rhempel

Copy link
Copy Markdown
Member

@jabrena - we're going to skip this PR for now. Kernel release is coming up and I want to get the new spec.json as the baseline. Your API generator will likely need some minor adjustments.

@WasabiFan

Copy link
Copy Markdown
Member

To be clear, we are interested in the library -- but it'd be best for us to wait a bit so that we can get the main repo updated and ready for kernel changes. I'd expect that these changes could require some changes on your end as well as we update the spec (but nothing too major, code-wise).

@rhempel

Copy link
Copy Markdown
Member

Yes, we are definitely interested in the C# binding - sorry if that was not clear.

@jabrena

Copy link
Copy Markdown
ContributorAuthor

@rhempel
Yes of course, when you release the new kernel, I will update the template development.

@dlech

Copy link
Copy Markdown
Member

The kernel has been released. http://www.ev3dev.org/news/2015/12/15/Kernel-Release-Cycle-9/

@jabrena

Copy link
Copy Markdown
ContributorAuthor

Fun this weekend!!

This week, i read some new issues about motors. I will continúe with testing. I will make some questions.

Cheer mates.

Juan Antonio

@jabrena

Copy link
Copy Markdown
ContributorAuthor

Hi @dlech,

is the package available to upgrade in the brick to continue testing?

@dlech

Copy link
Copy Markdown
Member

Yes. sudo apt-get update; sudo apt-get upgrade; sudo apt-get dist-upgrade.

@WasabiFan

Copy link
Copy Markdown
Member

@jabrena We recently changed the structure of our repo to make it so that, aside from adding your language to the list of language comment formats, you don't have to modify the main repo to be able to add and autogen your code. See #142 for more information.

@jabrena

Copy link
Copy Markdown
ContributorAuthor

@WasabiFan Oki

I am going to close this pull request and follow the notes from #142

@jabrenajabrena closed this Jan 7, 2016
Sign up for freeto 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

@jabrena@rhempel@WasabiFan@dlech