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

154 eea files #48

Open
numeralnathan opened this issue Oct 20, 2017 · 8 comments
Open

154 eea files #48

numeralnathan opened this issue Oct 20, 2017 · 8 comments

Comments

@numeralnathan
Copy link

Here are 154 eea files. Enjoy!

External Annotations.zip

@sylvainlaurent
Copy link
Member

could you create one or several pull requests?
thanks

@vorburger
Copy link
Member

@nathanila OMG, wow - thanks a lot! This is the biggest lot so far, so I'm (mostly) "just curious" - where are these from? Auto. gen. by some tooling, or hand written? By yourself or someone else, who gave you permission to contribute it to this project? If you have the time to create a real PR for us instead of attaching a ZIP, that would be great just for "traceability" in Git of who contributed what when how...

@numeralnathan
Copy link
Author

I wrote a tool that reads the class file and outputs the eea lines without the 0 or 1 notations. I then go back through the Java source code and add the 0 or 1 notations. This makes it much faster to get to the final version of the eea.

I created these eea files over the past year or so.

Every time I make file an issue on github, everyone asks me to create a PR. Every time I think, "I do not know how and I do not want to take the time to figure it out." It is probably painfully easy to do.

@vorburger
Copy link
Member

@nathanila Thanks for the clarification. I gather that these are created by you alone (not based on other annotation metadata which could have had had an incomptible license, not taken from anywhere), and that by uploading your ZIP here to this issue you are proposing them for inclusion in this project. So no problem from my side for any other contributor to just take your ZIP and make a PR from it - if you cannot?

But we encourage you to learn more about Git and GitHub PRs - it's a great collaboration tool and will "unlock" you to be able to more actively directly participate in many open source communities - which is lots of fun and rewarding! In case you are interested and want to pursue but need a starting point, here are some useful pointers:

PS: Re. a tool, I've had some ideas that I think are somewhat similar to what you have probably done, see our discussion on #16 and my (so far empty) shell project on https://github.com/lastnpe/external-annotations-esperanto .. let us know if you want to join the party and help out more based on your experience with this!

@numeralnathan
Copy link
Author

I created these eea files. There are no incompatible licenses. I am proposing these eea files for including in this project. I have no problem with any one to just take my ZIP and make a PR from it.

Bananeweizen added a commit to Bananeweizen/eclipse-null-eea-augments that referenced this issue Nov 11, 2017
This is the merged ZIP file contribution for the Java runtime. All
changes in java/java have been reviewed by me (and only 4 of the
suggested changes were rejected by me). Changes in java/javax have not
been reviewed.
@vorburger
Copy link
Member

Closing this as we're now getting this in via #56 - thanks to @Bananeweizen !

@vorburger vorburger reopened this Nov 13, 2017
@vorburger
Copy link
Member

Actually hang on - the #56 only takes the EEA for JDK from this ZIP, right? There's more in it - any takers?

slf4j, Jackson, Apache HTTP Client, Glassfish HTTP, Freemarker, JSoup.

@Bananeweizen
Copy link
Contributor

I can convert those too. However, I will only be able to do some real testing and review on slfj and Freemarker, since I'm not using the other libraries.

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

No branches or pull requests

4 participants