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

Some improvements #42

Closed
wants to merge 13 commits into from
Closed

Some improvements #42

wants to merge 13 commits into from

Conversation

fgnm
Copy link
Contributor

@fgnm fgnm commented Jan 1, 2022

Hi,
These are the improvements I made to get HyperLap2D working with TeaVM backend:

  • Built with TeaVM 0.7.0 - 1200
  • Add artemis-odb reflection support
  • Add WebBuildConfiguration#getReflectionInclude and WebBuildConfiguration#getReflectionExclude to automatically scan classes and add to reflection when AdditionalClasspath are provided
  • Changed some hard coded windows path to make them working on linux too (For instance "C:\TeaVMCache" now points to system default temp directory with System.getProperty("java.io.tmpdir"))
  • Add HyperLap2DTest
  • Removed the jar file you commit because when switching from Windows and Linux jar is somehow corrupted

Thanks :)

@xpenatan
Copy link
Owner

xpenatan commented Jan 1, 2022

Hi, thanks.

I see that there are some classes with line ending problem. For example, there are no changes in bullet extension but its showing as a changed file.

@fgnm
Copy link
Contributor Author

fgnm commented Jan 1, 2022

Ah yes, changed automatically by the system (I use Ubuntu).. But with this formatting there shouldn't be problems with sharing code between Windows and Linux.. I could revert if it's a issue for you

@xpenatan
Copy link
Owner

xpenatan commented Jan 1, 2022

Hi, please make a pull to get the changes I made.

I think I fixed the line ending problem. There were some files with windows line ending so I converted everything to LF.

About the currupted jar file, I think git checkout was messing with it. I added to ignore in .gitattributes. I also removed eclipse files so its gradle only.

fgnm added 4 commits January 1, 2022 17:22
Add an easy way to include/exclude classes to Tea reflection
# Conflicts:
#	.gitignore
#	extensions/gdx-bullet/gdx-bullet/.settings/org.eclipse.jdt.ui.prefs
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.

2 participants